* [PATCH] iio: sx9360: enable push iio event
@ 2022-01-18 21:25 Gwendal Grignou
2022-01-22 17:29 ` Jonathan Cameron
0 siblings, 1 reply; 6+ messages in thread
From: Gwendal Grignou @ 2022-01-18 21:25 UTC (permalink / raw)
To: jongpil19.jung, jic23, lars, andy.shevchenko; +Cc: linux-iio, Gwendal Grignou
From: Jongpil Jung <jongpil19.jung@samsung.com>
Fixes: f75095753 ("iio:proximity:sx9360: Add sx9360 support")
To convert SX9360 status register ["REG_STAT"], into a channel
index, we need to right shift by |stat_offset|, not left shift.
Also the PROXSTAT bit (3) is for channel 1 (PHM, PHase Measured), not (PHR,
PHase Reference, channel 0), so the offset is 2 instead of 3.
Signed-off-by: Jongpil Jung <jongpil19.jung@samsung.com>
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
---
drivers/iio/proximity/sx9360.c | 2 +-
drivers/iio/proximity/sx_common.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iio/proximity/sx9360.c b/drivers/iio/proximity/sx9360.c
index 6fd6561bb6f5b8..3ebb30c8a4f61d 100644
--- a/drivers/iio/proximity/sx9360.c
+++ b/drivers/iio/proximity/sx9360.c
@@ -775,7 +775,7 @@ static const struct sx_common_chip_info sx9360_chip_info = {
.reg_reset = SX9360_REG_RESET,
.mask_enable_chan = SX9360_REG_GNRL_CTRL0_PHEN_MASK,
- .stat_offset = 3,
+ .stat_offset = 2,
.num_channels = SX9360_NUM_CHANNELS,
.num_default_regs = ARRAY_SIZE(sx9360_default_regs),
diff --git a/drivers/iio/proximity/sx_common.c b/drivers/iio/proximity/sx_common.c
index ac8fd5920481cb..a7c07316a0a91e 100644
--- a/drivers/iio/proximity/sx_common.c
+++ b/drivers/iio/proximity/sx_common.c
@@ -87,7 +87,7 @@ static void sx_common_push_events(struct iio_dev *indio_dev)
return;
}
- val <<= data->chip_info->stat_offset;
+ val >>= data->chip_info->stat_offset;
/*
* Only iterate over channels with changes on proximity status that have
--
2.34.1.703.g22d0c6ccf7-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] iio: sx9360: enable push iio event
2022-01-18 21:25 [PATCH] iio: sx9360: enable push iio event Gwendal Grignou
@ 2022-01-22 17:29 ` Jonathan Cameron
2022-01-22 18:15 ` Andy Shevchenko
0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Cameron @ 2022-01-22 17:29 UTC (permalink / raw)
To: Gwendal Grignou; +Cc: jongpil19.jung, lars, andy.shevchenko, linux-iio
On Tue, 18 Jan 2022 13:25:04 -0800
Gwendal Grignou <gwendal@chromium.org> wrote:
> From: Jongpil Jung <jongpil19.jung@samsung.com>
Hi,
Patch title is not really clear. It suggests this is enabling
a new feature rather than fixing anything. Please rephrase.
>
> Fixes: f75095753 ("iio:proximity:sx9360: Add sx9360 support")
This is part of the tag block so should appear.
>
> To convert SX9360 status register ["REG_STAT"], into a channel
> index, we need to right shift by |stat_offset|, not left shift.
> Also the PROXSTAT bit (3) is for channel 1 (PHM, PHase Measured), not (PHR,
> PHase Reference, channel 0), so the offset is 2 instead of 3.
>
Phase
fixes tag should be here.
> Signed-off-by: Jongpil Jung <jongpil19.jung@samsung.com>
> Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Thanks,
Jonathan
> ---
> drivers/iio/proximity/sx9360.c | 2 +-
> drivers/iio/proximity/sx_common.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/proximity/sx9360.c b/drivers/iio/proximity/sx9360.c
> index 6fd6561bb6f5b8..3ebb30c8a4f61d 100644
> --- a/drivers/iio/proximity/sx9360.c
> +++ b/drivers/iio/proximity/sx9360.c
> @@ -775,7 +775,7 @@ static const struct sx_common_chip_info sx9360_chip_info = {
> .reg_reset = SX9360_REG_RESET,
>
> .mask_enable_chan = SX9360_REG_GNRL_CTRL0_PHEN_MASK,
> - .stat_offset = 3,
> + .stat_offset = 2,
> .num_channels = SX9360_NUM_CHANNELS,
> .num_default_regs = ARRAY_SIZE(sx9360_default_regs),
>
> diff --git a/drivers/iio/proximity/sx_common.c b/drivers/iio/proximity/sx_common.c
> index ac8fd5920481cb..a7c07316a0a91e 100644
> --- a/drivers/iio/proximity/sx_common.c
> +++ b/drivers/iio/proximity/sx_common.c
> @@ -87,7 +87,7 @@ static void sx_common_push_events(struct iio_dev *indio_dev)
> return;
> }
>
> - val <<= data->chip_info->stat_offset;
> + val >>= data->chip_info->stat_offset;
>
> /*
> * Only iterate over channels with changes on proximity status that have
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] iio: sx9360: enable push iio event
2022-01-22 17:29 ` Jonathan Cameron
@ 2022-01-22 18:15 ` Andy Shevchenko
2022-01-22 19:14 ` Jonathan Cameron
0 siblings, 1 reply; 6+ messages in thread
From: Andy Shevchenko @ 2022-01-22 18:15 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Gwendal Grignou, jongpil19.jung, Lars-Peter Clausen, linux-iio
On Sat, Jan 22, 2022 at 7:23 PM Jonathan Cameron <jic23@kernel.org> wrote:
> On Tue, 18 Jan 2022 13:25:04 -0800
> Gwendal Grignou <gwendal@chromium.org> wrote:
...
> > Fixes: f75095753 ("iio:proximity:sx9360: Add sx9360 support")
> This is part of the tag block so should appear.
> fixes tag should be here.
> > Signed-off-by: Jongpil Jung <jongpil19.jung@samsung.com>
> > Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
...and the submitter's SoB must be last (according to the documentation).
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] iio: sx9360: enable push iio event
2022-01-22 18:15 ` Andy Shevchenko
@ 2022-01-22 19:14 ` Jonathan Cameron
2022-01-22 19:14 ` Jonathan Cameron
2022-01-24 6:00 ` Gwendal Grignou
0 siblings, 2 replies; 6+ messages in thread
From: Jonathan Cameron @ 2022-01-22 19:14 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Gwendal Grignou, jongpil19.jung, Lars-Peter Clausen, linux-iio
On Sat, 22 Jan 2022 20:15:57 +0200
Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
> On Sat, Jan 22, 2022 at 7:23 PM Jonathan Cameron <jic23@kernel.org> wrote:
> > On Tue, 18 Jan 2022 13:25:04 -0800
> > Gwendal Grignou <gwendal@chromium.org> wrote:
>
> ...
>
> > > Fixes: f75095753 ("iio:proximity:sx9360: Add sx9360 support")
> > This is part of the tag block so should appear.
>
>
> > fixes tag should be here.
> > > Signed-off-by: Jongpil Jung <jongpil19.jung@samsung.com>
> > > Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
>
> ...and the submitter's SoB must be last (according to the documentation).
>
Hi Andy,
If it's a handling chain, rather that about co development etc then
I'd expect it to be in this order to indicate that Gwendal was on the
route to upstream.
"Any further SoBs (Signed-off-by:'s) following the author's SoB are from
people handling and transporting the patch, but were not involved in its
development. SoB chains should reflect the **real** route a patch took
as it was propagated to the maintainers and ultimately to Linus, with
the first SoB entry signalling primary authorship of a single author."
Jonathan
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] iio: sx9360: enable push iio event
2022-01-22 19:14 ` Jonathan Cameron
@ 2022-01-22 19:14 ` Jonathan Cameron
2022-01-24 6:00 ` Gwendal Grignou
1 sibling, 0 replies; 6+ messages in thread
From: Jonathan Cameron @ 2022-01-22 19:14 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Gwendal Grignou, jongpil19.jung, Lars-Peter Clausen, linux-iio
On Sat, 22 Jan 2022 20:15:57 +0200
Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
> On Sat, Jan 22, 2022 at 7:23 PM Jonathan Cameron <jic23@kernel.org> wrote:
> > On Tue, 18 Jan 2022 13:25:04 -0800
> > Gwendal Grignou <gwendal@chromium.org> wrote:
>
> ...
>
> > > Fixes: f75095753 ("iio:proximity:sx9360: Add sx9360 support")
> > This is part of the tag block so should appear.
>
>
> > fixes tag should be here.
> > > Signed-off-by: Jongpil Jung <jongpil19.jung@samsung.com>
> > > Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
>
> ...and the submitter's SoB must be last (according to the documentation).
>
Hi Andy,
If it's a handling chain, rather that about co development etc then
I'd expect it to be in this order to indicate that Gwendal was on the
route to upstream.
"Any further SoBs (Signed-off-by:'s) following the author's SoB are from
people handling and transporting the patch, but were not involved in its
development. SoB chains should reflect the **real** route a patch took
as it was propagated to the maintainers and ultimately to Linus, with
the first SoB entry signalling primary authorship of a single author."
Jonathan
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] iio: sx9360: enable push iio event
2022-01-22 19:14 ` Jonathan Cameron
2022-01-22 19:14 ` Jonathan Cameron
@ 2022-01-24 6:00 ` Gwendal Grignou
1 sibling, 0 replies; 6+ messages in thread
From: Gwendal Grignou @ 2022-01-24 6:00 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Andy Shevchenko, jongpil19.jung, Lars-Peter Clausen, linux-iio
On Sat, Jan 22, 2022 at 11:08 AM Jonathan Cameron <jic23@kernel.org> wrote:
>
> On Sat, 22 Jan 2022 20:15:57 +0200
> Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
>
> > On Sat, Jan 22, 2022 at 7:23 PM Jonathan Cameron <jic23@kernel.org> wrote:
> > > On Tue, 18 Jan 2022 13:25:04 -0800
> > > Gwendal Grignou <gwendal@chromium.org> wrote:
> >
> > ...
> >
> > > > Fixes: f75095753 ("iio:proximity:sx9360: Add sx9360 support")
> > > This is part of the tag block so should appear.
Done: The patch is at
https://lore.kernel.org/all/20220122213444.745152-1-gwendal@chromium.org/.
Given the title change, it is not automatically connected to this
patch. I should have added a --in-reply-to argument to my git
send-mail command, will do better next time.
Gwendal.
> >
> >
> > > fixes tag should be here.
> > > > Signed-off-by: Jongpil Jung <jongpil19.jung@samsung.com>
> > > > Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
> >
> > ...and the submitter's SoB must be last (according to the documentation).
> >
> Hi Andy,
>
> If it's a handling chain, rather that about co development etc then
> I'd expect it to be in this order to indicate that Gwendal was on the
> route to upstream.
>
> "Any further SoBs (Signed-off-by:'s) following the author's SoB are from
> people handling and transporting the patch, but were not involved in its
> development. SoB chains should reflect the **real** route a patch took
> as it was propagated to the maintainers and ultimately to Linus, with
> the first SoB entry signalling primary authorship of a single author."
>
> Jonathan
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-01-24 6:01 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-01-18 21:25 [PATCH] iio: sx9360: enable push iio event Gwendal Grignou
2022-01-22 17:29 ` Jonathan Cameron
2022-01-22 18:15 ` Andy Shevchenko
2022-01-22 19:14 ` Jonathan Cameron
2022-01-22 19:14 ` Jonathan Cameron
2022-01-24 6:00 ` Gwendal Grignou
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox