Re: [PATCH v7 4/7] edac: synopsys: Add macro defines for ZynqMP DDRC

2018-09-24 Thread Borislav Petkov
On Mon, Sep 17, 2018 at 07:55:02PM +0530, Manish Narani wrote: > Add macro defines for ZynqMP DDR controller. These macros will be used > for ZyqnMP ECC operations. > > Signed-off-by: Manish Narani > --- > drivers/edac/synopsys_edac.c | 168 > +++ > 1 fil

RE: [PATCH v7 4/7] edac: synopsys: Add macro defines for ZynqMP DDRC

2018-09-24 Thread Manish Narani
p.com; > sudeep.ho...@arm.com; amit.kuche...@linaro.org; > devicet...@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > e...@vger.kernel.org; linux-arm-ker...@lists.infradead.org > Subject: Re: [PATCH v7 4/7] edac: synopsys: Add macro defines for ZynqMP > DDRC > > On Mon, S

Re: [PATCH v7 4/7] edac: synopsys: Add macro defines for ZynqMP DDRC

2018-09-24 Thread Borislav Petkov
On Mon, Sep 24, 2018 at 10:29:44AM +, Manish Narani wrote: > > Is there some significance here or can they all be vertically aligned? > Should I increase the alignment of the existing macros in the 'code > formatting' patches? New macros will need an increased indentation. If there is no signi