linux-kernel - Re: [PATCH v7 1/2] dt-bindings: iio: frequency: Add ADMV4420 doc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 27 Feb 2022 12:21:45 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>
Cc:     Cristian Pop <cristian.pop@...log.com>, linux-iio@...r.kernel.org,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
        robh+dt@...nel.org
Subject: Re: [PATCH v7 1/2] dt-bindings: iio: frequency: Add ADMV4420 doc

On Wed, 23 Feb 2022 14:26:05 +0100
Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com> wrote:

> On 23/02/2022 14:08, Cristian Pop wrote:
> > Add device tree bindings for the ADMV4420 K band downconverter.
> > 
> > Signed-off-by: Cristian Pop <cristian.pop@...log.com>
> > ---
> > Changes in v7:
> >  - Fix commit message  
> 
> Please include the tags accumulated in previous reviews.
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>
> 
> 
> Best regards,
> Krzysztof

Series applied to the togreg branch of iio.git and pushed out as testing
to let 0-day take a look before I go breaking linux-next (again :)

You get the same general feedback for future series as I just gave Antoniu:

1) Particularly when doing multiple revisions close together, please give
   a longer change log covering at least the last few versions as not
   everyone will have looked at v7 and it saves reviewers time otherwise
   spent checking back through earlier versions.
2) Cover letters are good for multipatch series because they both provide
   somewhere for general discussion that overlaps multiple patches and
   as somewhere for series wide tags to be given and that sometimes
   makes my life easier (complex cases where some tags are for individual
   patches and some are series wide which I have to go in an add by hand
   as no automated tool e.g. b4, could work it out!) 

Sometimes I like to complain about this trivial stuff that makes me
ever so slightly grumpy :)

Jonathan