> +
> +/* 0 = Clause 22, 1 = Clause 45 */
> +#define MDIO_MODE_BIT                                BIT(8)

How about calling this MDIO_MODE_C45

> +             /* Enter Clause 45 mode */
> +             data = readl(priv->membase + MDIO_MODE_REG);
> +
> +             data |= MDIO_MODE_BIT;
> +
> +             writel(data, priv->membase + MDIO_MODE_REG);

It then becomes clearer what this does.

Otherwise this looks O.K.

          Andrew

Reply via email to