LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Andy Shevchenko <andy.shevchenko@gmail.com>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Darren Hart <dvhart@infradead.org>,
	Andy Shevchenko <andy@infradead.org>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	"Luis R . Rodriguez" <mcgrof@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>, "H . Peter Anvin" <hpa@zytor.com>,
	Platform Driver <platform-driver-x86@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Peter Jones <pjones@redhat.com>, Dave Olsthoorn <dave@bewaar.me>,
	Will Deacon <will.deacon@arm.com>,
	Andy Lutomirski <luto@kernel.org>,
	Matt Fleming <matt@codeblueprint.co.uk>,
	David Howells <dhowells@redhat.com>,
	Mimi Zohar <zohar@linux.vnet.ibm.com>,
	Josh Triplett <josh@joshtriplett.org>,
	Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	mfuzzey@parkeon.com, Kees Cook <keescook@chromium.org>,
	Kalle Valo <kvalo@codeaurora.org>,
	Arend Van Spriel <arend.vanspriel@broadcom.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	nbroeking@me.com, Bjorn Andersson <bjorn.andersson@linaro.org>,
	Torsten Duwe <duwe@suse.de>,
	"maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)"
	<x86@kernel.org>, linux-efi <linux-efi@vger.kernel.org>
Subject: Re: [PATCH v3 3/5] platform/x86: Rename silead_dmi to touchscreen_dmi
Date: Mon, 9 Apr 2018 11:07:03 +0300	[thread overview]
Message-ID: <CAHp75Vdx0fJ0mmE1tSZjKEoHbRMubJpA8M1QYg1joKxMdLcVig@mail.gmail.com> (raw)
In-Reply-To: <20180408174014.21908-4-hdegoede@redhat.com>

On Sun, Apr 8, 2018 at 8:40 PM, Hans de Goede <hdegoede@redhat.com> wrote:
> Not only silead touchscreens need some extra info not available in the
> ACPI tables to work properly. X86 devices with a Chipone ICN8505 chip also
> need some DMI based extra configuration.
>
> There is no reason to have separate dmi config code per touchscreen
> controller vendor. This commit renames silead_dmi to a more generic
> touchscreen_dmi name (and Kconfig option) in preparation of adding
> info for tablets with an ICN8505 based touchscreen.
>
> Note there are no functional changes all code changes are limited to
> removing references to silead where these are no longer applicable.
>

I have no objections from my side, though consider the following:
- I would like to be in sync with Darren on this
- make oldconfig will be broken after your change for existing users
- the usual pattern in kernel that we don't rename drivers; I guess
here we are on the safe side b/c this driver is used standalone

