[PATCH] net: bootp: Drop unused code
Ilias Apalodimas
ilias.apalodimas at linaro.org
Wed Mar 25 11:29:08 CET 2026
On Wed, 25 Mar 2026 at 03:07, Marek Vasut
<marek.vasut+renesas at mailbox.org> wrote:
>
> This code is surely unused and there are not even commented out
> references to the function name. Drop the code.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
> ---
Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> Cc: "Kory Maincent (TI.com)" <kory.maincent at bootlin.com>
> Cc: Andrew Goodbody <andrew.goodbody at linaro.org>
> Cc: Heiko Schocher <hs at nabladev.com>
> Cc: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Cc: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> Cc: Jerome Forissier <jerome.forissier at arm.com> <jerome at forissier.org>
> Cc: Mattijs Korpershoek <mkorpershoek at kernel.org>
> Cc: Michael Trimarchi <michael at amarulasolutions.com>
> Cc: Mikhail Kshevetskiy <mikhail.kshevetskiy at iopsys.eu>
> Cc: Paul Barker <paul.barker.ct at bp.renesas.com>
> Cc: Tom Rini <trini at konsulko.com>
> Cc: u-boot at lists.denx.de
> ---
> net/bootp.c | 17 -----------------
> 1 file changed, 17 deletions(-)
>
> diff --git a/net/bootp.c b/net/bootp.c
> index 1d905a01a47..8976936b184 100644
> --- a/net/bootp.c
> +++ b/net/bootp.c
> @@ -87,23 +87,6 @@ static u8 dhcp_option_overload;
> #define OVERLOAD_SNAME 2
> static void dhcp_handler(uchar *pkt, unsigned dest, struct in_addr sip,
> unsigned src, unsigned len);
> -
> -/* For Debug */
> -#if 0
> -static char *dhcpmsg2str(int type)
> -{
> - switch (type) {
> - case 1: return "DHCPDISCOVER"; break;
> - case 2: return "DHCPOFFER"; break;
> - case 3: return "DHCPREQUEST"; break;
> - case 4: return "DHCPDECLINE"; break;
> - case 5: return "DHCPACK"; break;
> - case 6: return "DHCPNACK"; break;
> - case 7: return "DHCPRELEASE"; break;
> - default: return "UNKNOWN/INVALID MSG TYPE"; break;
> - }
> -}
> -#endif
> #endif
>
> static void bootp_add_id(ulong id)
> --
> 2.53.0
>
More information about the U-Boot
mailing list