Re: [PATCH] iio: accel: kxcjk-1013: Add the "KXJ2109" ACPI HID

[Stephan Gerhold <stephan@gerhold.net>]

On Sun, Nov 25, 2018 at 09:09:30AM +0000, Jonathan Cameron wrote:
> On Sat, 24 Nov 2018 17:22:20 +0100
> Stephan Gerhold <stephan@gerhold.net> wrote:
> 
> > This HID is used on the ASUS MeMO Pad 7 (ME176C) tablet.
> > 
> > Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
> Other than a distinct temptation to rant about formats
> of ACPI IDs (4 characters the a number), looks good.
> 
> For reference (I was waiting for a build so decided to dig
> it out ;)
> ACPI 6.2 spec. 6.5.1
> 
> "A valid ACPI ID must be of the form "NNNN####" where N is an uppercase letter or a
> digit ('0'-'9') and # is a hex digit." is the text about a manufacturer defined
> ID.
> 
> If anyone has a contact at ASUS who I can rant at do pass it on!

I didn't realize the ACPI ID is technically invalid, but I have to admit 
I'm not very surprised about this: The DSDT of this tablet is absolutely 
horrible - there is a lot more I could rant about :)

(The tablet was originally shipped with Android; rather than fixing
their firmware, they chose to hardcode hacks and workarounds for many 
things in their heavily patched Linux kernel...)

The sensors are one of the few things that work out of the box
without additional workarounds (except adding the ACPI ID)...

> 
> Applied to the togreg branch of iio.git and pushed out as
> testing for the autobuilders to play with it.

Thanks!

> 
> Thanks,
> 
> Jonathan
> 
> > ---
> >  drivers/iio/accel/kxcjk-1013.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/iio/accel/kxcjk-1013.c b/drivers/iio/accel/kxcjk-1013.c
> > index af53a1084ee5..cc5b2beb1328 100644
> > --- a/drivers/iio/accel/kxcjk-1013.c
> > +++ b/drivers/iio/accel/kxcjk-1013.c
> > @@ -1491,6 +1491,7 @@ static const struct acpi_device_id kx_acpi_match[] = {
> >  	{"KXCJ9000", KXCJ91008},
> >  	{"KIOX000A", KXCJ91008},
> >  	{"KXTJ1009", KXTJ21009},
> > +	{"KXJ2109",  KXTJ21009},
> >  	{"SMO8500",  KXCJ91008},
> >  	{ },
> >  };
>