diff mbox series

rtw88: 8821c: Add RFE 2 support

Message ID 20200805084559.30092-1-kai.heng.feng@canonical.com (mailing list archive)
State Rejected
Delegated to: Kalle Valo
Headers show
Series rtw88: 8821c: Add RFE 2 support | expand

Commit Message

Kai-Heng Feng Aug. 5, 2020, 8:45 a.m. UTC
8821CE with RFE 2 isn't supported:
[   12.404834] rtw_8821ce 0000:02:00.0: rfe 2 isn't supported
[   12.404937] rtw_8821ce 0000:02:00.0: failed to setup chip efuse info
[   12.404939] rtw_8821ce 0000:02:00.0: failed to setup chip information

It works well if both type0 tables are in use, so add it to the RFE
default.

Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
---
 drivers/net/wireless/realtek/rtw88/rtw8821c.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Tony Chuang Aug. 5, 2020, 11:18 a.m. UTC | #1
> 8821CE with RFE 2 isn't supported:
> [   12.404834] rtw_8821ce 0000:02:00.0: rfe 2 isn't supported
> [   12.404937] rtw_8821ce 0000:02:00.0: failed to setup chip efuse info
> [   12.404939] rtw_8821ce 0000:02:00.0: failed to setup chip information
> 

NACK

The RFE type 2 should be working with some additional fixes.
Did you tested connecting to AP with BT paired?

The antenna configuration is different with RFE type 0.
I will ask someone else to fix them.
Then the RFE type 2 modules can be supported.

Yen-Hsuan
Kai-Heng Feng Aug. 5, 2020, 11:24 a.m. UTC | #2
Hi Tony,

> On Aug 5, 2020, at 19:18, Tony Chuang <yhchuang@realtek.com> wrote:
> 
>> 8821CE with RFE 2 isn't supported:
>> [   12.404834] rtw_8821ce 0000:02:00.0: rfe 2 isn't supported
>> [   12.404937] rtw_8821ce 0000:02:00.0: failed to setup chip efuse info
>> [   12.404939] rtw_8821ce 0000:02:00.0: failed to setup chip information
>> 
> 
> NACK
> 
> The RFE type 2 should be working with some additional fixes.
> Did you tested connecting to AP with BT paired?

No, I only tested WiFi.

> The antenna configuration is different with RFE type 0.
> I will ask someone else to fix them.
> Then the RFE type 2 modules can be supported.

Good to know that, I'll be patient and wait for a real fix.

Kai-Heng

> 
> Yen-Hsuan
Kai-Heng Feng Jan. 7, 2021, 6:38 a.m. UTC | #3
On Wed, Aug 5, 2020 at 7:24 PM Kai-Heng Feng
<kai.heng.feng@canonical.com> wrote:
>
> Hi Tony,
>
> > On Aug 5, 2020, at 19:18, Tony Chuang <yhchuang@realtek.com> wrote:
> >
> >> 8821CE with RFE 2 isn't supported:
> >> [   12.404834] rtw_8821ce 0000:02:00.0: rfe 2 isn't supported
> >> [   12.404937] rtw_8821ce 0000:02:00.0: failed to setup chip efuse info
> >> [   12.404939] rtw_8821ce 0000:02:00.0: failed to setup chip information
> >>
> >
> > NACK
> >
> > The RFE type 2 should be working with some additional fixes.
> > Did you tested connecting to AP with BT paired?
>
> No, I only tested WiFi.
>
> > The antenna configuration is different with RFE type 0.
> > I will ask someone else to fix them.
> > Then the RFE type 2 modules can be supported.
>
> Good to know that, I'll be patient and wait for a real fix.

It's been quite some time, is support for RFE type 2 ready now?

Kai-Heng

>
> Kai-Heng
>
> >
> > Yen-Hsuan
>
Kalle Valo Feb. 2, 2021, 6:29 a.m. UTC | #4
Kai-Heng Feng <kai.heng.feng@canonical.com> writes:

> On Wed, Aug 5, 2020 at 7:24 PM Kai-Heng Feng
> <kai.heng.feng@canonical.com> wrote:
>>
>> Hi Tony,
>>
>> > On Aug 5, 2020, at 19:18, Tony Chuang <yhchuang@realtek.com> wrote:
>> >
>> >> 8821CE with RFE 2 isn't supported:
>> >> [   12.404834] rtw_8821ce 0000:02:00.0: rfe 2 isn't supported
>> >> [   12.404937] rtw_8821ce 0000:02:00.0: failed to setup chip efuse info
>> >> [   12.404939] rtw_8821ce 0000:02:00.0: failed to setup chip information
>> >>
>> >
>> > NACK
>> >
>> > The RFE type 2 should be working with some additional fixes.
>> > Did you tested connecting to AP with BT paired?
>>
>> No, I only tested WiFi.
>>
>> > The antenna configuration is different with RFE type 0.
>> > I will ask someone else to fix them.
>> > Then the RFE type 2 modules can be supported.
>>
>> Good to know that, I'll be patient and wait for a real fix.
>
> It's been quite some time, is support for RFE type 2 ready now?

