* Fwd: [PATCH] iio: accel: kxcjk-1013: add the "KIOX000A" ACPI id [not found] <CACA7eURhTD8RzGOoYciOEp_oC2iuy+197gS7M3qnMhAEiH=v8A@mail.gmail.com> @ 2016-07-19 10:01 ` Christophe Chapuis 2016-07-19 15:58 ` Jonathan Cameron 0 siblings, 1 reply; 5+ messages in thread From: Christophe Chapuis @ 2016-07-19 10:01 UTC (permalink / raw) To: linux-iio On the Cube i9 tablet, the ACPI id for the Kionix kxcj9 accelerometer is "KIOX000A" (as can be seen in the DSDT of the Cube i9 tablet). It is the same accelerometer, so only adding the ACPI id is needed. Signed-off-by: Christophe Chapuis <chris.chapuis@gmail.com> --- 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 edec1d0..cf392c7 100644 --- a/drivers/iio/accel/kxcjk-1013.c +++ b/drivers/iio/accel/kxcjk-1013.c @@ -1393,6 +1393,7 @@ static const struct acpi_device_id kx_acpi_match[] = { {"KXCJ1013", KXCJK1013}, {"KXCJ1008", KXCJ91008}, {"KXCJ9000", KXCJ91008}, + {"KIOX000A", KXCJ91008}, {"KXTJ1009", KXTJ21009}, {"SMO8500", KXCJ91008}, { }, -- 2.7.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: Fwd: [PATCH] iio: accel: kxcjk-1013: add the "KIOX000A" ACPI id 2016-07-19 10:01 ` Fwd: [PATCH] iio: accel: kxcjk-1013: add the "KIOX000A" ACPI id Christophe Chapuis @ 2016-07-19 15:58 ` Jonathan Cameron 2016-07-19 16:11 ` Srinivas Pandruvada 0 siblings, 1 reply; 5+ messages in thread From: Jonathan Cameron @ 2016-07-19 15:58 UTC (permalink / raw) To: Christophe Chapuis, linux-iio, Srinivas Pandruvada On 19/07/16 11:01, Christophe Chapuis wrote: > On the Cube i9 tablet, the ACPI id for the Kionix kxcj9 accelerometer > is "KIOX000A" (as can be seen in the DSDT of the Cube i9 tablet). > It is the same accelerometer, so only adding the ACPI id is needed. > > Signed-off-by: Christophe Chapuis <chris.chapuis@gmail.com> Hi Christophe, Looks like it made it through fine this time! Please cc the original driver author for any patches to a driver. Here Srinivas (added) Can't see why it would be an issue, but best to make sure thy are in the loop! Funnily enough, not everyone keeps up with all the mails on the list ;) > --- > 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 edec1d0..cf392c7 100644 > --- a/drivers/iio/accel/kxcjk-1013.c > +++ b/drivers/iio/accel/kxcjk-1013.c > @@ -1393,6 +1393,7 @@ static const struct acpi_device_id kx_acpi_match[] = { > {"KXCJ1013", KXCJK1013}, > {"KXCJ1008", KXCJ91008}, > {"KXCJ9000", KXCJ91008}, > + {"KIOX000A", KXCJ91008}, > {"KXTJ1009", KXTJ21009}, > {"SMO8500", KXCJ91008}, > { }, > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Fwd: [PATCH] iio: accel: kxcjk-1013: add the "KIOX000A" ACPI id 2016-07-19 15:58 ` Jonathan Cameron @ 2016-07-19 16:11 ` Srinivas Pandruvada 2016-07-20 6:26 ` Christophe Chapuis 0 siblings, 1 reply; 5+ messages in thread From: Srinivas Pandruvada @ 2016-07-19 16:11 UTC (permalink / raw) To: Jonathan Cameron, Christophe Chapuis, linux-iio On Tue, 2016-07-19 at 16:58 +0100, Jonathan Cameron wrote: > On 19/07/16 11:01, Christophe Chapuis wrote: > > > > On the Cube i9 tablet, the ACPI id for the Kionix kxcj9 > > accelerometer > > is "KIOX000A" (as can be seen in the DSDT of the Cube i9 tablet). > > It is the same accelerometer, so only adding the ACPI id is needed. > > > > Signed-off-by: Christophe Chapuis <chris.chapuis@gmail.com> Reviewed-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> > Hi Christophe, > > Looks like it made it through fine this time! > > Please cc the original driver author for any patches to a driver. > Here > Srinivas (added) > Can't see why it would be an issue, but best to make sure thy are in > the loop! I don't see any issue with the patch. > > Funnily enough, not everyone keeps up with all the mails on the list > ;) We can blame mail filters :) > > > > --- > > 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 edec1d0..cf392c7 100644 > > --- a/drivers/iio/accel/kxcjk-1013.c > > +++ b/drivers/iio/accel/kxcjk-1013.c > > @@ -1393,6 +1393,7 @@ static const struct acpi_device_id > > kx_acpi_match[] = { > > {"KXCJ1013", KXCJK1013}, > > {"KXCJ1008", KXCJ91008}, > > {"KXCJ9000", KXCJ91008}, > > + {"KIOX000A", KXCJ91008}, > > {"KXTJ1009", KXTJ21009}, > > {"SMO8500", KXCJ91008}, > > { }, > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Fwd: [PATCH] iio: accel: kxcjk-1013: add the "KIOX000A" ACPI id 2016-07-19 16:11 ` Srinivas Pandruvada @ 2016-07-20 6:26 ` Christophe Chapuis 2016-07-24 11:33 ` Jonathan Cameron 0 siblings, 1 reply; 5+ messages in thread From: Christophe Chapuis @ 2016-07-20 6:26 UTC (permalink / raw) To: Srinivas Pandruvada; +Cc: Jonathan Cameron, linux-iio Hi, On Tue, Jul 19, 2016 at 6:11 PM, Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> wrote: > On Tue, 2016-07-19 at 16:58 +0100, Jonathan Cameron wrote: >> On 19/07/16 11:01, Christophe Chapuis wrote: >> > >> > On the Cube i9 tablet, the ACPI id for the Kionix kxcj9 >> > accelerometer >> > is "KIOX000A" (as can be seen in the DSDT of the Cube i9 tablet). >> > It is the same accelerometer, so only adding the ACPI id is needed. >> > >> > Signed-off-by: Christophe Chapuis <chris.chapuis@gmail.com> > Reviewed-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> > >> Hi Christophe, >> >> Looks like it made it through fine this time! >> >> Please cc the original driver author for any patches to a driver. >> Here >> Srinivas (added) >> Can't see why it would be an issue, but best to make sure thy are in >> the loop! > I don't see any issue with the patch. Ok great :) >> >> Funnily enough, not everyone keeps up with all the mails on the list >> ;) > We can blame mail filters :) >> > >> > --- >> > 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 edec1d0..cf392c7 100644 >> > --- a/drivers/iio/accel/kxcjk-1013.c >> > +++ b/drivers/iio/accel/kxcjk-1013.c >> > @@ -1393,6 +1393,7 @@ static const struct acpi_device_id >> > kx_acpi_match[] = { >> > {"KXCJ1013", KXCJK1013}, >> > {"KXCJ1008", KXCJ91008}, >> > {"KXCJ9000", KXCJ91008}, >> > + {"KIOX000A", KXCJ91008}, >> > {"KXTJ1009", KXTJ21009}, >> > {"SMO8500", KXCJ91008}, >> > { }, >> > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Fwd: [PATCH] iio: accel: kxcjk-1013: add the "KIOX000A" ACPI id 2016-07-20 6:26 ` Christophe Chapuis @ 2016-07-24 11:33 ` Jonathan Cameron 0 siblings, 0 replies; 5+ messages in thread From: Jonathan Cameron @ 2016-07-24 11:33 UTC (permalink / raw) To: Christophe Chapuis, Srinivas Pandruvada; +Cc: linux-iio On 20/07/16 07:26, Christophe Chapuis wrote: > Hi, > > On Tue, Jul 19, 2016 at 6:11 PM, Srinivas Pandruvada > <srinivas.pandruvada@linux.intel.com> wrote: >> On Tue, 2016-07-19 at 16:58 +0100, Jonathan Cameron wrote: >>> On 19/07/16 11:01, Christophe Chapuis wrote: >>>> >>>> On the Cube i9 tablet, the ACPI id for the Kionix kxcj9 >>>> accelerometer >>>> is "KIOX000A" (as can be seen in the DSDT of the Cube i9 tablet). >>>> It is the same accelerometer, so only adding the ACPI id is needed. >>>> >>>> Signed-off-by: Christophe Chapuis <chris.chapuis@gmail.com> >> Reviewed-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to play with it. Thanks, Jonathan >> >>> Hi Christophe, >>> >>> Looks like it made it through fine this time! >>> >>> Please cc the original driver author for any patches to a driver. >>> Here >>> Srinivas (added) >>> Can't see why it would be an issue, but best to make sure thy are in >>> the loop! >> I don't see any issue with the patch. > > Ok great :) > >>> >>> Funnily enough, not everyone keeps up with all the mails on the list >>> ;) >> We can blame mail filters :) >>>> >>>> --- >>>> 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 edec1d0..cf392c7 100644 >>>> --- a/drivers/iio/accel/kxcjk-1013.c >>>> +++ b/drivers/iio/accel/kxcjk-1013.c >>>> @@ -1393,6 +1393,7 @@ static const struct acpi_device_id >>>> kx_acpi_match[] = { >>>> {"KXCJ1013", KXCJK1013}, >>>> {"KXCJ1008", KXCJ91008}, >>>> {"KXCJ9000", KXCJ91008}, >>>> + {"KIOX000A", KXCJ91008}, >>>> {"KXTJ1009", KXTJ21009}, >>>> {"SMO8500", KXCJ91008}, >>>> { }, >>>> ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-07-24 11:33 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CACA7eURhTD8RzGOoYciOEp_oC2iuy+197gS7M3qnMhAEiH=v8A@mail.gmail.com>
2016-07-19 10:01 ` Fwd: [PATCH] iio: accel: kxcjk-1013: add the "KIOX000A" ACPI id Christophe Chapuis
2016-07-19 15:58 ` Jonathan Cameron
2016-07-19 16:11 ` Srinivas Pandruvada
2016-07-20 6:26 ` Christophe Chapuis
2016-07-24 11:33 ` Jonathan Cameron
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).