[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <55fd5868-0481-42e1-b56b-80f59d724364@foss.st.com>
Date: Tue, 10 Jun 2025 11:31:05 +0200
From: Benjamin Mugnier <benjamin.mugnier@...s.st.com>
To: Sakari Ailus <sakari.ailus@...ux.intel.com>
CC: Sylvain Petinot <sylvain.petinot@...s.st.com>,
Mauro Carvalho Chehab
<mchehab@...nel.org>,
<linux-media@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/4] media: i2c: vd55g1: Miscellaneous fixes
Hi Sakari,
Thank you for your review.
On 6/10/25 11:00, Sakari Ailus wrote:
> Hi Benjamin,
>
> Thanks for the set.
>
> On Mon, Jun 09, 2025 at 03:46:20PM +0200, Benjamin Mugnier wrote:
>> This series provides small fixes and style improvements to the vd55g1
>> driver.
>> Nothing fancy really, just to keep everything up to date.
>
> On all (or at least more than one) patches:
>
> - please add a proper commit message beyond the subject line and
>
> - properly wrap the commit paragraphs (up to 75 characters per line, e.g.
> the above paragraph would fit on two lines).
>
Yes, commit descriptions have been added in v2. Media-ci rightfully
yelled at me for that ;)
I don't see any commit paragraphs being above 75 characters, my vim
seems to be properly configured.
Do you mean the commit header too perhaps ? For example "media: i2c:
vd55g1: Miscellaneous fixes".
--
Regards,
Benjamin
Powered by blists - more mailing lists