Taking above into attention, and assuming it will go via some other tree,
Acked-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>  MAINTAINERS                                   |  2 +-
>  drivers/platform/x86/Kconfig                  | 16 ++---
>  drivers/platform/x86/Makefile                 |  2 +-
>  .../x86/{silead_dmi.c => touchscreen_dmi.c}   | 66 +++++++++----------
>  4 files changed, 43 insertions(+), 43 deletions(-)
>  rename drivers/platform/x86/{silead_dmi.c => touchscreen_dmi.c} (87%)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 0d5c55daeeba..99dd47e3b0dd 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12618,7 +12618,7 @@ L:      linux-input@vger.kernel.org
>  L:     platform-driver-x86@vger.kernel.org
>  S:     Maintained
>  F:     drivers/input/touchscreen/silead.c
> -F:     drivers/platform/x86/silead_dmi.c
> +F:     drivers/platform/x86/touchscreen_dmi.c
>
>  SILICON MOTION SM712 FRAME BUFFER DRIVER
>  M:     Sudip Mukherjee <sudipm.mukherjee@gmail.com>
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index 1868aab0282a..b836576f0fe4 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -1194,16 +1194,16 @@ config INTEL_TURBO_MAX_3
>           This driver is only required when the system is not using Hardware
>           P-States (HWP). In HWP mode, priority can be read from ACPI tables.
>
> -config SILEAD_DMI
> -       bool "Tablets with Silead touchscreens"
> +config TOUCHSCREEN_DMI
> +       bool "DMI based touchscreen configuration info"
>         depends on ACPI && DMI && I2C=y && TOUCHSCREEN_SILEAD
>         ---help---
> -         Certain ACPI based tablets with Silead touchscreens do not have
> -         enough data in ACPI tables for the touchscreen driver to handle
> -         the touchscreen properly, as OEMs expected the data to be baked
> -         into the tablet model specific version of the driver shipped
> -         with the OS-image for the device. This option supplies the missing
> -         information. Enable this for x86 tablets with Silead touchscreens.
> +         Certain ACPI based tablets with e.g. Silead or Chipone touchscreens
> +         do not have enough data in ACPI tables for the touchscreen driver to
> +         handle the touchscreen properly, as OEMs expect the data to be baked
> +         into the tablet model specific version of the driver shipped with the
> +         the OS-image for the device. This option supplies the missing info.
> +         Enable this for x86 tablets with Silead or Chipone touchscreens.
>
>  config INTEL_CHTDC_TI_PWRBTN
>         tristate "Intel Cherry Trail Dollar Cove TI power button driver"
> diff --git a/drivers/platform/x86/Makefile b/drivers/platform/x86/Makefile
> index 2ba6cb795338..8d9477114fb5 100644
> --- a/drivers/platform/x86/Makefile
> +++ b/drivers/platform/x86/Makefile
> @@ -78,7 +78,7 @@ obj-$(CONFIG_INTEL_SMARTCONNECT)      += intel-smartconnect.o
>  obj-$(CONFIG_PVPANIC)           += pvpanic.o
>  obj-$(CONFIG_ALIENWARE_WMI)    += alienware-wmi.o
>  obj-$(CONFIG_INTEL_PMC_IPC)    += intel_pmc_ipc.o
> -obj-$(CONFIG_SILEAD_DMI)       += silead_dmi.o
> +obj-$(CONFIG_TOUCHSCREEN_DMI)  += touchscreen_dmi.o
>  obj-$(CONFIG_SURFACE_PRO3_BUTTON)      += surfacepro3_button.o
>  obj-$(CONFIG_SURFACE_3_BUTTON) += surface3_button.o
>  obj-$(CONFIG_INTEL_PUNIT_IPC)  += intel_punit_ipc.o
> diff --git a/drivers/platform/x86/silead_dmi.c b/drivers/platform/x86/touchscreen_dmi.c
> similarity index 87%
> rename from drivers/platform/x86/silead_dmi.c
> rename to drivers/platform/x86/touchscreen_dmi.c
> index 452aacabaa8e..87fc839b28f7 100644
> --- a/drivers/platform/x86/silead_dmi.c
> +++ b/drivers/platform/x86/touchscreen_dmi.c
> @@ -1,5 +1,5 @@
>  /*
> - * Silead touchscreen driver DMI based configuration code
> + * Touchscreen driver DMI based configuration code
>   *
>   * Copyright (c) 2017 Red Hat Inc.
>   *
> @@ -20,7 +20,7 @@
>  #include <linux/property.h>
>  #include <linux/string.h>
>
> -struct silead_ts_dmi_data {
> +struct ts_dmi_data {
>         const char *acpi_name;
>         const struct property_entry *properties;
>  };
> @@ -34,7 +34,7 @@ static const struct property_entry cube_iwork8_air_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data cube_iwork8_air_data = {
> +static const struct ts_dmi_data cube_iwork8_air_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = cube_iwork8_air_props,
>  };
> @@ -48,7 +48,7 @@ static const struct property_entry jumper_ezpad_mini3_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data jumper_ezpad_mini3_data = {
> +static const struct ts_dmi_data jumper_ezpad_mini3_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = jumper_ezpad_mini3_props,
>  };
> @@ -62,7 +62,7 @@ static const struct property_entry dexp_ursus_7w_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data dexp_ursus_7w_data = {
> +static const struct ts_dmi_data dexp_ursus_7w_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = dexp_ursus_7w_props,
>  };
> @@ -77,7 +77,7 @@ static const struct property_entry surftab_twin_10_1_st10432_8_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data surftab_twin_10_1_st10432_8_data = {
> +static const struct ts_dmi_data surftab_twin_10_1_st10432_8_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = surftab_twin_10_1_st10432_8_props,
>  };
> @@ -92,7 +92,7 @@ static const struct property_entry surftab_wintron70_st70416_6_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data surftab_wintron70_st70416_6_data = {
> +static const struct ts_dmi_data surftab_wintron70_st70416_6_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = surftab_wintron70_st70416_6_props,
>  };
> @@ -107,7 +107,7 @@ static const struct property_entry gp_electronic_t701_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data gp_electronic_t701_data = {
> +static const struct ts_dmi_data gp_electronic_t701_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = gp_electronic_t701_props,
>  };
> @@ -122,7 +122,7 @@ static const struct property_entry pipo_w2s_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data pipo_w2s_data = {
> +static const struct ts_dmi_data pipo_w2s_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = pipo_w2s_props,
>  };
> @@ -137,7 +137,7 @@ static const struct property_entry pov_mobii_wintab_p800w_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data pov_mobii_wintab_p800w_data = {
> +static const struct ts_dmi_data pov_mobii_wintab_p800w_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = pov_mobii_wintab_p800w_props,
>  };
> @@ -151,7 +151,7 @@ static const struct property_entry itworks_tw891_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data itworks_tw891_data = {
> +static const struct ts_dmi_data itworks_tw891_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = itworks_tw891_props,
>  };
> @@ -165,7 +165,7 @@ static const struct property_entry chuwi_hi8_pro_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data chuwi_hi8_pro_data = {
> +static const struct ts_dmi_data chuwi_hi8_pro_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = chuwi_hi8_pro_props,
>  };
> @@ -181,7 +181,7 @@ static const struct property_entry digma_citi_e200_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data digma_citi_e200_data = {
> +static const struct ts_dmi_data digma_citi_e200_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = digma_citi_e200_props,
>  };
> @@ -198,7 +198,7 @@ static const struct property_entry onda_obook_20_plus_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data onda_obook_20_plus_data = {
> +static const struct ts_dmi_data onda_obook_20_plus_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = onda_obook_20_plus_props,
>  };
> @@ -212,7 +212,7 @@ static const struct property_entry chuwi_hi8_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data chuwi_hi8_data = {
> +static const struct ts_dmi_data chuwi_hi8_data = {
>         .acpi_name      = "MSSL0001:00",
>         .properties     = chuwi_hi8_props,
>  };
> @@ -227,7 +227,7 @@ static const struct property_entry chuwi_vi8_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data chuwi_vi8_data = {
> +static const struct ts_dmi_data chuwi_vi8_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = chuwi_vi8_props,
>  };
> @@ -242,7 +242,7 @@ static const struct property_entry trekstor_primebook_c13_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data trekstor_primebook_c13_data = {
> +static const struct ts_dmi_data trekstor_primebook_c13_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = trekstor_primebook_c13_props,
>  };
> @@ -258,7 +258,7 @@ static const struct property_entry teclast_x98plus2_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data teclast_x98plus2_data = {
> +static const struct ts_dmi_data teclast_x98plus2_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = teclast_x98plus2_props,
>  };
> @@ -272,12 +272,12 @@ static const struct property_entry teclast_x3_plus_props[] = {
>         { }
>  };
>
> -static const struct silead_ts_dmi_data teclast_x3_plus_data = {
> +static const struct ts_dmi_data teclast_x3_plus_data = {
>         .acpi_name      = "MSSL1680:00",
>         .properties     = teclast_x3_plus_props,
>  };
>
> -static const struct dmi_system_id silead_ts_dmi_table[] = {
> +static const struct dmi_system_id touchscreen_dmi_table[] = {
>         {
>                 /* CUBE iwork8 Air */
>                 .driver_data = (void *)&cube_iwork8_air_data,
> @@ -466,22 +466,22 @@ static const struct dmi_system_id silead_ts_dmi_table[] = {
>         { },
>  };
>
> -static const struct silead_ts_dmi_data *silead_ts_data;
> +static const struct ts_dmi_data *ts_data;
>
> -static void silead_ts_dmi_add_props(struct i2c_client *client)
> +static void ts_dmi_add_props(struct i2c_client *client)
>  {
>         struct device *dev = &client->dev;
>         int error;
>
>         if (has_acpi_companion(dev) &&
> -           !strncmp(silead_ts_data->acpi_name, client->name, I2C_NAME_SIZE)) {
> -               error = device_add_properties(dev, silead_ts_data->properties);
> +           !strncmp(ts_data->acpi_name, client->name, I2C_NAME_SIZE)) {
> +               error = device_add_properties(dev, ts_data->properties);
>                 if (error)
>                         dev_err(dev, "failed to add properties: %d\n", error);
>         }
>  }
>
> -static int silead_ts_dmi_notifier_call(struct notifier_block *nb,
> +static int ts_dmi_notifier_call(struct notifier_block *nb,
>                                        unsigned long action, void *data)
>  {
>         struct device *dev = data;
> @@ -491,7 +491,7 @@ static int silead_ts_dmi_notifier_call(struct notifier_block *nb,
>         case BUS_NOTIFY_ADD_DEVICE:
>                 client = i2c_verify_client(dev);
>                 if (client)
> -                       silead_ts_dmi_add_props(client);
> +                       ts_dmi_add_props(client);
>                 break;
>
>         default:
> @@ -501,22 +501,22 @@ static int silead_ts_dmi_notifier_call(struct notifier_block *nb,
>         return 0;
>  }
>
> -static struct notifier_block silead_ts_dmi_notifier = {
> -       .notifier_call = silead_ts_dmi_notifier_call,
> +static struct notifier_block ts_dmi_notifier = {
> +       .notifier_call = ts_dmi_notifier_call,
>  };
>
> -static int __init silead_ts_dmi_init(void)
> +static int __init ts_dmi_init(void)
>  {
>         const struct dmi_system_id *dmi_id;
>         int error;
>
> -       dmi_id = dmi_first_match(silead_ts_dmi_table);
> +       dmi_id = dmi_first_match(touchscreen_dmi_table);
>         if (!dmi_id)
>                 return 0; /* Not an error */
>
> -       silead_ts_data = dmi_id->driver_data;
> +       ts_data = dmi_id->driver_data;
>
> -       error = bus_register_notifier(&i2c_bus_type, &silead_ts_dmi_notifier);
> +       error = bus_register_notifier(&i2c_bus_type, &ts_dmi_notifier);
>         if (error)
>                 pr_err("%s: failed to register i2c bus notifier: %d\n",
>                         __func__, error);
> @@ -529,4 +529,4 @@ static int __init silead_ts_dmi_init(void)
>   * itself is ready (which happens at postcore initcall level), but before
>   * ACPI starts enumerating devices (at subsys initcall level).
>   */
> -arch_initcall(silead_ts_dmi_init);
> +arch_initcall(ts_dmi_init);
> --
> 2.17.0
>



-- 
With Best Regards,
Andy Shevchenko

  reply	other threads:[~2018-04-09  8:07 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-08 17:40 [PATCH v3 0/5] efi/firmware/platform-x86: Add EFI embedded fw support Hans de Goede
2018-04-08 17:40 ` [PATCH v3 1/5] efi: Export boot-services code and data as debugfs-blobs Hans de Goede
2018-04-16  8:23   ` Ard Biesheuvel
2018-04-24 13:11     ` Hans de Goede
2018-04-23 11:55   ` Greg Kroah-Hartman
2018-04-08 17:40 ` [PATCH v3 2/5] efi: Add embedded peripheral firmware support Hans de Goede
2018-04-16  8:28   ` Ard Biesheuvel
2018-04-24 13:17     ` Hans de Goede
2018-04-17  0:17   ` Luis R. Rodriguez
2018-04-17  8:58     ` Hans de Goede
2018-04-17  9:19     ` Hans de Goede
2018-04-23 21:11   ` Luis R. Rodriguez
2018-04-24 15:09     ` Hans de Goede
2018-04-24 16:07       ` Mimi Zohar
2018-04-24 18:33         ` Hans de Goede
2018-04-24 23:42         ` Luis R. Rodriguez
2018-04-25  5:00           ` Mimi Zohar
2018-04-25 17:55             ` Luis R. Rodriguez
2018-05-04  0:21               ` Luis R. Rodriguez
2018-05-04 15:26                 ` Martijn Coenen
2018-05-04 19:44               ` Martijn Coenen
2018-05-08 15:38                 ` Luis R. Rodriguez
2018-05-08 16:10                   ` Luis R. Rodriguez
2018-06-07 16:49                     ` Bjorn Andersson
2018-06-07 18:22                       ` Luis R. Rodriguez
2018-06-01 19:23                   ` Luis R. Rodriguez
2018-06-06 20:32                     ` Do Qualcomm drivers use DMA buffers for request_firmware_into_buf()? Luis R. Rodriguez
2018-06-07 16:18                       ` Bjorn Andersson
     [not found]                         ` <CAKv+Gu8+Fq7BD4XD-YCyXzZh0mg6Z2k-0styj0cw6_uZfaqy4Q@mail.gmail.com>
     [not found]                           ` <20180607163308.GA18834@kroah.com>
2018-06-08  6:41                             ` Vlastimil Babka
2018-06-07 16:33             ` [PATCH v3 2/5] efi: Add embedded peripheral firmware support Bjorn Andersson
2018-04-08 17:40 ` [PATCH v3 3/5] platform/x86: Rename silead_dmi to touchscreen_dmi Hans de Goede
2018-04-09  8:07   ` Andy Shevchenko [this message]
2018-04-20  0:20     ` Darren Hart
2018-04-08 17:40 ` [PATCH v3 4/5] platform/x86: touchscreen_dmi: Add EFI embedded firmware info support Hans de Goede
2018-04-08 17:40 ` [PATCH v3 5/5] platform/x86: touchscreen_dmi: Add info for the Chuwi Vi8 Plus tablet Hans de Goede
2018-04-09  8:10   ` Andy Shevchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAHp75Vdx0fJ0mmE1tSZjKEoHbRMubJpA8M1QYg1joKxMdLcVig@mail.gmail.com \
    --to=andy.shevchenko@gmail.com \
    --cc=andy@infradead.org \
    --cc=ard.biesheuvel@linaro.org \
    --cc=arend.vanspriel@broadcom.com \
    --cc=bjorn.andersson@linaro.org \
    --cc=dave@bewaar.me \
    --cc=dhowells@redhat.com \
    --cc=dmitry.torokhov@gmail.com \
    --cc=duwe@suse.de \
    --cc=dvhart@infradead.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=hdegoede@redhat.com \
    --cc=hpa@zytor.com \
    --cc=josh@joshtriplett.org \
    --cc=keescook@chromium.org \
    --cc=kvalo@codeaurora.org \
    --cc=linux-efi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luto@kernel.org \
    --cc=matt@codeblueprint.co.uk \
    --cc=mcgrof@kernel.org \
    --cc=mfuzzey@parkeon.com \
    --cc=mingo@redhat.com \
    --cc=nbroeking@me.com \
    --cc=pjones@redhat.com \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --cc=will.deacon@arm.com \
    --cc=x86@kernel.org \
    --cc=zohar@linux.vnet.ibm.com \
    --subject='Re: [PATCH v3 3/5] platform/x86: Rename silead_dmi to touchscreen_dmi' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).