[U-Boot] [PATCH 3/9] net: Move the DHCP command below the BOOTP command

Chris Packham judge.packham at gmail.com
Thu Mar 29 03:14:33 UTC 2018


On Thu, Mar 29, 2018 at 9:51 AM, Joe Hershberger <joe.hershberger at ni.com> wrote:
> Move DHCP to directly follow BOOTP so that Kconfig can show the
> dependency as a hierarchy.
>
> Signed-off-by: Joe Hershberger <joe.hershberger at ni.com>
> ---

I never quite understood the distinction between bootp and dhcp. I
know technically bootp was an earlier version of what eventually
became bootp but isn't dhcp fully backwards compatible with the bootp
protocol? Code-wise it looks like bootp is dhcp without the option
processing. Do we save much by keeping the separate?

Regardless ...

Reviewed-by: Chris Packham <judge.packham at gmail.com>

>
>  cmd/Kconfig | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index d714f73..7ef9501 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -1024,6 +1024,12 @@ config CMD_BOOTP
>         help
>           bootp - boot image via network using BOOTP/TFTP protocol
>
> +config CMD_DHCP
> +       bool "dhcp"
> +       depends on CMD_BOOTP
> +       help
> +         Boot image via network using DHCP/TFTP protocol
> +
>  config BOOTP_BOOTPATH
>         bool "Enable BOOTP BOOTPATH"
>         depends on CMD_BOOTP
> @@ -1097,12 +1103,6 @@ config CMD_RARP
>         help
>           Boot image via network using RARP/TFTP protocol
>
> -config CMD_DHCP
> -       bool "dhcp"
> -       depends on CMD_BOOTP
> -       help
> -         Boot image via network using DHCP/TFTP protocol
> -
>  config CMD_PXE
>         bool "pxe"
>         select MENU
> --
> 1.7.11.5
>
> _______________________________________________
> 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