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

Ramon Fried rfried.dev at gmail.com
Tue Jul 16 05:02:40 UTC 2019


On Mon, Jul 15, 2019 at 11:48 PM Joe Hershberger <joe.hershberger at ni.com> wrote:
>
> 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.
Just cherry-picked to a brand new master branch. there are no UR left
on the file
and it compiles correctly.
Did you apply something else beforehand ?
>
> > > >
> > > > >
> > > > > 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