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

2021-02-16 Thread Jonathan Cameron
On Tue, 16 Feb 2021 10:20:59 + Cristian Marussi wrote: > Hi Jyoti, > > On Mon, Feb 15, 2021 at 04:14:57PM -0800, Jyoti Bhayana wrote: > > Hi Jonathan/Cristian, > > > > I have uploaded v6 of the IIO SCMI patch. Not sure if you got a chance to > > review that version and if any further change

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

2021-02-16 Thread Cristian Marussi
Hi Jyoti, On Mon, Feb 15, 2021 at 04:14:57PM -0800, Jyoti Bhayana wrote: > Hi Jonathan/Cristian, > > I have uploaded v6 of the IIO SCMI patch. Not sure if you got a chance to > review that version and if any further changes are needed. Seen that, for the SCMI part seemed fine to me, I'll just ha

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

2021-02-15 Thread Cristian Marussi
On Mon, Feb 15, 2021 at 11:07:56AM +, Jonathan Cameron wrote: > On Mon, 15 Feb 2021 09:25:26 + > Cristian Marussi wrote: > > > On Fri, Feb 12, 2021 at 07:18:58PM +, Jonathan Cameron wrote: > > > On Wed, 10 Feb 2021 21:46:19 + > > > Cristian Marussi wrote: > > > > > > > Hi Jyot

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

2021-02-15 Thread Sudeep Holla
On Mon, Feb 15, 2021 at 11:07:56AM +, Jonathan Cameron wrote: > Hi Cristian, > > So this driver will also be 5.13 material now (merge window for IIO > effectively > closes 1-2 weeks before Linus opens the main one). > I guessed so. > The way we normally handle cases like this where we likely

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

2021-02-15 Thread Jonathan Cameron
On Mon, 15 Feb 2021 09:25:26 + Cristian Marussi wrote: > On Fri, Feb 12, 2021 at 07:18:58PM +, Jonathan Cameron wrote: > > On Wed, 10 Feb 2021 21:46:19 + > > Cristian Marussi wrote: > > > > > Hi Jyoti, > > > > > > On Wed, Feb 10, 2021 at 11:19:35AM -0800, Jyoti Bhayana wrote: >

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

2021-02-15 Thread Cristian Marussi
On Fri, Feb 12, 2021 at 07:18:58PM +, Jonathan Cameron wrote: > On Wed, 10 Feb 2021 21:46:19 + > Cristian Marussi wrote: > > > Hi Jyoti, > > > > On Wed, Feb 10, 2021 at 11:19:35AM -0800, Jyoti Bhayana wrote: > > > Hi Cristian, > > > > > > Thanks for the feedback. Regarding registering c

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

2021-02-12 Thread Jonathan Cameron
On Wed, 10 Feb 2021 21:46:19 + Cristian Marussi wrote: > Hi Jyoti, > > On Wed, Feb 10, 2021 at 11:19:35AM -0800, Jyoti Bhayana wrote: > > Hi Cristian, > > > > Thanks for the feedback. Regarding registering callbacks at the probe time > > instead of .preenable, I have tried it before but I t

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

2021-02-10 Thread Cristian Marussi
Hi Jyoti, On Wed, Feb 10, 2021 at 11:19:35AM -0800, Jyoti Bhayana wrote: > Hi Cristian, > > Thanks for the feedback. Regarding registering callbacks at the probe time > instead of .preenable, I have tried it before but I think due to some > issues(don't remember it now maybe on the platform side)

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

2021-02-09 Thread Cristian Marussi
Hi Jyoti some minor things down below. Other than that, FWIW about the SCMI side of this: Reviewed-by: Cristian Marussi Thanks Cristian On Mon, Feb 08, 2021 at 09:19:18PM +, Jyoti Bhayana wrote: > This change provides ARM SCMI Protocol based IIO device. > This driver provides support for