Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-16 Thread Jyoti Bhayana
Hi Christian, Thanks for porting the driver to the new interface. I have commented on changes in v7-0025-iio-scmi-port-driver-to-the-new-scmi_sensor_proto.patch. Thanks, Jyoti On Tue, Mar 16, 2021 at 6:13 AM Cristian Marussi wrote: > > Hi > > I reposted my series on top of the take3 immutable

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-16 Thread Cristian Marussi
Hi I reposted my series on top of the take3 immutable branch from Jonathan at: https://lore.kernel.org/lkml/20210316124903.35011-1-cristian.maru...@arm.com/T/#t You should have received an email regarding the relevant IIO SCMI patch

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-15 Thread Sudeep Holla
Hi Jonathan, On Sun, Mar 14, 2021 at 03:40:33PM +, Jonathan Cameron wrote: > On Sat, 13 Mar 2021 11:55:39 -0800 > Jyoti Bhayana wrote: > > > Hi Jonathan, > > > > I still see the old version 6 in ib-iio-scmi-5.12-rc2-take2 as well. > > OK. I'm completely confused as to what is going with my

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-14 Thread Jyoti Bhayana
Hi Jonathan, No worries. Thanks for the update. It looks good now. Thanks, Jyoti On Sun, Mar 14, 2021 at 8:40 AM Jonathan Cameron wrote: > > On Sat, 13 Mar 2021 11:55:39 -0800 > Jyoti Bhayana wrote: > > > Hi Jonathan, > > > > I still see the old version 6 in ib-iio-scmi-5.12-rc2-take2 as

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-14 Thread Jonathan Cameron
On Sat, 13 Mar 2021 11:55:39 -0800 Jyoti Bhayana wrote: > Hi Jonathan, > > I still see the old version 6 in ib-iio-scmi-5.12-rc2-take2 as well. OK. I'm completely confused as to what is going with my local tree. I have the right patch in the history but it didn't end up in the final pushed out

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-13 Thread Jyoti Bhayana
Hi Jonathan, I still see the old version 6 in ib-iio-scmi-5.12-rc2-take2 as well. Thanks, Jyoti On Sat, Mar 13, 2021 at 9:11 AM Jonathan Cameron wrote: > > On Fri, 12 Mar 2021 09:54:12 -0800 > Jyoti Bhayana wrote: > > > Hi Jonathan, > > > > I also see what Cristian has observed that rc2 is

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-13 Thread Jonathan Cameron
On Fri, 12 Mar 2021 09:54:12 -0800 Jyoti Bhayana wrote: > Hi Jonathan, > > I also see what Cristian has observed that rc2 is still pointing to v6 > of the SCMI IIO driver patch and it also doesn't have the changes > which you did in rc1 for changing long long to s64. > > Thanks, > Jyoti Gah!

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-12 Thread Jyoti Bhayana
Hi Jonathan, I also see what Cristian has observed that rc2 is still pointing to v6 of the SCMI IIO driver patch and it also doesn't have the changes which you did in rc1 for changing long long to s64. Thanks, Jyoti On Fri, Mar 12, 2021 at 5:31 AM Cristian Marussi wrote: > > Hi Jonathan, > >

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-12 Thread Cristian Marussi
Hi Jonathan, thanks for this, I was starting working on top of this new immutable branch BUT I spotted a thing that I wanted to check. You latest immutable: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git/log/?h=ib-iio-scmi-5.12-rc2 looking at the code in the SCMI IIO Jyoti

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-12 Thread Jonathan Cameron
On Thu, 11 Mar 2021 21:08:44 + Jonathan Cameron wrote: > On Tue, 9 Mar 2021 23:12:59 + > Jyoti Bhayana wrote: > > > This change provides ARM SCMI Protocol based IIO device. > > This driver provides support for Accelerometer and Gyroscope using > > SCMI Sensor Protocol extensions added

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-11 Thread Jonathan Cameron
On Tue, 9 Mar 2021 23:12:59 + Jyoti Bhayana wrote: > This change provides ARM SCMI Protocol based IIO device. > This driver provides support for Accelerometer and Gyroscope using > SCMI Sensor Protocol extensions added in the SCMIv3.0 ARM specification > > Reported-by: kernel test robot >

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-11 Thread Peter Hilber
On 10.03.21 18:19, Jyoti Bhayana wrote: > Hi Peter, > > As already discussed with ARM, the spec clearly mentions that it has > to be uppercase and not case insensitive. So this patch is consistent > with the specs and changing it with means that the spec would need to > change as well. Therefore,

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-10 Thread Jyoti Bhayana
Hi Peter, As already discussed with ARM, the spec clearly mentions that it has to be uppercase and not case insensitive. So this patch is consistent with the specs and changing it with means that the spec would need to change as well. Therefore, there is no need for another version of the patch

Re: [PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-10 Thread Peter Hilber
On 10.03.21 00:12, Jyoti Bhayana wrote: > This change provides ARM SCMI Protocol based IIO device. > This driver provides support for Accelerometer and Gyroscope using > SCMI Sensor Protocol extensions added in the SCMIv3.0 ARM specification > [snip] > + > +static int

[PATCH v7 1/1] iio/scmi: Adding support for IIO SCMI Based Sensors

2021-03-09 Thread Jyoti Bhayana
This change provides ARM SCMI Protocol based IIO device. This driver provides support for Accelerometer and Gyroscope using SCMI Sensor Protocol extensions added in the SCMIv3.0 ARM specification Reported-by: kernel test robot Signed-off-by: Jyoti Bhayana Link: