Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1667723

Re: [PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for PoV mobii wintab p800w

Show all headers | View raw

On Thu, Jun 15, 2017 at 7:53 PM, Darren Hart <dvhart@infradead.org> wrote:
> On Thu, Jun 15, 2017 at 08:48:31AM +0200, Hans de Goede wrote:

>> +             /* Point of View mobii wintab p800w */
>> +             .driver_data = (void *)&pov_mobii_wintab_p800w_data,
>> +             .matches = {
>> +                     DMI_MATCH(DMI_BOARD_VENDOR, "AMI Corporation"),
>> +                     DMI_MATCH(DMI_BOARD_NAME, "Aptio CRB"),
>> +                     DMI_MATCH(DMI_BIOS_VERSION, "3BAIR1013"),
>> +                     /* Above matches are too generic, add bios-date match */
>> +                     DMI_MATCH(DMI_BIOS_DATE, "08/22/2014"),
>
> This is the first time I've seen a BIOS date match used to determine hardware
> features. DMI matching is a (necessary) hack to begin with (the vendors should
> be providing this data via ACPI _DSD anyway) but a date match means we would
> need a kernel patch every time one of these tablets gets a BIOS update...
>
> With words like "Aptio CRB" it's clear the vendor isn't doing their job and just
> using unmodified reference code. The problem with this of course is that the
> vendor is not providing a way to identify this hardware.
>
> Andy, I'd appreciate your thoughts on this... I'm leaning towards not accepting
> bios date (or indeed, BIOS version) as a way to identify a platform.

The question is what is the anticipated amount of affected devices
with BIOS date included and otherwise?

If Hans believes that there will be no update for some devices, while
there are devices with the same DMI strings, but different date and
_fixed_ issue, I think we have no other choice for now.
Also can we use some other strings to distinguish group of devices
which are affected?

In pinctrl-cherryview.c we faced same issue, though for now there all
platforms are affected (first approach was to distinguish with BIOS
date) and we decide to go with black and white lists (white is empty
for now and will be created whenever we will have an update).

-- 
With Best Regards,
Andy Shevchenko

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for PoV mobii wintab p800w Hans de Goede <hdegoede@redhat.com> - 2017-06-15 08:50 +0200
  Re: [PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for  PoV mobii wintab p800w Darren Hart <dvhart@infradead.org> - 2017-06-15 19:00 +0200
    Re: [PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for  PoV mobii wintab p800w Hans de Goede <hdegoede@redhat.com> - 2017-06-16 14:40 +0200
    Re: [PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for  PoV mobii wintab p800w Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-06-16 14:50 +0200
      Re: [PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for  PoV mobii wintab p800w Hans de Goede <hdegoede@redhat.com> - 2017-06-16 15:30 +0200
        Re: [PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for  PoV mobii wintab p800w Darren Hart <dvhart@infradead.org> - 2017-06-17 01:40 +0200
          Re: [PATCH 1/2] platform/x86: silead_dmi: Add touchscreen info for  PoV mobii wintab p800w Hans de Goede <hdegoede@redhat.com> - 2017-06-17 12:40 +0200

csiph-web