Hi Sjoerd,
----- Original Message -----
From: "Sjoerd Simons" sjoerd@collabora.com To: "Simon Glass" sjg@chromium.org, "Maxim Cournoyer" maxim.cournoyer@gmail.com Cc: "Maxim Cournoyer" maxim.cournoyer@savoirfairelinux.com, "U-Boot Mailing List" u-boot@lists.denx.de Sent: Friday, January 13, 2023 5:19:39 AM Subject: Re: [PATCH v3 3/8] patman: invoke the checkpatch.pl script with '--u-boot' and '--strict'
Hey,
This breaks using patman for linux as it's check-patch has no clue about `--u-boot` :). I'm not sure if people using patman for linux patches is meant to be a supported use-case though (the documentation seems to indicate it is)
Sorry for the breakage, and thank you for reporting it! I had overlooked that. I've reverted that commit and fixed the test suite via symlinking the top level .checkpatch.conf to tools/patman/.checkpatch.conf instead, which seems to work.
You are CC'd to that change [0], in case you'd like to test it in your environment.
Thank you,
Maxim
[0] https://patchwork.ozlabs.org/project/uboot/patch/20230113135051.22738-2-maxi...