On 14 February 2014 20:43, Mikko Hissa <mikko.hi...@werzek.com> wrote:
>
> On 14 Feb 2014, at 01:04, Roman Yeryomin <leroi.li...@gmail.com> wrote:
>
>> On 14 February 2014 00:34, Mikko Hissa <mikko.hi...@werzek.com> wrote:
>>> In rt2800lib.c function rt2800_get_txwi_rxwi_size()
>>> Add case MT7620 with RT5592.
>>>
>>>                *txwi_size = TXWI_DESC_SIZE_5WORDS;
>>>                *rxwi_size = RXWI_DESC_SIZE_6WORDS;
>>>                break;
>>>
>>
>> I suppose you mean RT5390 because:
>> ieee80211 phy0: rt2x00_set_rt: Info - RT chipset 5390, rev 0500 detected
>>
>> Also as far as I understand (from datasheet) it's not the same as
>> RT5592. Correct me if I'm wrong.
>> So I added:
>>
>>    case RT5390:
>>        *txwi_size = TXWI_DESC_SIZE_5WORDS;
>>        *rxwi_size = RXWI_DESC_SIZE_4WORDS;
>>        break;
>
> They should be 5 and 6 words not 5 and 4!
>
>                 *txwi_size = TXWI_DESC_SIZE_5WORDS;
>                 *rxwi_size = RXWI_DESC_SIZE_6WORDS;
>                 break;

You are right, it's 5 and 6 words - I misread the datasheet. Also
seems there is a typo first stating txwi is 4 words but describing 5
below.
Anyway I tried txwi 5 and rxwi 6 and results are the same.

> The original driver detects MT7620 by chip id 0x5390 and bus type. The driver 
> in rt2x00 for RT5390 is for a another chip. So you might not want to run the 
> bits of code meant for RT5390 but create some dummy definition for MT7620 and 
> set it in rt2800_probe_rt().
>

Ook.. I'll try to separate and clean that.


Regards,
Roman
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to