Re: [PATCH] docs: submit-checklist: structure by category

2024-02-28 Thread Randy Dunlap
Hi Lukas, Sorry about the (my) delay. On 2/26/24 02:46, Lukas Bulwahn wrote: > > Documentation/process/submit-checklist.rst | 157 +++-- > 1 file changed, 84 insertions(+), 73 deletions(-) > > diff --git a/Documentation/process/submit-checklist.rst > b/Documentation/process/

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-28 Thread Jonathan Corbet
Lukas Bulwahn writes: > While going through the submit checklist, the list order seemed rather > random, probably just by historical coincidences of always adding yet the > next point someone thought of at the end of the list. > > Structure and order them by the category of such activity, > revie

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-27 Thread Geert Uytterhoeven
Hi Lukas, On Tue, Feb 27, 2024 at 12:04 PM Lukas Bulwahn wrote: > On Tue, Feb 27, 2024 at 9:57 AM Geert Uytterhoeven > wrote: > > On Tue, Feb 27, 2024 at 1:41 AM Randy Dunlap wrote: > > > > - Concerning checking with tools, checkpatch probably still makes > > > > sense; > > > > it point

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-27 Thread Lukas Bulwahn
On Tue, Feb 27, 2024 at 9:57 AM Geert Uytterhoeven wrote: > > Hi Randy, > > On Tue, Feb 27, 2024 at 1:41 AM Randy Dunlap wrote: > > > - Concerning checking with tools, checkpatch probably still makes sense; > > > it pointed out in several places. If sparse and checkstack are really > > >

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-27 Thread Geert Uytterhoeven
Hi Randy, On Tue, Feb 27, 2024 at 1:41 AM Randy Dunlap wrote: > > - Concerning checking with tools, checkpatch probably still makes sense; > > it pointed out in several places. If sparse and checkstack are really > > the next two tools to point out, I am not so sure about. > > I doubt t

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-27 Thread Lukas Bulwahn
On Tue, Feb 27, 2024 at 1:41 AM Randy Dunlap wrote: > > Hi Lukas, > > I'll review the file changes separately. This is just replying > to the patch description comments. > > > On 2/26/24 02:46, Lukas Bulwahn wrote: (snipped) > > > Note that the previous first point still remains the first list eve

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-26 Thread Lukas Bulwahn
Hi Jani, On Mon, Feb 26, 2024 at 1:48 PM Jani Nikula wrote: > > On Mon, 26 Feb 2024, Lukas Bulwahn wrote: > > diff --git a/Documentation/process/submit-checklist.rst > > b/Documentation/process/submit-checklist.rst > > index b1bc2d37bd0a..7d8dba942fe8 100644 > > --- a/Documentation/process/subm

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-26 Thread Randy Dunlap
Hi Lukas, I'll review the file changes separately. This is just replying to the patch description comments. On 2/26/24 02:46, Lukas Bulwahn wrote: > While going through the submit checklist, the list order seemed rather > random, probably just by historical coincidences of always adding yet the

Re: [PATCH] docs: submit-checklist: structure by category

2024-02-26 Thread Jani Nikula
On Mon, 26 Feb 2024, Lukas Bulwahn wrote: > diff --git a/Documentation/process/submit-checklist.rst > b/Documentation/process/submit-checklist.rst > index b1bc2d37bd0a..7d8dba942fe8 100644 > --- a/Documentation/process/submit-checklist.rst > +++ b/Documentation/process/submit-checklist.rst > @@ -