[PATCH] Revert "net: gem: Disable broadcast setting"

Ramon Fried rfried.dev at gmail.com
Sat Jan 15 17:14:34 CET 2022


On Fri, Jan 14, 2022 at 2:15 PM Michal Simek <michal.simek at xilinx.com> wrote:
>
> This reverts commit eafdcda4a854932c0319656de7bf3f017f17ae67.
>
> The main reason is that QEMU is using BOOTP protocol which is sending DHCP
> Offer to a broadcast address that's why it can't be disabled.
> DHCP protocol has no issue because it returns directly to client MAC
> address.
> Both of these options are described in RFC951
> (https://datatracker.ietf.org/doc/html/rfc951#section-4)
>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
>  drivers/net/zynq_gem.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/zynq_gem.c b/drivers/net/zynq_gem.c
> index f238811786c6..3118d1472669 100644
> --- a/drivers/net/zynq_gem.c
> +++ b/drivers/net/zynq_gem.c
> @@ -62,7 +62,6 @@
>  #define ZYNQ_GEM_NWCFG_SPEED100                0x00000001 /* 100 Mbps operation */
>  #define ZYNQ_GEM_NWCFG_SPEED1000       0x00000400 /* 1Gbps operation */
>  #define ZYNQ_GEM_NWCFG_FDEN            0x00000002 /* Full Duplex mode */
> -#define ZYNQ_GEM_NWCFG_NO_BRDC         BIT(5) /* No broadcast */
>  #define ZYNQ_GEM_NWCFG_FSREM           0x00020000 /* FCS removal */
>  #define ZYNQ_GEM_NWCFG_SGMII_ENBL      0x08000000 /* SGMII Enable */
>  #define ZYNQ_GEM_NWCFG_PCS_SEL         0x00000800 /* PCS select */
> @@ -80,7 +79,6 @@
>
>  #define ZYNQ_GEM_NWCFG_INIT            (ZYNQ_GEM_DBUS_WIDTH | \
>                                         ZYNQ_GEM_NWCFG_FDEN | \
> -                                       ZYNQ_GEM_NWCFG_NO_BRDC | \
>                                         ZYNQ_GEM_NWCFG_FSREM | \
>                                         ZYNQ_GEM_NWCFG_MDCCLKDIV)
>
> --
> 2.34.1
>
Reviewed-by: Ramon Fried <rfried.dev at gmail.com>


More information about the U-Boot mailing list