Re: [PATCH 1/3] Staging: rtl8187se: Move rtl8187se/ieee80211/* to rtl8187se/

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Aug 28, 2009 at 12:23:25PM -0400, Anders Kaseorg wrote:
> The current rtl8187se Makefile builds objects in the ieee80211
> subdirectory, which is a violation of the kbuild specification,
> according to Documentation/kbuild/makefiles.txt:

All of drivers/staging/ violates the kernel coding style "rules", so
this is not a real issue :)

I'd prefer to not accept this and the other move patch, as the code
should just be dropped entirely and the in-kernel ieee80211 code used
instead.  Keeping it in a subdirectory like it is makes it more obvious
as to what needs to be done here.

I will take your "remove unused files" patch, that is good to do.

Also note that there is a rtl8192e driver in the tree that also does
this same thing (have ieee80211 code in a subdir).

I'm trying to work with RealTek to keep this from happening in their
future drivers, but communication with them is slow.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux