On Thu, Jun 14, 2018 at 5:04 AM, Alexander Graf <[email protected]> wrote: > The ax25-ae350 target currently uses CONFIG_BOOTP_SERVERIP which means we > ignore the DHCP provided TFTP ip address. This breaks every case where we > do now provide a serverip environment variable. > > Instead, let's use the new CONFIG_BOOT_PREFER_SERVERIP option to fall back > to the DHCP provided TFTP IP if no serverip environment variable is set. > > Signed-off-by: Alexander Graf <[email protected]>
Reviewed-by: Joe Hershberger <[email protected]> > --- > configs/ax25-ae350_defconfig | 1 + > include/configs/ax25-ae350.h | 1 - > 2 files changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configs/ax25-ae350_defconfig b/configs/ax25-ae350_defconfig > index fc04c87485..a328555af6 100644 > --- a/configs/ax25-ae350_defconfig > +++ b/configs/ax25-ae350_defconfig > @@ -40,3 +40,4 @@ CONFIG_DM_SPI=y > CONFIG_ATCSPI200_SPI=y > CONFIG_TIMER=y > CONFIG_ATCPIT100_TIMER=y > +CONFIG_BOOTP_PREFER_SERVERIP=y > diff --git a/include/configs/ax25-ae350.h b/include/configs/ax25-ae350.h > index b1ca5ac11a..b230896734 100644 > --- a/include/configs/ax25-ae350.h > +++ b/include/configs/ax25-ae350.h > @@ -11,7 +11,6 @@ > * CPU and Board Configuration Options > */ > #define CONFIG_BOOTP_SEND_HOSTNAME > -#define CONFIG_BOOTP_SERVERIP Feel like moving this to Kconfig? > > /* > * Miscellaneous configurable options > -- > 2.12.3 > > _______________________________________________ > U-Boot mailing list > [email protected] > https://lists.denx.de/listinfo/u-boot _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
