[U-Boot] [PATCH v2 19/40] env: net: Move eth_parse_enetaddr() to net.c/h

Simon Glass sjg at chromium.org
Thu Sep 12 19:28:10 UTC 2019


Hi,

On Thu, 12 Sep 2019 at 12:28, Ondřej Jirman <megous at megous.com> wrote:
>
> Hi,
>
> On Thu, Sep 12, 2019 at 12:22:15PM -0600, Simon Glass wrote:
> > Hi,
> >
> > On Thu, 12 Sep 2019 at 10:59, Ondřej Jirman <megous at megous.com> wrote:
> > >
> > > Hello,
> > >
> > > On Thu, Aug 01, 2019 at 09:46:54AM -0600, Simon Glass wrote:
> > > > This function fits better with the network subsystem, so move it.
> > >
> > > Unfortunately, this breaks builds without CONFIG_NET. Reverting it
> > > fixes the issue.
> > >
> > >   LD      u-boot
> > > cmd/built-in.o: In function `eth_env_get_enetaddr':
> > > u-boot-v2019.10/cmd/nvedit.c:363: undefined reference to `eth_parse_enetaddr'
> > > make[1]: *** [u-boot-v2019.10/Makefile:1594: u-boot] Error 1
> > > make[1]: Leaving directory 'builds/.tmp/u-pc-5.4'
> > > make: *** [Makefile:148: sub-make] Error 2
> > >
> >
> > Which board is this please?
>
> It's orangepi_pc_defconfig with some menuconfig changes to disable CONFIG_NET.
>
> The issue is that by moving eth_parse_enetaddr to net/ if net is disabled, this
> function will not be available to nvedit.c.
>
> The board doesn't really matter.

The offending call is in setup_environment() in board/sunxi/board.c so
I think the board does matter. There might be something else I am
missing but I cannot see it at present.

Would you like to send a patch?

Regards,
Simon


More information about the U-Boot mailing list