On Sat, 2 Nov 2024 at 12:57, Bernhard Beschow <[email protected]> wrote:
>
> Turns 0x70 into 0xe0 (== 0x70 << 1) which adds the missing MII_ANLPAR_TX and
> fixes the MSB of selector field to be zero, as specified in the datasheet.
>
> Fixes: 2a424990170b "LAN9118 emulation"
> Signed-off-by: Bernhard Beschow <[email protected]>
> ---
>  hw/net/lan9118_phy.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/net/lan9118_phy.c b/hw/net/lan9118_phy.c
> index d2dcd732ac..d8fc9ddd3a 100644
> --- a/hw/net/lan9118_phy.c
> +++ b/hw/net/lan9118_phy.c
> @@ -54,7 +54,7 @@ uint16_t lan9118_phy_read(Lan9118PhyState *s, int reg)
>          val = s->advertise;
>          break;
>      case 5: /* Auto-neg Link Partner Ability */
> -        val = 0x0f71;
> +        val = 0x0fe1;
>          break;
>      case 6: /* Auto-neg Expansion */
>          val = 1;

Reviewed-by: Peter Maydell <[email protected]>

thanks
-- PMM

Reply via email to