DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org AB331206B2
Subject: Re: [PATCH 01/12] rtwlan: main files
To:     Stanislaw Gruszka <sgruszka@redhat.com>,
        Kalle Valo <kvalo@codeaurora.org>
Cc:     Tony Chuang <yhchuang@realtek.com>,
        "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
        Pkshih <pkshih@realtek.com>, Andy Huang <tehuang@realtek.com>
References: <1537509847-21087-2-git-send-email-yhchuang@realtek.com>
 <20180927135040.GA4712@redhat.com>
 <F7CD281DE3E379468C6D07993EA72F84D08D9F56@RTITMBSVM01.realtek.com.tw>
 <20180928092918.GC8323@redhat.com>
 <F7CD281DE3E379468C6D07993EA72F84D08DB7E3@RTITMBSVM01.realtek.com.tw>
 <20181002102957.GB25116@redhat.com>
 <a54dec4e-1164-f9c5-c791-3396a239eeba@lwfinger.net>
 <F7CD281DE3E379468C6D07993EA72F84D1728978@RTITMBSVM01.realtek.com.tw>
 <4b50e06f-2fd0-9378-d283-1ca6d02f9586@lwfinger.net>
 <87k1mx28dg.fsf@codeaurora.org> <20181004134252.GA20484@redhat.com>
From:   Larry Finger <Larry.Finger@lwfinger.net>
Message-ID: <2fa15cec-bcef-35af-43af-5365d52d82c8@lwfinger.net>
Date:   Thu, 4 Oct 2018 11:19:59 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.0
MIME-Version: 1.0
In-Reply-To: <20181004134252.GA20484@redhat.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

On 10/4/18 8:42 AM, Stanislaw Gruszka wrote:
> On Thu, Oct 04, 2018 at 03:39:55PM +0300, Kalle Valo wrote:
>>>> Can we put the configuration file in the firmware directory?
>>>> Should we package them into binary files? Or just put the raw data.
>>>>
>>>> We can test the performance for it. After we got the result, we will make a decision
>>>> about it. And if we decide to put them in the firmware directory, will send a patch.
>>>> For now, I think we can just leave them in the .c.
>>>
>>> Yes, you could put the configuration files in the firmware directory.
>>> I would put them in binary form, not as text files. That way the size
>>> would be smaller, and it would not be possible to alter them,
>>> particularly if the binary file is checksummed.
>>>
>>> It would likely be OK if only the agc table was stored in this way.
>>> That would take away about half of the lines in the 8822b table file.
>>
>> So what's the worry here? The lines of source code, binary size or what?
>>
>>   .../net/wireless/realtek/rtw88/rtw8822b_table.c    | 20783 +++++++++++++++++++
>>
>> Looking at the diffstat rtw8822b_table.c seems to be 20 kLOC, IMHO it's
>> not that bad as it's just data. But of course I might be missing
>> something as I haven't checked patches yet.
> 
> My concern was it's plenty of redundant data, for example:
> 
>          0x81C, 0xFF000003,
>          0x81C, 0xFE000003,
>          0x81C, 0xFD020003,
>          0x81C, 0xFC040003,
>          0x81C, 0xFB060003,
>          0x81C, 0xFA080003,
>          0x81C, 0xF90A0003,
>          0x81C, 0xF80C0003,
>          0x81C, 0xF70E0003,
>          0x81C, 0xF6100003,
> 
> Approx 10000 lines like this, braked by lines like this
> 
>          0x90000012,     0x00000000,     0x40000000,     0x00000000,
> 
> in more or less regular way.
> 
> Not big deal, but perhaps this could be coded in much more compact way.

What should be the tradeoff between large tables of redundant data and 
complicated generation and interpretation? I think this table should be 
converted to binary in its present form and added to the "firmware", the way 
that is done for b43. That way the source is smaller, and the loading will be 
only a bit more time consuming.

Larry