zynqmp: remove CONFIG_BOOTP_SERVERIP
PATCH 1/1 zynqmp: remove CONFIG_BOOTP_SERVERIP
When the CONFIG_BOOTP_SERVERIP option is set, we ignore all dhcp values for the tftp server and use our own serverip and file name instead.
This is usually not what we want and I doubt it's set for a good reason on ZynqMP. It definitely hurts if we want to support uEFI PXE boot on it. So just remove the option for now.
--- include/configs/xilinx_zynqmp.h | 1 - 1 file changed, 1 deletion(-)
include/configs/xilinx_zynqmp.h +0 -1
@@ -76,7 +76,6 @@#define CONFIG_BOOTP_GATEWAY #define CONFIG_BOOTP_HOSTNAME #define CONFIG_BOOTP_MAY_FAIL -#define CONFIG_BOOTP_SERVERIP #define CONFIG_BOOTP_DNS #define CONFIG_BOOTP_PXE #define CONFIG_BOOTP_SUBNETMASK
Signature · 1 line
1.8.5.6
PATCH 1/1 zynqmp: remove CONFIG_BOOTP_SERVERIP
Quoted 19 lines · Alexander Graf
When the
CONFIG_BOOTP_SERVERIPoption is set, we ignore all dhcp values for the tftp server and use our own serverip and file name instead.This is usually not what we want and I doubt it's set for a good reason on ZynqMP. It definitely hurts if we want to support uEFI PXE boot on it. So just remove the option for now.
Signed-off-by Alexander Graf <[email protected]>--- include/configs/xilinx_zynqmp.h | 1 - 1 file changed, 1 deletion(-)include/configs/xilinx_zynqmp.h +0
-0
include/configs/xilinx_zynqmp.h line 81
Quoted 7 lines
include/configs/xilinx_zynqmp.h +0
-1@@ -76,7 +76,6 @@#define CONFIG_BOOTP_GATEWAY #define CONFIG_BOOTP_HOSTNAME #define CONFIG_BOOTP_MAY_FAIL -#define CONFIG_BOOTP_SERVERIP #define CONFIG_BOOTP_DNS #define CONFIG_BOOTP_PXE #define CONFIG_BOOTP_SUBNETMASK
Looks reasonable.
Applied with changed subject ARM64: zynqmp: Remove...
Thanks,
Michal