[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
<BL0PR04MB65640AD67700DA31A3E192D6FC4B2@BL0PR04MB6564.namprd04.prod.outlook.com>
Date: Tue, 29 Oct 2024 21:33:24 +0000
From: Avri Altman <Avri.Altman@....com>
To: Bart Van Assche <bvanassche@....org>, "Martin K . Petersen"
<martin.petersen@...cle.com>
CC: "linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v2 1/2] scsi: ufs: core: Introduce a new clock_gating lock
> On 10/29/24 11:39 AM, Avri Altman wrote:
> >> On 10/29/24 3:29 AM, Avri Altman wrote:
> >> and do not exceed the 80 column limit. git clang-format HEAD^ can
> >> help with restricting code to the 80 column limit.
> > Isn't the 80 characters restriction was changed long ago to 100 characters?
> > I always use strict checkpatch and doesn't get any warning about this.
>
> 80 columns is the limit currently used in the UFS driver so I think we should stick
> to this limit to keep formatting of the UFS driver consistent. Here are two
> additional arguments:
> * From Documentation/process/coding-style.rst:
>
> The preferred limit on the length of a single line is 80 columns.
>
> * From .clang-format:
>
> ColumnLimit: 80
Done.
Thanks,
Avri
>
>
> Thanks,
>
> Bart.
Powered by blists - more mailing lists