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.