Thread (12 messages) 12 messages, 4 authors, 2022-03-31

Re: [PATCH v2 net-next 1/6] net: dsa: reorder PHY initialization with MTU setup in slave.c

From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2022-01-06 03:28:05


On 1/5/2022 3:11 PM, Vladimir Oltean wrote:
In dsa_slave_create() there are 2 sections that take rtnl_lock():
MTU change and netdev registration. They are separated by PHY
initialization.

There isn't any strict ordering requirement except for the fact that
netdev registration should be last. Therefore, we can perform the MTU
change a bit later, after the PHY setup. A future change will then be
able to merge the two rtnl_lock sections into one.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
-- 
Florian
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help