Re: [PATCH v2 2/3] ASoC: dt-bindings: irondevice,sma1307: Add initial DT binding

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



On Tue, Sep 03, 2024 at 11:14:56AM +0200, Krzysztof Kozlowski wrote:
> On 03/09/2024 11:08, Ki-Seok Jo wrote:

> > I have no intention of opposing the content. I am asking again because I didn't receive any warnings when I did the following, and I suspect I might have done something wrong.

> > ./scripts/checkpatch.pl Documentation/devicetree/bindings/sound/irondevice,sma1307.yaml

> > I was under the impression that this only applied to patched files as described above. It turns out it can also be used with patch files. Thank you for the useful information!

> That's not how you run checkpatch. You run it on the patch. Please read
> submitting-patches document. It explains everything.

If you *do* want to run checkpatch on a file for some reason (eg, so you
can fix issues in existing code) you can use the --file option for that:

   ./scripts/checkpatch.pl --file Documentation/devicetree/bindings/sound/irondevice,sma1307.yaml

Also I don't know if you're aware of b4 but it's pretty handy for
automating a bunch of the procedural stuff around submitting patches,
one of the things it can help with is running checkpatch over your
series:

   https://b4.docs.kernel.org/en/latest/

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Pulseaudio]     [Linux Audio Users]     [ALSA Devel]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux