Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: staging: rtl8723bs: questions on TODO list
To:     Fabio Aiuto <fabioaiuto83@gmail.com>, gregkh@linuxfoundation.org,
        Larry.Finger@lwfinger.net, youling 257 <youling257@gmail.com>
Cc:     linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
References: <20210514083856.GA13800@agape.jhs>
From:   Hans de Goede <hdegoede@redhat.com>
Message-ID: <85289dbc-e986-9beb-f7d6-eca9e8750c8a@redhat.com>
Date:   Fri, 14 May 2021 11:15:42 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.10.1
MIME-Version: 1.0
In-Reply-To: <20210514083856.GA13800@agape.jhs>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Fabio,

On 5/14/21 10:38 AM, Fabio Aiuto wrote:
> Hello all,
> 
> I'd like to have some clarifications about rtl8723bs driver.
> In order to make this driver ready for moving out of staging
> I would like to know:
> 
> - find and remove remaining code valid only for 5 GHz. Most of the obvious
>   ones have been removed, but things like channel > 14 still exist.
> 
> is it possible to remove all 5g code, even the one related to power
> regulation when on band 5g? As far as I know about this card is that
> it doesn't support 5g, so may I just delete all 5g code or there are some
> constraints I'd take care of?

These cards are 2.4G only so any code of which you are sure it is for
5G only can be removed.

> - find and remove any code for other chips that is left over
> 
> Ok this seems clear, are there some suggestion to do it safely?

Get some hardware and test that things still work, preferably
against multiple access-points.

> - convert any remaining unusual variable types
> 
> Ok (but feel free to suggest anything)
> 
> - find codes that can use %pM and %Nph formatting
> 
> Ok (but feel free to suggest anything)
> 
> - checkpatch.pl fixes - most of the remaining ones are lines too long. Many
>   of them will require refactoring
> 
> Ok
> 
> - merge Realtek's bugfixes and new features into the driver
> 
> Please, can you explain what one could do that?

I know that "youling 257 <youling257@gmail.com>" is using a different
version of the original (out-of-tree, non cleaned-up) Realtek driver
code for the Android-X86 kernels he works on. You should probably
ask youling why that version is being used and try to get a copy of
the original Realtek sources on which the staging driver is based
and do a "diff -ur" between them and see what is different.

> - switch to use LIB80211
> - switch to use MAC80211
> 
> I think I need a few details for these last points as well.

I'm afraid I cannot help there, you should ask for help with this
on the linux wireless email list.

> Do you think that one will need real hardware to complete
> these tasks? I don't have rtl8723bs card at the moment, so
> I think I will focus on those TODO activities which
> don't need it.

Yes you really do need hardware, even simple coding style
cleanups are known to (accidentally) break driver functionality,
so you really should ensure that you are capable of testing your
changes on real hardware.

Regards,

Hans