Messages in this thread | | | Date | Mon, 8 Oct 2018 20:03:38 +0200 | From | Borislav Petkov <> | 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.
| |