Re: [RFC PATCH 2/2] Documentation: devictree: Add macb mdio bindings

From: Andrew Lunn
Date: Fri May 13 2016 - 13:43:15 EST


Hi Harini

Is this backward compatible? Will devices using the old binding still
work?

/* Disable RX and TX (XXX: Should we halt the transmission
* more gracefully?)
*/
- macb_writel(bp, NCR, 0);
+ ctrl = macb_readl(bp, NCR);
+ ctrl &= ~(MACB_BIT(RE) | MACB_BIT(TE));
+ macb_writel(bp, NCR, ctrl);

/* Clear the stats registers (XXX: Update stats first?) */
- macb_writel(bp, NCR, MACB_BIT(CLRSTAT));
+ ctrl |= MACB_BIT(CLRSTAT);
+ macb_writel(bp, NCR, ctrl);

/* Clear all status flags */
macb_writel(bp, TSR, -1);

It is not clear to me what this part has to do with MDIO.

Andrew