RE: [PATCH rtw-next v2 05/14] wifi: rtw89: Add rtw8851b_hfc_param_ini_usb

[Ping-Ke Shih <pkshih@realtek.com>]

Bitterblue Smith <rtl8821cerfe2@gmail.com> wrote:
> "hfc" seems to mean "hci fc" which may be "Host Control Interface Flow
> Control". 

Yes, it is. Just mention it in certain tense. 

> These are some parameters needed for RTL8851BU.
> 
> Signed-off-by: Bitterblue Smith <rtl8821cerfe2@gmail.com>

Acked-by: Ping-Ke Shih <pkshih@realtek.com>


> ---
> v2:
>  - Adapt to the changes in patch 3.
>  - Add more information in the commit message.
> ---
>  drivers/net/wireless/realtek/rtw89/mac.c      |  1 +
>  drivers/net/wireless/realtek/rtw89/rtw8851b.c | 46 ++++++++++++++++++-
>  2 files changed, 46 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/realtek/rtw89/mac.c b/drivers/net/wireless/realtek/rtw89/mac.c
> index 9ca88a293f99..7f3c816d4704 100644
> --- a/drivers/net/wireless/realtek/rtw89/mac.c
> +++ b/drivers/net/wireless/realtek/rtw89/mac.c
> @@ -881,6 +881,7 @@ static int hfc_reset_param(struct rtw89_dev *rtwdev)
> 
>         switch (rtwdev->hci.type) {
>         case RTW89_HCI_TYPE_PCIE:
> +       case RTW89_HCI_TYPE_USB:

Like I mention in patch 03/14, you will no need to add this. 

>                 break;
>         default:
>                 return -EINVAL;