On 8/23/2019 2:47 PM, Wei Hu (Xavier) wrote:
> This patch adds support for MAC PAUSE flow control and priority flow
> control of hns3 PMD driver. All user priorities(up) must be mapped to
> tc0 when MAC PAUSE flow control is enabled. Ups can be mapped to other
> tcs driver permit when PFC is enabled. Flow control function by default
> is turned off to ensure that app startup state is the same each time.

As far as I can see the patch both enable DCB and flow control, can you please
either split the patch or update the commit log to cover both features?

> 
> Signed-off-by: Huisong Li <lihuis...@huawei.com>
> Signed-off-by: Wei Hu (Xavier) <xavier.hu...@huawei.com>
> Signed-off-by: Chunsong Feng <fengchuns...@huawei.com>
> Signed-off-by: Min Hu (Connor) <humi...@huawei.com>
> Signed-off-by: Hao Chen <chenhao...@huawei.com>

<...>

>  static const struct eth_dev_ops hns3_eth_dev_ops = {
>       .dev_close          = hns3_dev_close,
>       .mtu_set            = hns3_dev_mtu_set,
>       .dev_infos_get          = hns3_dev_infos_get,
>       .fw_version_get         = hns3_fw_version_get,
> +     .flow_ctrl_get          = hns3_flow_ctrl_get,
> +     .flow_ctrl_set          = hns3_flow_ctrl_set,
> +     .priority_flow_ctrl_set = hns3_priority_flow_ctrl_set,

Can you please update .ini file in this patch and mark following features as
supported:
Flow control

>       .mac_addr_add           = hns3_add_mac_addr,
>       .mac_addr_remove        = hns3_remove_mac_addr,
>       .mac_addr_set           = hns3_set_default_mac_addr,
> @@ -2753,6 +2949,7 @@ static const struct eth_dev_ops hns3_eth_dev_ops = {
>       .reta_update            = hns3_dev_rss_reta_update,
>       .reta_query             = hns3_dev_rss_reta_query,
>       .filter_ctrl            = hns3_dev_filter_ctrl,
> +     .get_dcb_info           = hns3_get_dcb_info,

Can you please update .ini file in this patch and mark following features as
supported:
DCB

Reply via email to