* [U-Boot] [PATCH] zynqmp: remove CONFIG_BOOTP_SERVERIP
@ 2016-06-01 20:41 Alexander Graf
2016-06-02 9:26 ` Michal Simek
0 siblings, 1 reply; 2+ messages in thread
From: Alexander Graf @ 2016-06-01 20:41 UTC (permalink / raw)
To: u-boot
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.
Signed-off-by: Alexander Graf <agraf@suse.de>
---
include/configs/xilinx_zynqmp.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h
index 6800d34..9c55c08 100644
--- a/include/configs/xilinx_zynqmp.h
+++ b/include/configs/xilinx_zynqmp.h
@@ -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
--
1.8.5.6
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [U-Boot] [PATCH] zynqmp: remove CONFIG_BOOTP_SERVERIP
2016-06-01 20:41 [U-Boot] [PATCH] zynqmp: remove CONFIG_BOOTP_SERVERIP Alexander Graf
@ 2016-06-02 9:26 ` Michal Simek
0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2016-06-02 9:26 UTC (permalink / raw)
To: u-boot
On 1.6.2016 22:41, Alexander Graf wrote:
> 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.
>
> Signed-off-by: Alexander Graf <agraf@suse.de>
> ---
> include/configs/xilinx_zynqmp.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h
> index 6800d34..9c55c08 100644
> --- a/include/configs/xilinx_zynqmp.h
> +++ b/include/configs/xilinx_zynqmp.h
> @@ -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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-06-02 9:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-01 20:41 [U-Boot] [PATCH] zynqmp: remove CONFIG_BOOTP_SERVERIP Alexander Graf
2016-06-02 9:26 ` Michal Simek
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.