Re: [PATCH v6 1/3] ARM: dts: nuvoton: Add memory controller node

[Paul Menzel]

Dear Medad,


Thank you for your reply.

Am 11.04.22 um 09:56 schrieb Medad Young:

[…]

> Paul Menzel 於 2022年4月9日 週六 下午1:57寫道：

> > Thank you for the patch.
> >
> > Am 22.03.22 um 04:01 schrieb Medad CChien:
> > > ECC must be configured in the BootBlock header.
> >
> > bootblock
> >
> > I search for *bootblock* in Linux and the git commit messages, and does
> > not seem to be a common term. Is that term used in the datasheet?
>
> Yes, bootblock is a bootloader of our SOC

Nice. Never heard of it before. Maybe add the project URL as reference 
for the ignorant like me.

> > > Then, you can read error counts via
> > > the EDAC kernel framework.
> >
> > Please reflow for 75 characters per line. (Also, there is no need to
> > break lines after a sentence, unless 75 characters are reached or a new
> > paragraph starts.)
> >
> > Tested on what board?
>
> I tested this on Nuvoton's BMC board

It would be nice to have that with the model number documented in the 
commit message.

> > > Signed-off-by: Medad CChien <ctcchien@xxxxxxxxxxx>
> >
> > Out of curiosity, is the first C in CChien the letter of your middle
> > name, or the last name really spelled with two capital letters in the
> > beginning?
>
> this is a special last name in chinese.
> my last name does really spell with two capital letters in the beginning.

Interesting. Thank you for teaching me new things.


Kind regards,

Paul