It looks like this patch should add it:

https://patchwork.kernel.org/project/linux-wireless/patch/20210202055012.8296-4-pkshih@realtek.com/
Larry Finger Feb. 2, 2021, 7:02 a.m. UTC | #5
On 2/2/21 12:29 AM, Kalle Valo wrote:
> Kai-Heng Feng <kai.heng.feng@canonical.com> writes:
> 
>> On Wed, Aug 5, 2020 at 7:24 PM Kai-Heng Feng
>> <kai.heng.feng@canonical.com> wrote:
>>>
>>> Hi Tony,
>>>
>>>> On Aug 5, 2020, at 19:18, Tony Chuang <yhchuang@realtek.com> wrote:
>>>>
>>>>> 8821CE with RFE 2 isn't supported:
>>>>> [   12.404834] rtw_8821ce 0000:02:00.0: rfe 2 isn't supported
>>>>> [   12.404937] rtw_8821ce 0000:02:00.0: failed to setup chip efuse info
>>>>> [   12.404939] rtw_8821ce 0000:02:00.0: failed to setup chip information
>>>>>
>>>>
>>>> NACK
>>>>
>>>> The RFE type 2 should be working with some additional fixes.
>>>> Did you tested connecting to AP with BT paired?
>>>
>>> No, I only tested WiFi.
>>>
>>>> The antenna configuration is different with RFE type 0.
>>>> I will ask someone else to fix them.
>>>> Then the RFE type 2 modules can be supported.
>>>
>>> Good to know that, I'll be patient and wait for a real fix.
>>
>> It's been quite some time, is support for RFE type 2 ready now?
> 
> It looks like this patch should add it:
> 
> https://patchwork.kernel.org/project/linux-wireless/patch/20210202055012.8296-4-pkshih@realtek.com/
> 
New firmware (rtw8821c_fw.bin) is also needed. That is available at 
https://github.com/lwfinger/rtw88.git.

Larry
Kai-Heng Feng Feb. 2, 2021, 7:36 a.m. UTC | #6
On Tue, Feb 2, 2021 at 3:02 PM Larry Finger <Larry.Finger@lwfinger.net> wrote:
>
> On 2/2/21 12:29 AM, Kalle Valo wrote:
> > Kai-Heng Feng <kai.heng.feng@canonical.com> writes:
> >
> >> On Wed, Aug 5, 2020 at 7:24 PM Kai-Heng Feng
> >> <kai.heng.feng@canonical.com> wrote:
> >>>
> >>> Hi Tony,
> >>>
> >>>> On Aug 5, 2020, at 19:18, Tony Chuang <yhchuang@realtek.com> wrote:
> >>>>
> >>>>> 8821CE with RFE 2 isn't supported:
> >>>>> [   12.404834] rtw_8821ce 0000:02:00.0: rfe 2 isn't supported
> >>>>> [   12.404937] rtw_8821ce 0000:02:00.0: failed to setup chip efuse info
> >>>>> [   12.404939] rtw_8821ce 0000:02:00.0: failed to setup chip information
> >>>>>
> >>>>
> >>>> NACK
> >>>>
> >>>> The RFE type 2 should be working with some additional fixes.
> >>>> Did you tested connecting to AP with BT paired?
> >>>
> >>> No, I only tested WiFi.
> >>>
> >>>> The antenna configuration is different with RFE type 0.
> >>>> I will ask someone else to fix them.
> >>>> Then the RFE type 2 modules can be supported.
> >>>
> >>> Good to know that, I'll be patient and wait for a real fix.
> >>
> >> It's been quite some time, is support for RFE type 2 ready now?
> >
> > It looks like this patch should add it:
> >
> > https://patchwork.kernel.org/project/linux-wireless/patch/20210202055012.8296-4-pkshih@realtek.com/
> >
> New firmware (rtw8821c_fw.bin) is also needed. That is available at
> https://github.com/lwfinger/rtw88.git.

Thanks. RFE2 works with the new firmware.

Kai-Heng

>
> Larry
>
diff mbox series

Patch

diff --git a/drivers/net/wireless/realtek/rtw88/rtw8821c.c b/drivers/net/wireless/realtek/rtw88/rtw8821c.c
index d8863d8a5468..f7270d0f1d55 100644
--- a/drivers/net/wireless/realtek/rtw88/rtw8821c.c
+++ b/drivers/net/wireless/realtek/rtw88/rtw8821c.c
@@ -1410,6 +1410,7 @@  static const struct rtw_intf_phy_para_table phy_para_table_8821c = {
 
 static const struct rtw_rfe_def rtw8821c_rfe_defs[] = {
 	[0] = RTW_DEF_RFE(8821c, 0, 0),
+	[2] = RTW_DEF_RFE(8821c, 0, 0),
 };
 
 static struct rtw_hw_reg rtw8821c_dig[] = {