Re: [PATCH net-next v3 1/6] net: dsa: qca: ar9331: reorder MDIO write sequence
From: Vladimir Oltean <olteanv@gmail.com>
Date: 2021-08-02 14:18:07
Also in:
lkml, netdev
From: Vladimir Oltean <olteanv@gmail.com>
Date: 2021-08-02 14:18:07
Also in:
lkml, netdev
On Mon, Aug 02, 2021 at 03:10:32PM +0200, Oleksij Rempel wrote:
In case of this switch we work with 32bit registers on top of 16bit bus. Some registers (for example access to forwarding database) have trigger bit on the first 16bit half of request and the result + configuration of request in the second half. Without this patch, we would trigger database operation and overwrite result in one run. To make it work properly, we should do the second part of transfer before the first one is done. So far, this rule seems to work for all registers on this switch. Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> Reviewed-by: Andrew Lunn <andrew@lunn.ch> ---
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>