On Tue, Feb 10, 2026 at 01:04:57PM -0300, Fabio Estevam wrote:
> 
> On Tue, Feb 10, 2026 at 12:55???PM Brian Ruley
> <[email protected]> wrote:
> 
> > I'm simply following U-Boot coding style:
> > https://docs.u-boot.org/en/stable/develop/codingstyle.html
> >
> > Checkpatch has already complained a bunch whenever I touched this
> > particular code base, so I applied clang-format (since the project
> > has it) before making my changes to suppress some of the warnings.
> >
> > If you feel like there's no visual improvement, I can of course remove
> > the patch, but I feel it odd to have a coding style in the docs if it's
> > completely optional.
> 
> Please explain in the commit log that you're fixing checkpatch/clang warnings.
> 
> Simply stating "Conform with U-boot guidelines" is too vague.
> 
> Thanks

Okay, fair point. I'll fix it, but I'd rather not send a v3 just for
a commit message unless that's the only issue here.

Br,
Brian

Reply via email to