diff mbox series

rtw88: 8821c: Add RFE 2 support

Message ID 20200805084559.30092-1-kai.heng.feng@canonical.com
State New
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

Kai-Heng Feng Jan. 7, 2021, 6:38 a.m. UTC | #1
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 | #2
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/

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
Larry Finger Feb. 2, 2021, 7:02 a.m. UTC | #3
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 | #4
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[] = {