net/hns3: fix firmware compatibility configuration
authorHuisong Li <lihuisong@huawei.com>
Tue, 13 Apr 2021 13:47:14 +0000 (21:47 +0800)
committerFerruh Yigit <ferruh.yigit@intel.com>
Thu, 15 Apr 2021 00:55:04 +0000 (02:55 +0200)
commit2192c428f9a6a92a8d8913fd9cf727ffa5810245
treee8bf3ff0ed4a5190e76c40926115b8084e396673
parentd4fdb71a0e7beb95b6e2c6ffe13fb159904317bf
net/hns3: fix firmware compatibility configuration

The firmware compatibility configuration in PF driver is used to
maintain the compatibility of some features of the driver and
firmware, and requires firmware to enable these features. Currently,
the configuration is in hns3_init_hardware(), which is a little back.
Because firmware may clear some configurations (such as, MAC related)
after receiving the command. And firmware can not be aware of some
default initializations (such as, flow control) before executing the
command to set the copper PHY when the PHY is controlled by firmware.
Therefore, it is recommended that no other hardware resources are
configured before the compatibility configuration. And it should be
moved to hns3_cmd_init(), which is responsible for the firmware
command initialization of driver.

In addition, the driver needs to perform corresponding processing
if the command fails to be sent.
1) If firmware fails to take over the copper PHY, the copper port fails
   to initialize.
2) If fails to enable the report of link events, the device does not
   support the LSC capability.

Fixes: bff6ebfe30d4 ("net/hns3: refactor PF LSC event report")
Fixes: bac6a0644121 ("net/hns3: fix link status change from firmware")
Cc: stable@dpdk.org
Signed-off-by: Huisong Li <lihuisong@huawei.com>
Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
drivers/net/hns3/hns3_cmd.c
drivers/net/hns3/hns3_ethdev.c