mbox series

[v2,0/4] Xilinx AMS fixes

Message ID 20220127173450.3684318-1-robert.hancock@calian.com
Headers show
Series Xilinx AMS fixes | expand

Message

Robert Hancock Jan. 27, 2022, 5:34 p.m. UTC
Various fixes for the Xilinx AMS driver.

Changes since v1:
-drop addition to ZynqMP device tree, will be submitted elsewhere
-add patch to fix DT binding to add missing clock entry

Robert Hancock (4):
  dt-bindings: iio: adc: zynqmp_ams: Add clock entry
  iio: adc: xilinx-ams: Fixed missing PS channels
  iio: adc: xilinx-ams: Fixed wrong sequencer register settings
  iio: adc: xilinx-ams: Fix single channel switching sequence

 .../bindings/iio/adc/xlnx,zynqmp-ams.yaml         |  8 ++++++++
 drivers/iio/adc/xilinx-ams.c                      | 15 +++++++++++----
 2 files changed, 19 insertions(+), 4 deletions(-)

Comments

Jonathan Cameron Jan. 30, 2022, 12:46 p.m. UTC | #1
On Thu, 27 Jan 2022 11:34:46 -0600
Robert Hancock <robert.hancock@calian.com> wrote:

> Various fixes for the Xilinx AMS driver.
> 
> Changes since v1:
> -drop addition to ZynqMP device tree, will be submitted elsewhere
> -add patch to fix DT binding to add missing clock entry

I'm fine with these but would like to leave them all on list a tiny
bit longer so we can hopefully get some review, particularly on patches
1 and 4.

Jonathan

> 
> Robert Hancock (4):
>   dt-bindings: iio: adc: zynqmp_ams: Add clock entry
>   iio: adc: xilinx-ams: Fixed missing PS channels
>   iio: adc: xilinx-ams: Fixed wrong sequencer register settings
>   iio: adc: xilinx-ams: Fix single channel switching sequence
> 
>  .../bindings/iio/adc/xlnx,zynqmp-ams.yaml         |  8 ++++++++
>  drivers/iio/adc/xilinx-ams.c                      | 15 +++++++++++----
>  2 files changed, 19 insertions(+), 4 deletions(-)
>
Robert Hancock Feb. 9, 2022, 7:49 p.m. UTC | #2
On Sun, 2022-01-30 at 12:46 +0000, Jonathan Cameron wrote:
> On Thu, 27 Jan 2022 11:34:46 -0600
> Robert Hancock <robert.hancock@calian.com> wrote:
> 
> > Various fixes for the Xilinx AMS driver.
> > 
> > Changes since v1:
> > -drop addition to ZynqMP device tree, will be submitted elsewhere
> > -add patch to fix DT binding to add missing clock entry
> 
> I'm fine with these but would like to leave them all on list a tiny
> bit longer so we can hopefully get some review, particularly on patches
> 1 and 4.
> 
> Jonathan

Hi all,

I don't think I've gotten any feedback. Is anyone able to review/test?

> 
> > Robert Hancock (4):
> >   dt-bindings: iio: adc: zynqmp_ams: Add clock entry
> >   iio: adc: xilinx-ams: Fixed missing PS channels
> >   iio: adc: xilinx-ams: Fixed wrong sequencer register settings
> >   iio: adc: xilinx-ams: Fix single channel switching sequence
> > 
> >  .../bindings/iio/adc/xlnx,zynqmp-ams.yaml         |  8 ++++++++
> >  drivers/iio/adc/xilinx-ams.c                      | 15 +++++++++++----
> >  2 files changed, 19 insertions(+), 4 deletions(-)
> >
Jonathan Cameron Feb. 26, 2022, 4:45 p.m. UTC | #3
On Wed, 9 Feb 2022 19:49:27 +0000
Robert Hancock <robert.hancock@calian.com> wrote:

> On Sun, 2022-01-30 at 12:46 +0000, Jonathan Cameron wrote:
> > On Thu, 27 Jan 2022 11:34:46 -0600
> > Robert Hancock <robert.hancock@calian.com> wrote:
> >   
> > > Various fixes for the Xilinx AMS driver.
> > > 
> > > Changes since v1:
> > > -drop addition to ZynqMP device tree, will be submitted elsewhere
> > > -add patch to fix DT binding to add missing clock entry  
> > 
> > I'm fine with these but would like to leave them all on list a tiny
> > bit longer so we can hopefully get some review, particularly on patches
> > 1 and 4.
> > 
> > Jonathan  
> 
> Hi all,
> 
> I don't think I've gotten any feedback. Is anyone able to review/test?
> 
Michal,

If you have a chance to take a quick look at this series that would
be great.

Thanks,

Jonathan


> >   
> > > Robert Hancock (4):
> > >   dt-bindings: iio: adc: zynqmp_ams: Add clock entry
> > >   iio: adc: xilinx-ams: Fixed missing PS channels
> > >   iio: adc: xilinx-ams: Fixed wrong sequencer register settings
> > >   iio: adc: xilinx-ams: Fix single channel switching sequence
> > > 
> > >  .../bindings/iio/adc/xlnx,zynqmp-ams.yaml         |  8 ++++++++
> > >  drivers/iio/adc/xilinx-ams.c                      | 15 +++++++++++----
> > >  2 files changed, 19 insertions(+), 4 deletions(-)
> > >
Michal Simek Feb. 28, 2022, 6:40 a.m. UTC | #4
On 2/26/22 17:45, Jonathan Cameron wrote:
> On Wed, 9 Feb 2022 19:49:27 +0000
> Robert Hancock <robert.hancock@calian.com> wrote:
> 
>> On Sun, 2022-01-30 at 12:46 +0000, Jonathan Cameron wrote:
>>> On Thu, 27 Jan 2022 11:34:46 -0600
>>> Robert Hancock <robert.hancock@calian.com> wrote:
>>>    
>>>> Various fixes for the Xilinx AMS driver.
>>>>
>>>> Changes since v1:
>>>> -drop addition to ZynqMP device tree, will be submitted elsewhere
>>>> -add patch to fix DT binding to add missing clock entry
>>>
>>> I'm fine with these but would like to leave them all on list a tiny
>>> bit longer so we can hopefully get some review, particularly on patches
>>> 1 and 4.
>>>
>>> Jonathan
>>
>> Hi all,
>>
>> I don't think I've gotten any feedback. Is anyone able to review/test?
>>
> Michal,
> 
> If you have a chance to take a quick look at this series that would
> be great.

That series looks good to me. Please apply and feel free to add my

Acked-by: Michal Simek <michal.simek@xilinx.com>

Thanks,
Michal