[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <78674bb4-925f-4b1b-939a-7fc2adf1de64@amlogic.com>
Date: Mon, 31 Mar 2025 18:53:34 +0800
From: Keke Li <keke.li@...ogic.com>
To: Markus Elfring <Markus.Elfring@....de>, linux-media@...r.kernel.org,
devicetree@...r.kernel.org, Conor Dooley <conor+dt@...nel.org>,
Jacopo Mondi <jacopo.mondi@...asonboard.com>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Mauro Carvalho Chehab <mchehab@...nel.org>, Rob Herring <robh@...nel.org>
Cc: LKML <linux-kernel@...r.kernel.org>,
Daniel Scally <dan.scally@...asonboard.com>,
Kieran Bingham <kieran.bingham@...asonboard.com>,
Laurent Pinchart <laurent.pinchart@...asonboard.com>
Subject: Re: [v7 08/10] media: platform: Add C3 ISP driver
Hi Markus
Thanks for your reply.
On 2025/3/31 14:10, Markus Elfring wrote:
> [ EXTERNAL EMAIL ]
>
>>> You may occasionally put more than 60 characters into text lines
>>> of such a change description.
>> Are you sure the character limit for a single line is 60?
> It seems that you interpreted my patch review for this issue
> into other directions than intended.
>
>
>> I find the description 'wrap at 75 columns':
> I suggest to benefit from word wrapping adjustments accordingly.
OK, I see.
Will check the commit message.
>
> Regards,
> Markus
Powered by blists - more mailing lists