Messages in this thread | | | From | Manish Narani <> | Subject | RE: [PATCH v7 2/7] edac: synps: Add platform specific structures for ddrc controller | Date | Wed, 19 Sep 2018 13:33:58 +0000 |
| |
Hi Boris,
> -----Original Message----- > From: Borislav Petkov [mailto:bp@alien8.de] > Sent: Wednesday, September 19, 2018 4:46 PM > To: Manish Narani <MNARANI@xilinx.com> > Cc: robh+dt@kernel.org; mark.rutland@arm.com; mchehab@kernel.org; > Michal Simek <michals@xilinx.com>; leoyang.li@nxp.com; > sudeep.holla@arm.com; amit.kucheria@linaro.org; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > edac@vger.kernel.org; linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH v7 2/7] edac: synps: Add platform specific structures for > ddrc controller > > On Wed, Sep 19, 2018 at 05:14:51AM +0000, Manish Narani wrote: > > > So I'm going to stop reviewing this patchset until you go through > > > the old review round again and make sure you've incorporated *all* > > > review feedback and haven't forgotten some, like the one above. > > > > > Okay, I will double check again for the comments from previous review > > and rectify the same in v8. Can you please review the remaining > > patches so that I can accommodate any changes for them in v8? > > Read again what I said: "I'm going to stop reviewing this patchset until you go > through the old review..." You can go through the old review and check your v7. > If you haven't missed anything, lemme know and I'll look at the rest.
Apart from this one, I have covered all the comments from the previous review.
Thanks, Manish
| |