Re: [PATCH v2] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

From: Pali RohÃr
Date: Wed Jan 31 2018 - 07:03:57 EST


On Sunday 28 January 2018 17:00:35 Andy Shevchenko wrote:
> On Sun, Jan 28, 2018 at 4:45 PM, Pali RohÃr <pali.rohar@xxxxxxxxx> wrote:
> > On Sunday 28 January 2018 16:39:25 Andy Shevchenko wrote:
> >> On Sat, Jan 27, 2018 at 3:32 PM, Pali RohÃr <pali.rohar@xxxxxxxxx> wrote:
> >> > +static bool is_dell_system_with_lis3lv02d(void)
> >> > +{
> >>
> >> > + /*
> >> > + * Check that ACPI device SMO88xx exists and is enabled. That ACPI
> >> > + * device represent our ST microelectronics lis3lv02d accelerometer but
> >> > + * unfortunately without any other information (like i2c address).
> >> > + */
> >>
> >> Isn't it simple
> >>
> >> return acpi_dev_present("SMO88", NULL, -1);
> >>
> >> call?
> >
> > ACPI device name is SMO8800, SMO8810, ... Will that acpi_dev_present
> > function match only prefix and not exact string?
>
> OK, fair enough.
>
> Do we have more users of such pattern?

I have not seen this ACPI pattern yet, so probably not.

--
Pali RohÃr
pali.rohar@xxxxxxxxx