Re: [PATCH v5 net-next 1/5] net: dsa: propagate extack to ds->ops->port_hsr_join()

From: Florian Fainelli
Date: Thu Sep 21 2023 - 16:30:54 EST


On 9/20/23 04:43, Lukasz Majewski wrote:
From: Vladimir Oltean <vladimir.oltean@xxxxxxx>

Drivers can provide meaningful error messages which state a reason why
they can't perform an offload, and dsa_slave_changeupper() already has
the infrastructure to propagate these over netlink rather than printing
to the kernel log. So pass the extack argument and modify the xrs700x
driver's port_hsr_join() prototype.

Also take the opportunity and use the extack for the 2 -EOPNOTSUPP cases
from xrs700x_hsr_join().

Signed-off-by: Vladimir Oltean <vladimir.oltean@xxxxxxx>
Signed-off-by: Lukasz Majewski <lukma@xxxxxxx>

Reviewed-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx>
--
Florian