On Mon, Feb 26, 2018 at 10:49 PM, Calvin Johnson <calvin.john...@nxp.com> wrote: > Hi Joe, > >> -----Original Message----- >> From: Joe Hershberger [mailto:joe.hershber...@ni.com] >> Sent: Tuesday, February 27, 2018 1:31 AM >> To: Calvin Johnson <calvin.john...@nxp.com> >> Cc: joe.hershber...@ni.com; u-boot@lists.denx.de; Anji Jagarlmudi >> <anji.jagarlm...@nxp.com> >> Subject: Re: [U-Boot] [PATCH v2 00/13] LS1012A PFE driver patch series >> >> Hi Calvin, >> >> On Wed, Feb 21, 2018 at 10:32 PM, Calvin Johnson <calvin.john...@nxp.com> >> wrote: >> > Hi Joe, >> > >> >> -----Original Message----- >> >> From: Calvin Johnson [mailto:calvin.john...@nxp.com] >> >> Sent: Thursday, February 15, 2018 7:47 PM >> >> To: u-boot@lists.denx.de >> >> Cc: joe.hershber...@ni.com; York Sun <york....@nxp.com>; Anji Jagarlmudi >> >> <anji.jagarlm...@nxp.com>; Calvin Johnson <calvin.john...@nxp.com> >> >> Subject: [PATCH v2 00/13] LS1012A PFE driver patch series >> >> >> > >> > <snip> >> > >> >> Changes in v2 series: >> >> 1. PFE patches submitted on top of this base patch are now merged to >> >> this patch. >> >> 2. Platform changes are segregated into different patches. >> >> 3. Network enabled on 2g5rdb platform >> >> 4. Moved from legacy to new driver model. >> > >> > Gentle reminder. Hope the PFE patches didn't miss your sight. >> >> I tried applying your series and it had many checkpatch issues. Please >> resolve those and resend. >> >> Thanks, >> -Joe >> >> ----------------- >> >> >> Fetched U-Boot-1-9-drivers-net-pfe_eth-LS1012A-PFE-driver-introduction >> (823206.mbox) >> 823206.mbox:46: WARNING: added, moved or deleted file(s), does >> MAINTAINERS need updating? >> 823206.mbox:534: CHECK: Avoid CamelCase: <Elf32_Shdr> >> 823206.mbox:551: CHECK: No space is necessary after a cast >> 823206.mbox:669: CHECK: spaces preferred around that '&' (ctx:ExV) > > <snip> > > Looks like you have run checkpatch on the v1 series (9 patches). > I had submitted the v2 series (13 patches). > > However, in the v2 series, there are some checkpatch warnings like ' line > over 80 characters' > which I had to ignore due to valid reasons. If there is any concern on these, > I can work on them > after discussion.
Sorry about that - Only your V1 was assigned to me in patchwork, so I missed it. Looks like V2 is assigned to York at the moment, but I'm going to assign it to myself since it is all about networking. Cheers, -Joe _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot