Re: [PATCH net-next v3 2/2] net: mdio: add a driver for PIC64-HPSC/HX MDIO controller

Russell King (Oracle)
2026-03-31T12:57:16Z

On Tue, Mar 31, 2026 at 05:38:54AM -0700, Charles Perry wrote:
> + if (!(bus->phy_ignore_ta_mask & 1 << mii_id) &&
> + !FIELD_GET(MDIO_READOK_BIT, val)) {
> + dev_dbg(&bus->dev, "READOK bit cleared\n");
> + return -EIO;
> + }
> +
> + ret = FIELD_GET(MDIO_RDATA_MASK, val);
> +
> + return ret;

You don't need "ret" here, this can simply be:

return FIELD_GET(MDIO_RDATA_MASK, val);

...

> + writel(MDIO_TRIGGER_BIT | FIELD_PREP(MDIO_REG_DEV_ADDR_MASK, regnum) |
> + FIELD_PREP(MDIO_PHY_PRT_ADDR_MASK, mii_id) |
> + FIELD_PREP(MDIO_OPERATION_MASK, MDIO_OPERATION_WRITE) |
> + FIELD_PREP(MDIO_START_OF_FRAME_MASK, 1),
> + priv->regs + MDIO_REG_FRAME_CFG_2);

Shouldn't this wait for the write to complete?

Thanks.

--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!