Messages in this thread | | | Date | Mon, 11 Apr 2022 10:12:32 +0200 | Subject | Re: [PATCH v6 1/3] ARM: dts: nuvoton: Add memory controller node | From | 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@nuvoton.com> >> >> 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
| |