Re: [PATCH] staging: r8188eu: add extra TODO entries

[Pavel Skripkin]

On 8/28/21 3:02 AM, Phillip Potter wrote:
> Add entry for CFG80211 to reflect that it is currently being worked on
> and is a desirable feature to have for this driver. Leave entries for
> LIB80211 and MAC80211 in case anyone else wishes to work on them - they
> can always be removed later, but MAC80211 in particular would certainly
> be good to aim for. Also, include an entry for improving error handling,
> as this is another goal of the cleanup effort.
>
> Signed-off-by: Phillip Potter <phil@xxxxxxxxxxxxxxxx>
> ---
>   drivers/staging/r8188eu/TODO | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/drivers/staging/r8188eu/TODO b/drivers/staging/r8188eu/TODO
> index 98f918480990..ab9d5d145b3b 100644
> --- a/drivers/staging/r8188eu/TODO
> +++ b/drivers/staging/r8188eu/TODO
> @@ -11,3 +11,6 @@ To-do list:
>     the driver.
>   * Switch to use LIB80211.
>   * Switch to use MAC80211.
> +* Switch to use CFG80211.
> +* Improve the error handling of various functions, particularly those that use
> +  existing kernel APIs.


Cool, I like it. There is one more extra task, but I will take care of 
it. I hope. We need to delete dead proc code and move to sysfs. I have a 
plan to finish this part until next weekends, but maybe it worth adding 
to TODO.

Thank you



With regards,
Pavel Skripkin