[U-Boot] [PATCH 1/6] net: macb: sync header definitions as taken from Linux

Joe Hershberger joe.hershberger at ni.com
Mon Jul 15 20:48:10 UTC 2019


On Mon, Jul 15, 2019 at 3:26 PM Ramon Fried <rfried.dev at gmail.com> wrote:
>
> On Mon, Jul 15, 2019 at 11:19 PM Ramon Fried <rfried.dev at gmail.com> wrote:
> >
> > On Mon, Jul 15, 2019 at 9:31 PM Joe Hershberger <joe.hershberger at ni.com> wrote:
> > >
> > > On Tue, Jun 11, 2019 at 10:51 AM Ramon Fried <rfried.dev at gmail.com> wrote:
> > > >
> > > > Few registers and bits were added by Cadence and
> > > > they were not updated in the headers.
> > > > Take the latest definitions as defined in Linux
> > > > header (5.1) that also includes some comments
> > > > about existing registers.
> > > >
> > > > One register was improperly named (UR), fix that.
> > >
> > > This is breaking the build.  "UR" is still referred to in
> > > drivers/net/macb.c line 805.
> > >
> > > Did you build test this?
> > Well of course, but apparently I screwed this one in rebasing, prior to submit.
> > Didn't run buildman afterwards. sorry.
> > Will issue a revised patchset.
> > Sorry.
> > Ramon.
> Actually, now that I look at this. it looks fine. the patch replaces
> the line you mentioned:
> https://patchwork.ozlabs.org/patch/1114041/
> Can take a look ?

It addresses some of them, but not the ones a few lines below.

> > >
> > > >
> > > > Signed-off-by: Ramon Fried <rfried.dev at gmail.com>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot


More information about the U-Boot mailing list