Search Linux Wireless

Re: [PATCH 1/1] New driver: rtl8723au (mac80211)

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

 



On 09/07/2015 01:43 PM, Jes Sorensen wrote:
Kalle Valo <kvalo@xxxxxxxxxxxxxx> writes:
Hi,

replying to an old thread first.

Jes Sorensen <Jes.Sorensen@xxxxxxxxxx> writes:

Kalle Valo <kvalo@xxxxxxxxxxxxxx> writes:
Jes.Sorensen@xxxxxxxxxx writes:

  MAINTAINERS                          |    8 +
  drivers/net/wireless/Kconfig         |   19 +
  drivers/net/wireless/Makefile        |    2 +
  drivers/net/wireless/rtl8xxxu.c | 4500
++++++++++++++++++++++++++++++++++
  drivers/net/wireless/rtl8xxxu.h      |  497 ++++
  drivers/net/wireless/rtl8xxxu_regs.h |  941 +++++++

I think someone else already mentioned, but it would be better that has
it's own directory. Or should this actually be under rtlwifi
directory?

I didn't see the need here - it's just 3 files, as long as it doesn't
have a huge hierachy of files, a new directory doesn't add much
value. If it becomes an issue later, we can move it into a
subdirectory.

It's easier that the driver has it's on directory as everything
(makefile, kconfig etc) is cleanly separated. And actually I would like
to create a new vendor directory for realtek and have this in
drivers/net/wireless/realtek/rtl8xxxu/. And later we could move rtlwifi
under realtek directory as well.

If you want to create drivers/net/wireless/realtek, that is fine with
me, but maybe you can go ahead and create that first and move the
existing drivers in there?

I will submit a patch to do that.

Larry


--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux