On Tue, Jul 9, 2019 at 3:50 AM George Hung (洪忠敬) <[email protected]> wrote: > > Hi Rob, > > > > -----Original Message----- > > From: openbmc > > [mailto:[email protected]] On > > Behalf Of Rob Herring > > Sent: Tuesday, July 09, 2019 9:41 AM > > To: George Hung > > Cc: Mark Rutland; Linus Walleij; Tali Perry; [email protected]; > > [email protected]; [email protected]; [email protected]; > > [email protected]; [email protected]; Borislav Petkov; > > [email protected]; Jonathan Cameron; Mauro Carvalho Chehab; > > linux-edac; Patrick Venture; Nicolas Ferre; linux-kernel; James Morse; Greg > > Kroah-Hartman; [email protected] > > Subject: Re: [PATCH 5.2 v2 2/2] dt-binding: edac: add NPCM ECC > > documentation > > > > On Wed, Jun 05, 2019 at 10:12:53PM +0800, George Hung wrote: > > > Add device tree documentation for Nuvoton BMC ECC > > > > > > Signed-off-by: George Hung <[email protected]> > > > --- > > > .../bindings/edac/npcm7xx-sdram-edac.txt | 17 > > +++++++++++++++++ > > > 1 file changed, 17 insertions(+) > > > create mode 100644 > > > Documentation/devicetree/bindings/edac/npcm7xx-sdram-edac.txt > > > > > > diff --git > > > a/Documentation/devicetree/bindings/edac/npcm7xx-sdram-edac.txt > > > b/Documentation/devicetree/bindings/edac/npcm7xx-sdram-edac.txt > > > new file mode 100644 > > > index 000000000000..dd4dac59a5bd > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/edac/npcm7xx-sdram-edac.txt > > > @@ -0,0 +1,17 @@ > > > +Nuvoton NPCM7xx SoC EDAC device driver > > > + > > > +The Nuvoton NPCM7xx SoC supports DDR4 memory with/without ECC and > > the > > > +driver uses the EDAC framework to implement the ECC detection and > > corrtection. > > > + > > > +Required properties: > > > +- compatible: should be "nuvoton,npcm7xx-sdram-edac" > > > > Is this for the whole SDRAM controller or just ECC related registers? > > In the former case, the naming should just reflect the block name and not a > > Linux term. > > Sorry for confused naming, the address space is for the whole memory > controller registers indeed, > but the driver only uses the ECC related registers. > Should I change the name to "nuvoton,npcm7xx-edac" ?
No, you should drop the 'edac' part. The DT describes the h/w, not what one driver (currently) uses. Rob
