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: