Re: [PATCH v5] LoongArch: Fix CONFIG_CMDLINE_EXTEND and CONFIG_CMDLINE_BOOTLOADER handling

[‎ donmor]

在 2023-07-19星期三的 18:29 +0800，WANG Xuerui写道：
> On 2023/7/19 15:22, Huacai Chen wrote:
> > Hi, Markus,
> > 
> > On Wed, Jul 19, 2023 at 2:51 PM Markus Elfring <Markus.Elfring@xxxxxx> wrote:
> > > >                                                    …, so this patch add
> > > > some code to fix it.
> > > 
> > > Would you like to avoid a typo here?
> > > 
> > > Will any other imperative change description variant become more helpful?
> > Thank you for pointing this out, but since Zhihong is the original
> > author, I don't want to completely rewrite the commit message, so just
> > fix the typo...
> 
> AFAICT the commit message is totally uninformative even if "an 
> imperative change description" were used. It basically:
> 
> 1. repeated the patch title,
> 2. spent one sentence only for mentioning a function name without giving 
> any more information,
> 3. mentioned why some change was not necessary due to some other 
> existing code, but not explicitly calling that part out, then
> 4. finished with a sentence that boiled down to "we should do the 
> similar thing".
> 
> My take:
> 
>  > Subject: Fix CMDLINE_EXTEND and CMDLINE_BOOTLOADER on non-FDT systems
>  >
>  > On FDT systems these command line processing are already taken care of
>  > by early_init_dt_scan_chosen(). Add similar handling to the non-FDT
>  > code path to allow these config options to work for non-FDT boxes too.
> 
> Would this sound better?
> 
Xuerui's take is fine. Do I need to make a v6 patch?

donmor