[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20181008180338.GB5919@zn.tnic>
Date: Mon, 8 Oct 2018 20:03:38 +0200
From: Borislav Petkov <bp@...en8.de>
To: Manish Narani <MNARANI@...inx.com>
Cc: "robh+dt@...nel.org" <robh+dt@...nel.org>,
"mark.rutland@....com" <mark.rutland@....com>,
Michal Simek <michals@...inx.com>,
"mchehab@...nel.org" <mchehab@...nel.org>,
"sudeep.holla@....com" <sudeep.holla@....com>,
"amit.kucheria@...aro.org" <amit.kucheria@...aro.org>,
"leoyang.li@....com" <leoyang.li@....com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-edac@...r.kernel.org" <linux-edac@...r.kernel.org>
Subject: Re: [PATCH v8 02/10] edac: synopsys: Rename the static functions to
a shorter name
On Mon, Oct 08, 2018 at 01:45:28PM +0000, Manish Narani wrote:
> > /**
> > - * synps_edac_get_eccstate - Return the controller ecc enable/disable status
> > - * @base: Pointer to the ddr memory controller base address
> > + * get_ecc_state - Return the controller ECC enable/disable status
>
> Nit: ecc --> ECC correction can be moved to Comments correction patch. Minor. Can keep it here too.
>
All fixed except this one - this one can remain here because the patch
is touching the line anyway.
Thx.
--
Regards/Gruss,
Boris.
Good mailing practices for 400: avoid top-posting and trim the reply.
Powered by blists - more mailing lists