[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aL2nBBmyQ-gks8Q3@hangmanPC>
Date: Sun, 7 Sep 2025 18:38:44 +0300
From: Cezar Chiru <chiru.cezar.89@...il.com>
To: Markus.Elfring@....de, andi.shyti@...nel.org
Cc: linux-i2c@...r.kernel.org, linux-kernel@...r.kernel.org,
chiru.cezar.89@...il.com
Subject: Re: [PATCH v2] i2c : algos : i2c-algo-pcf.c : fixed errors shown by
checkpatch
On Sun, Sep 07, 2025 at 03:19:20PM +0200, Markus Elfring wrote:
Hello Markus,
> > Fixed all 18 errors revealed using checkpatch.pl on i2c-algo-pcf.c
> …
>
> See also:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.17-rc4#n81
Refactored the commit message to be more compact
> …
> > Motivation is to fix all errors and warnings i2c-algo-pcf kerenel
> …
> kernel?
Yes, "kernel" was intended. Fixed.
> > v2:
> > Fixed build errors generated by missing ; after do - while.
> > Missed to git add latest changes to patch. Build is ok.
> >
> > Signed-off-by: Cezar Chiru <chiru.cezar.89@...il.com>
> > ---
> > drivers/i2c/algos/i2c-algo-pcf.c | 42 +++++++++++++++++++-------------
> …
I moved the version info and information from commit message that didn't
needed to reach the commit message after the --- marker line of the patch.
>
> * Please move your patch version descriptions behind the marker line.
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.17-rc4#n784
>
> * Will enumerations become more helpful?
If you were talking about enumerating the type of errors fixed in the commit
message then I done that. If something else, could you please explain it to me?
Markus, Andi, new [PATCH v3] attached to this email.
Thank you,
Best regards,
Cezar Chiru
View attachment "0001-i2c-algos-i2c-algo-pcf.c-fixed-errors-shown-by-check.patch" of type "text/x-diff" (6984 bytes)
Powered by blists - more mailing lists