* 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).