Messages in this thread | | | From | "Miclaus, Antoniu" <> | Subject | RE: [PATCH v4 2/3] dt-bindings:iio:amplifiers: add ada4250 doc | Date | Tue, 22 Feb 2022 09:21:28 +0000 |
| |
-- Antoniu Miclăuş
> -----Original Message----- > From: Krzysztof Kozlowski <krzk@kernel.org> > Sent: Monday, February 21, 2022 6:36 PM > To: Miclaus, Antoniu <Antoniu.Miclaus@analog.com>; jic23@kernel.org; > robh+dt@kernel.org; linux-iio@vger.kernel.org; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org > 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@analog.com> > > --- > > 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
| |