net: ravb: Move the update of ndev->features to ravb_set_features()
authorClaudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Wed, 14 Feb 2024 13:57:58 +0000 (15:57 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 16 Feb 2024 08:32:04 +0000 (08:32 +0000)
commit7bddccc9911cdff377e16b9a5a386721279b4438
treeb7f61c3f1ee7896db92cb946c6bc945d15a2abd6
parentbbf2345fa6582c5292bc5c537e7a29aad918be0c
net: ravb: Move the update of ndev->features to ravb_set_features()

Commit c2da9408579d ("ravb: Add Rx checksum offload support for GbEth")
introduced support for setting GbEth features. With this the IP-specific
features update functions update the ndev->features individually.

Next commits add runtime PM support for the ravb driver. The runtime PM
implementation will enable/disable the IP clocks on
the ravb_open()/ravb_close() functions. Accessing the IP registers with
clocks disabled blocks the system.

The ravb_set_features() function could be executed when the Ethernet
interface is closed so we need to ensure we don't access IP registers while
the interface is down when runtime PM support will be in place.

For these, move the update of ndev->features to ravb_set_features(). In
this way we update the ndev->features only when the IP-specific features
set function returns success and we can avoid code duplication when
introducing runtime PM registers protection.

Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/renesas/ravb_main.c