Re: [PATCH v8 02/10] edac: synopsys: Rename the static functions to a shorter name

2018-10-08 Thread Borislav Petkov
On Mon, Oct 08, 2018 at 01:45:28PM +, 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 >

RE: [PATCH v8 02/10] edac: synopsys: Rename the static functions to a shorter name

2018-10-08 Thread Manish Narani
Hi Boris, Thanks a lot for the review. > -Original Message- > From: Borislav Petkov [mailto:b...@alien8.de] > Sent: Friday, October 5, 2018 2:06 AM > Subject: Re: [PATCH v8 02/10] edac: synopsys: Rename the static functions to a > shorter name > > On Thu, Oct 0

Re: [PATCH v8 02/10] edac: synopsys: Rename the static functions to a shorter name

2018-10-04 Thread Borislav Petkov
On Thu, Oct 04, 2018 at 09:05:20PM +0530, Manish Narani wrote: > Rename the static functions to a shorter name. Since this is Synopsys > EDAC driver, better to remove unnecessary 'synps_' prefix in function > names. > > Signed-off-by: Manish Narani > --- > drivers/edac/synopsys_edac.c | 56 > ++