Re: [PATCH v8 02/10] edac: synopsys: Rename the static functions to a shorter name
From: Borislav Petkov
Date: Mon Oct 08 2018 - 14:03:47 EST
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.