* [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
@ 2023-01-02 18:51 Dmitry Perchanov
2023-01-08 12:24 ` Jonathan Cameron
0 siblings, 1 reply; 4+ messages in thread
From: Dmitry Perchanov @ 2023-01-02 18:51 UTC (permalink / raw)
To: jic23; +Cc: linux-iio, lars, demisrael, Dmitry Perchanov
From d12f607f83c0bdae6529d8cfbb0cf17f8920c380 Mon Sep 17 00:00:00 2001
From: Dmitry Perchanov <dmitry.perchanov@intel.com>
Date: Tue, 13 Dec 2022 17:44:01 +0200
Subject: [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
Return value should be zero for success.
This was forgotten for timestamp feature.
Verified on RealSense cameras.
Signed-off-by: Dmitry Perchanov <dmitry.perchanov@intel.com>
---
drivers/iio/accel/hid-sensor-accel-3d.c | 1 +
drivers/iio/gyro/hid-sensor-gyro-3d.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c
index a2def6f93..5eac7ea19 100644
--- a/drivers/iio/accel/hid-sensor-accel-3d.c
+++ b/drivers/iio/accel/hid-sensor-accel-3d.c
@@ -280,6 +280,7 @@ static int accel_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
hid_sensor_convert_timestamp(
&accel_state->common_attributes,
*(int64_t *)raw_data);
+ ret = 0;
break;
default:
break;
diff --git a/drivers/iio/gyro/hid-sensor-gyro-3d.c b/drivers/iio/gyro/hid-sensor-gyro-3d.c
index 8f0ad022c..698c50da1 100644
--- a/drivers/iio/gyro/hid-sensor-gyro-3d.c
+++ b/drivers/iio/gyro/hid-sensor-gyro-3d.c
@@ -231,6 +231,7 @@ static int gyro_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
gyro_state->timestamp =
hid_sensor_convert_timestamp(&gyro_state->common_attributes,
*(s64 *)raw_data);
+ ret = 0;
break;
default:
break;
--
2.39.0
---------------------------------------------------------------------
Intel Israel (74) Limited
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
2023-01-02 18:51 [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample Dmitry Perchanov
@ 2023-01-08 12:24 ` Jonathan Cameron
2023-01-08 12:32 ` Jonathan Cameron
0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Cameron @ 2023-01-08 12:24 UTC (permalink / raw)
To: Dmitry Perchanov; +Cc: linux-iio, lars, demisrael
On Mon, 02 Jan 2023 20:51:27 +0200
Dmitry Perchanov <dmitry.perchanov@intel.com> wrote:
> From d12f607f83c0bdae6529d8cfbb0cf17f8920c380 Mon Sep 17 00:00:00 2001
> From: Dmitry Perchanov <dmitry.perchanov@intel.com>
> Date: Tue, 13 Dec 2022 17:44:01 +0200
> Subject: [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
>
> Return value should be zero for success.
> This was forgotten for timestamp feature.
> Verified on RealSense cameras.
>
> Signed-off-by: Dmitry Perchanov <dmitry.perchanov@intel.com>
Hi Dmitry,
I've taken this as it stands (needed to do a bit of hand editing to get
git to accept your email). However, if you fancy doing some cleanup
it would be better still if these functions just returned directly in each
arm of the switch statement. Drop the final return and the compiler
will moan if any path is not covered...
Applied to the fixes-togreg branch of iio.git and marked for stable inclusion.
Thanks,
Jonathan
> ---
> drivers/iio/accel/hid-sensor-accel-3d.c | 1 +
> drivers/iio/gyro/hid-sensor-gyro-3d.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c
> index a2def6f93..5eac7ea19 100644
> --- a/drivers/iio/accel/hid-sensor-accel-3d.c
> +++ b/drivers/iio/accel/hid-sensor-accel-3d.c
> @@ -280,6 +280,7 @@ static int accel_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
> hid_sensor_convert_timestamp(
> &accel_state->common_attributes,
> *(int64_t *)raw_data);
> + ret = 0;
> break;
> default:
> break;
> diff --git a/drivers/iio/gyro/hid-sensor-gyro-3d.c b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> index 8f0ad022c..698c50da1 100644
> --- a/drivers/iio/gyro/hid-sensor-gyro-3d.c
> +++ b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> @@ -231,6 +231,7 @@ static int gyro_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
> gyro_state->timestamp =
> hid_sensor_convert_timestamp(&gyro_state->common_attributes,
> *(s64 *)raw_data);
> + ret = 0;
> break;
> default:
> break;
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
2023-01-08 12:24 ` Jonathan Cameron
@ 2023-01-08 12:32 ` Jonathan Cameron
2023-01-09 7:46 ` Dmitry Perchanov
0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Cameron @ 2023-01-08 12:32 UTC (permalink / raw)
To: Dmitry Perchanov; +Cc: linux-iio, lars, demisrael
On Sun, 8 Jan 2023 12:24:34 +0000
Jonathan Cameron <jic23@kernel.org> wrote:
> On Mon, 02 Jan 2023 20:51:27 +0200
> Dmitry Perchanov <dmitry.perchanov@intel.com> wrote:
>
> > From d12f607f83c0bdae6529d8cfbb0cf17f8920c380 Mon Sep 17 00:00:00 2001
> > From: Dmitry Perchanov <dmitry.perchanov@intel.com>
> > Date: Tue, 13 Dec 2022 17:44:01 +0200
> > Subject: [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
This block shouldn't be here in the email.
> >
> > Return value should be zero for success.
> > This was forgotten for timestamp feature.
> > Verified on RealSense cameras.
> >
> > Signed-off-by: Dmitry Perchanov <dmitry.perchanov@intel.com>
> Hi Dmitry,
>
> I've taken this as it stands (needed to do a bit of hand editing to get
> git to accept your email). However, if you fancy doing some cleanup
> it would be better still if these functions just returned directly in each
> arm of the switch statement. Drop the final return and the compiler
> will moan if any path is not covered...
>
> Applied to the fixes-togreg branch of iio.git and marked for stable inclusion.
Actually, whilst applying I tried to add a fixes tag with the assumption these
were probably both added at a similar time. Unfortunately not (one was 2021,
the other 2016).
As such I've dropped the patch. Could you resend this as two separate patches, with
fixes tags as appropriate.
Fixes: 4648cbd8fb92 ("iio: hid-sensor-gyro-3d: Add timestamp channel")
Fixes: a96cd0f901ee ("iio: accel: hid-sensor-accel-3d: Add timestamp")
This is a fix we definitely want backported, so making it easy to do is important.
Thanks,
Jonathan
>
> Thanks,
>
> Jonathan
>
> > ---
> > drivers/iio/accel/hid-sensor-accel-3d.c | 1 +
> > drivers/iio/gyro/hid-sensor-gyro-3d.c | 1 +
> > 2 files changed, 2 insertions(+)
> >
> > diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c
> > index a2def6f93..5eac7ea19 100644
> > --- a/drivers/iio/accel/hid-sensor-accel-3d.c
> > +++ b/drivers/iio/accel/hid-sensor-accel-3d.c
> > @@ -280,6 +280,7 @@ static int accel_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
> > hid_sensor_convert_timestamp(
> > &accel_state->common_attributes,
> > *(int64_t *)raw_data);
> > + ret = 0;
> > break;
> > default:
> > break;
> > diff --git a/drivers/iio/gyro/hid-sensor-gyro-3d.c b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > index 8f0ad022c..698c50da1 100644
> > --- a/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > +++ b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > @@ -231,6 +231,7 @@ static int gyro_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
> > gyro_state->timestamp =
> > hid_sensor_convert_timestamp(&gyro_state->common_attributes,
> > *(s64 *)raw_data);
> > + ret = 0;
> > break;
> > default:
> > break;
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
2023-01-08 12:32 ` Jonathan Cameron
@ 2023-01-09 7:46 ` Dmitry Perchanov
0 siblings, 0 replies; 4+ messages in thread
From: Dmitry Perchanov @ 2023-01-09 7:46 UTC (permalink / raw)
To: Jonathan Cameron; +Cc: linux-iio, lars, demisrael
On Sun, 2023-01-08 at 12:32 +0000, Jonathan Cameron wrote:
> On Sun, 8 Jan 2023 12:24:34 +0000
> Jonathan Cameron <jic23@kernel.org> wrote:
>
> > On Mon, 02 Jan 2023 20:51:27 +0200
> > Dmitry Perchanov <dmitry.perchanov@intel.com> wrote:
> >
> > > From d12f607f83c0bdae6529d8cfbb0cf17f8920c380 Mon Sep 17 00:00:00 2001
> > > From: Dmitry Perchanov <dmitry.perchanov@intel.com>
> > > Date: Tue, 13 Dec 2022 17:44:01 +0200
> > > Subject: [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample
>
> This block shouldn't be here in the email.
>
> > >
> > > Return value should be zero for success.
> > > This was forgotten for timestamp feature.
> > > Verified on RealSense cameras.
> > >
> > > Signed-off-by: Dmitry Perchanov <dmitry.perchanov@intel.com>
> >
> > Hi Dmitry,
> >
> > I've taken this as it stands (needed to do a bit of hand editing to get
> > git to accept your email). However, if you fancy doing some cleanup
> > it would be better still if these functions just returned directly in each
> > arm of the switch statement. Drop the final return and the compiler
> > will moan if any path is not covered...
> >
> > Applied to the fixes-togreg branch of iio.git and marked for stable inclusion.
>
> Actually, whilst applying I tried to add a fixes tag with the assumption these
> were probably both added at a similar time. Unfortunately not (one was 2021,
> the other 2016).
>
> As such I've dropped the patch. Could you resend this as two separate patches, with
> fixes tags as appropriate.
> Fixes: 4648cbd8fb92 ("iio: hid-sensor-gyro-3d: Add timestamp channel")
> Fixes: a96cd0f901ee ("iio: accel: hid-sensor-accel-3d: Add timestamp")
>
I will resend two separate patches.
Thanks,
Dmitry
> This is a fix we definitely want backported, so making it easy to do is important.
>
> Thanks,
>
> Jonathan
>
>
>
> >
> > Thanks,
> >
> > Jonathan
> >
> > > ---
> > > drivers/iio/accel/hid-sensor-accel-3d.c | 1 +
> > > drivers/iio/gyro/hid-sensor-gyro-3d.c | 1 +
> > > 2 files changed, 2 insertions(+)
> > >
> > > diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c
> > > index a2def6f93..5eac7ea19 100644
> > > --- a/drivers/iio/accel/hid-sensor-accel-3d.c
> > > +++ b/drivers/iio/accel/hid-sensor-accel-3d.c
> > > @@ -280,6 +280,7 @@ static int accel_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
> > > hid_sensor_convert_timestamp(
> > > &accel_state->common_attributes,
> > > *(int64_t *)raw_data);
> > > + ret = 0;
> > > break;
> > > default:
> > > break;
> > > diff --git a/drivers/iio/gyro/hid-sensor-gyro-3d.c b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > > index 8f0ad022c..698c50da1 100644
> > > --- a/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > > +++ b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > > @@ -231,6 +231,7 @@ static int gyro_3d_capture_sample(struct hid_sensor_hub_device *hsdev,
> > > gyro_state->timestamp =
> > > hid_sensor_convert_timestamp(&gyro_state->common_attributes,
> > > *(s64 *)raw_data);
> > > + ret = 0;
> > > break;
> > > default:
> > > break;
>
>
---------------------------------------------------------------------
Intel Israel (74) Limited
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-01-09 7:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-02 18:51 [PATCH] iio: hid: fix the retval in [accel|gyro]_3d_capture_sample Dmitry Perchanov
2023-01-08 12:24 ` Jonathan Cameron
2023-01-08 12:32 ` Jonathan Cameron
2023-01-09 7:46 ` Dmitry Perchanov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox