Re: [PATCH] Staging: remove rtl8187se driver because of lack of developmentactivities

[Larry Finger]

On 11/09/2009 10:56 AM, Mauro Carvalho Chehab wrote:
> Hi Bartolomiej,
> 
> Em Mon, 9 Nov 2009 17:31:36 +0100
> Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx> escreveu:
> 
>> From: Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx>
>> Subject: [PATCH] Staging: remove rtl8187se driver because of lack of development activities
>>
>> This driver hasn't seen any real development activities since early July:
>>
>>    257bc86   Staging: rtl8187se: remove unused #include <linu
>>    2035beb   param:remove-useless-writable-charp
>>    305409a   tree-wide: fix some typos and punctuation in com
>>    b881c6c   Staging: fix wireless drivers depends
>>    a010a33   Staging: rtl8187se/rtl8192e/rtl8192su: allow mod
>>    125b181   staging: Add proper selection of WIRELESS_EXT an
>>    b3184e1   Staging: rtl8187se: Fix comment-out-typo
>>    b6ac161   Staging: ARRAY_SIZE changes
>>
>> so remove it for now.
>>
>> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx>
> 
> This driver has several points in common with rtl8192* drivers. As I'm
> currently working on cleaning up rtl8192u and see what can be done to merge the
> common code used by all those realtek drivers, I prefer to keep this driver for
> more time at the tree.
> 
> The issue I'm currently facing is that we should rearrange the drivers/staging/rtl* 
> into a common rtl component tree, moving the common components to there. This will
> avoid duplicated efforts to cleanup the same code several times, on different
> places.
> 
> I started working on it this weekend, but I haven't finish it yet.

As I just told Bart (off-list), I started my work on a mainline rtl8187se driver
with the rtl8180 code. There should be plenty of opportunity to set up common
code for the rtl* drivers. I will keep a close eye on your developments.

Larry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/