RE: [PATCH v4 2/3] dt-bindings:iio:amplifiers: add ada4250 doc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



--
Antoniu Miclăuş

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: Monday, February 21, 2022 6:36 PM
> To: Miclaus, Antoniu <Antoniu.Miclaus@xxxxxxxxxx>; jic23@xxxxxxxxxx;
> robh+dt@xxxxxxxxxx; linux-iio@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v4 2/3] dt-bindings:iio:amplifiers: add ada4250 doc
> 
> [External]
> 
> On 21/02/2022 10:27, Antoniu Miclaus wrote:
> > Add device tree bindings for the ADA4250 driver.
> >
> > Signed-off-by: Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx>
> > ---
> > changes in v4:
> >  - add `spi-max-frequency` property
> >  - use generic node naming
> >  .../bindings/iio/amplifiers/adi,ada4250.yaml  | 50 +++++++++++++++++++
> >  1 file changed, 50 insertions(+)
> >  create mode 100644
> Documentation/devicetree/bindings/iio/amplifiers/adi,ada4250.yaml
> >
> 
> Please do not resend without fixing pointed out issues. It just looks
> you ignore comments.
> 
> Please fix or respond to previous comments. Please also fix all common
> issues recently pointed out for all Analog devices - folks from Analog
> are making the same mistakes. Reviewing the same mistake is a bit a
> waste of time. Much more appreciated is when team applies one feedback
> to all its submissions. IOW, do not repeat the same mistake second time.
> 
> 
> Best regards,
> Krzysztof

I had a look over the previous comments and I saw that some of them were 
replied by Jonathan already, therefore I fixed only those that were still valid.
Indeed I missed the "Please put the bindings patch as first in the series",
my bad on that. Let me know if I missed anything else.

Regards,
Antoniu





[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux