| Message ID | 20230227121732.8967-3-kvalo@kernel.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 298e50ad8eb8fa12ea68bb2da45bb8ef4edcd0ec |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | wifi: create legacy and virtual directories | expand |
Hi Kalle,
I love your patch! Perhaps something to improve:
[auto build test WARNING on ec52d77d077529f198fd874c550a26b9cc86a331]
url: https://github.com/intel-lab-lkp/linux/commits/Kalle-Valo/wifi-move-mac80211_hwsim-and-virt_wifi-to-virtual-directory/20230227-201848
base: ec52d77d077529f198fd874c550a26b9cc86a331
patch link: https://lore.kernel.org/r/20230227121732.8967-3-kvalo%40kernel.org
patch subject: [PATCH 2/2] wifi: move raycs, wl3501 and rndis_wlan to legacy directory
config: sparc-allyesconfig (https://download.01.org/0day-ci/archive/20230228/202302280135.b1uG3bwe-lkp@intel.com/config)
compiler: sparc64-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/b3643ed46f437156c43b21bfc61dd622a6d53191
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Kalle-Valo/wifi-move-mac80211_hwsim-and-virt_wifi-to-virtual-directory/20230227-201848
git checkout b3643ed46f437156c43b21bfc61dd622a6d53191
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=sparc olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=sparc SHELL=/bin/bash drivers/net/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202302280135.b1uG3bwe-lkp@intel.com/
All warnings (new ones prefixed by >>):
In function 'init_startup_params',
inlined from 'ray_init' at drivers/net/wireless/legacy/ray_cs.c:506:2:
>> drivers/net/wireless/legacy/ray_cs.c:628:17: warning: 'strncpy' specified bound 32 equals destination size [-Wstringop-truncation]
628 | strncpy(local->sparm.b4.a_current_ess_id, essid, ESSID_SIZE);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +/strncpy +628 drivers/net/wireless/legacy/ray_cs.c
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 568
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 569 /*===========================================================================*/
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 570 static void init_startup_params(ray_dev_t *local)
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 571 {
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 572 int i;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 573
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 574 if (country > JAPAN_TEST)
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 575 country = USA;
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 576 else if (country < USA)
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 577 country = USA;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 578 /* structure for hop time and beacon period is defined here using
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 579 * New 802.11D6.1 format. Card firmware is still using old format
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 580 * until version 6.
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 581 * Before After
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 582 * a_hop_time ms byte a_hop_time ms byte
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 583 * a_hop_time 2s byte a_hop_time ls byte
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 584 * a_hop_time ls byte a_beacon_period ms byte
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 585 * a_beacon_period a_beacon_period ls byte
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 586 *
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 587 * a_hop_time = uS a_hop_time = KuS
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 588 * a_beacon_period = hops a_beacon_period = KuS
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 589 *//* 64ms = 010000 */
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 590 if (local->fw_ver == 0x55) {
e48d661eb13f2f8 drivers/net/wireless/ray_cs.c Kees Cook 2017-05-05 591 memcpy(&local->sparm.b4, b4_default_startup_parms,
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 592 sizeof(struct b4_startup_params));
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 593 /* Translate sane kus input values to old build 4/5 format */
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 594 /* i = hop time in uS truncated to 3 bytes */
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 595 i = (hop_dwell * 1024) & 0xffffff;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 596 local->sparm.b4.a_hop_time[0] = (i >> 16) & 0xff;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 597 local->sparm.b4.a_hop_time[1] = (i >> 8) & 0xff;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 598 local->sparm.b4.a_beacon_period[0] = 0;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 599 local->sparm.b4.a_beacon_period[1] =
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 600 ((beacon_period / hop_dwell) - 1) & 0xff;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 601 local->sparm.b4.a_curr_country_code = country;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 602 local->sparm.b4.a_hop_pattern_length =
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 603 hop_pattern_length[(int)country] - 1;
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 604 if (bc) {
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 605 local->sparm.b4.a_ack_timeout = 0x50;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 606 local->sparm.b4.a_sifs = 0x3f;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 607 }
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 608 } else { /* Version 5 uses real kus values */
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 609 memcpy((UCHAR *) &local->sparm.b5, b5_default_startup_parms,
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 610 sizeof(struct b5_startup_params));
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 611
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 612 local->sparm.b5.a_hop_time[0] = (hop_dwell >> 8) & 0xff;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 613 local->sparm.b5.a_hop_time[1] = hop_dwell & 0xff;
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 614 local->sparm.b5.a_beacon_period[0] =
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 615 (beacon_period >> 8) & 0xff;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 616 local->sparm.b5.a_beacon_period[1] = beacon_period & 0xff;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 617 if (psm)
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 618 local->sparm.b5.a_power_mgt_state = 1;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 619 local->sparm.b5.a_curr_country_code = country;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 620 local->sparm.b5.a_hop_pattern_length =
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 621 hop_pattern_length[(int)country];
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 622 }
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 623
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 624 local->sparm.b4.a_network_type = net_type & 0x01;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 625 local->sparm.b4.a_acting_as_ap_status = TYPE_STA;
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 626
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 627 if (essid != NULL)
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 @628 strncpy(local->sparm.b4.a_current_ess_id, essid, ESSID_SIZE);
^1da177e4c3f415 drivers/net/wireless/ray_cs.c Linus Torvalds 2005-04-16 629 } /* init_startup_params */
141fa61f10c419c drivers/net/wireless/ray_cs.c John Daiker 2009-03-10 630
On Mon, 2023-02-27 at 14:17 +0200, Kalle Valo wrote: > To clean up drivers/net/wireless move the old drivers drivers left in > the > directory to a new "legacy" directory. I did consider adding > CONFIG_WLAN_VENDOR_LEGACY like other vendors have but then dropped > the idea as > these are really old drivers and hopefully we get to remove them > soon. Why is rndis_wlan legacy? It supports devices that are way newer than ray_cs or wl3501... like this Linksys WUSB54GSC from late 2007: [1086339.589565] rndis_wlan 1-3:1.0 wlan0: register 'rndis_wlan' at usb-0000:00:14.0-3, Wireless RNDIS device, BCM4320b based, 00:1d:7e:9e:2f:bb [1086339.589961] usbcore: registered new interface driver rndis_wlan Dunno, just seems a completely different class of devices than old 802.11b-only PCMCIA ones... Dan > > There should be no changes in compilation or in Kconfig options, > merely moving files. > > Signed-off-by: Kalle Valo <kvalo@kernel.org> > --- > drivers/net/wireless/Kconfig | 55 +---------------- > -- > drivers/net/wireless/Makefile | 7 +-- > drivers/net/wireless/legacy/Kconfig | 55 > +++++++++++++++++++ > drivers/net/wireless/legacy/Makefile | 6 ++ > drivers/net/wireless/{ => legacy}/ray_cs.c | 0 > drivers/net/wireless/{ => legacy}/ray_cs.h | 0 > drivers/net/wireless/{ => legacy}/rayctl.h | 0 > .../net/wireless/{ => legacy}/rndis_wlan.c | 0 > drivers/net/wireless/{ => legacy}/wl3501.h | 0 > drivers/net/wireless/{ => legacy}/wl3501_cs.c | 0 > 10 files changed, 63 insertions(+), 60 deletions(-) > create mode 100644 drivers/net/wireless/legacy/Kconfig > create mode 100644 drivers/net/wireless/legacy/Makefile > rename drivers/net/wireless/{ => legacy}/ray_cs.c (100%) > rename drivers/net/wireless/{ => legacy}/ray_cs.h (100%) > rename drivers/net/wireless/{ => legacy}/rayctl.h (100%) > rename drivers/net/wireless/{ => legacy}/rndis_wlan.c (100%) > rename drivers/net/wireless/{ => legacy}/wl3501.h (100%) > rename drivers/net/wireless/{ => legacy}/wl3501_cs.c (100%) > > diff --git a/drivers/net/wireless/Kconfig > b/drivers/net/wireless/Kconfig > index 42b40cc96b21..7555af5195ec 100644 > --- a/drivers/net/wireless/Kconfig > +++ b/drivers/net/wireless/Kconfig > @@ -38,60 +38,7 @@ source "drivers/net/wireless/ti/Kconfig" > source "drivers/net/wireless/zydas/Kconfig" > source "drivers/net/wireless/quantenna/Kconfig" > > -config PCMCIA_RAYCS > - tristate "Aviator/Raytheon 2.4GHz wireless support" > - depends on PCMCIA > - select WIRELESS_EXT > - select WEXT_SPY > - select WEXT_PRIV > - help > - Say Y here if you intend to attach an Aviator/Raytheon > PCMCIA > - (PC-card) wireless Ethernet networking card to your > computer. > - Please read the file > - > <file:Documentation/networking/device_drivers/wifi/ray_cs.rst> for > - details. > - > - To compile this driver as a module, choose M here: the > module will be > - called ray_cs. If unsure, say N. > - > -config PCMCIA_WL3501 > - tristate "Planet WL3501 PCMCIA cards" > - depends on CFG80211 && PCMCIA > - select WIRELESS_EXT > - select WEXT_SPY > - help > - A driver for WL3501 PCMCIA 802.11 wireless cards made by > Planet. > - It has basic support for Linux wireless extensions and > initial > - micro support for ethtool. > - > -config USB_NET_RNDIS_WLAN > - tristate "Wireless RNDIS USB support" > - depends on USB > - depends on CFG80211 > - select USB_NET_DRIVERS > - select USB_USBNET > - select USB_NET_CDCETHER > - select USB_NET_RNDIS_HOST > - help > - This is a driver for wireless RNDIS devices. > - These are USB based adapters found in devices such as: > - > - Buffalo WLI-U2-KG125S > - U.S. Robotics USR5421 > - Belkin F5D7051 > - Linksys WUSB54GSv2 > - Linksys WUSB54GSC > - Asus WL169gE > - Eminent EM4045 > - BT Voyager 1055 > - Linksys WUSB54GSv1 > - U.S. Robotics USR5420 > - BUFFALO WLI-USB-G54 > - > - All of these devices are based on Broadcom 4320 chip which > is the > - only wireless RNDIS chip known to date. > - > - If you choose to build a module, it'll be called > rndis_wlan. > +source "drivers/net/wireless/legacy/Kconfig" > > source "drivers/net/wireless/virtual/Kconfig" > > diff --git a/drivers/net/wireless/Makefile > b/drivers/net/wireless/Makefile > index 1b697cfe0a13..4d7374d567d1 100644 > --- a/drivers/net/wireless/Makefile > +++ b/drivers/net/wireless/Makefile > @@ -23,10 +23,5 @@ obj-$(CONFIG_WLAN_VENDOR_ST) += st/ > obj-$(CONFIG_WLAN_VENDOR_TI) += ti/ > obj-$(CONFIG_WLAN_VENDOR_ZYDAS) += zydas/ > > -# 16-bit wireless PCMCIA client drivers > -obj-$(CONFIG_PCMCIA_RAYCS) += ray_cs.o > -obj-$(CONFIG_PCMCIA_WL3501) += wl3501_cs.o > - > -obj-$(CONFIG_USB_NET_RNDIS_WLAN) += rndis_wlan.o > - > +obj-$(CONFIG_WLAN) += legacy/ > obj-$(CONFIG_WLAN) += virtual/ > diff --git a/drivers/net/wireless/legacy/Kconfig > b/drivers/net/wireless/legacy/Kconfig > new file mode 100644 > index 000000000000..3a5275941212 > --- /dev/null > +++ b/drivers/net/wireless/legacy/Kconfig > @@ -0,0 +1,55 @@ > +config PCMCIA_RAYCS > + tristate "Aviator/Raytheon 2.4GHz wireless support" > + depends on PCMCIA > + select WIRELESS_EXT > + select WEXT_SPY > + select WEXT_PRIV > + help > + Say Y here if you intend to attach an Aviator/Raytheon > PCMCIA > + (PC-card) wireless Ethernet networking card to your > computer. > + Please read the file > + > <file:Documentation/networking/device_drivers/wifi/ray_cs.rst> for > + details. > + > + To compile this driver as a module, choose M here: the > module will be > + called ray_cs. If unsure, say N. > + > +config PCMCIA_WL3501 > + tristate "Planet WL3501 PCMCIA cards" > + depends on CFG80211 && PCMCIA > + select WIRELESS_EXT > + select WEXT_SPY > + help > + A driver for WL3501 PCMCIA 802.11 wireless cards made by > Planet. > + It has basic support for Linux wireless extensions and > initial > + micro support for ethtool. > + > +config USB_NET_RNDIS_WLAN > + tristate "Wireless RNDIS USB support" > + depends on USB > + depends on CFG80211 > + select USB_NET_DRIVERS > + select USB_USBNET > + select USB_NET_CDCETHER > + select USB_NET_RNDIS_HOST > + help > + This is a driver for wireless RNDIS devices. > + These are USB based adapters found in devices such as: > + > + Buffalo WLI-U2-KG125S > + U.S. Robotics USR5421 > + Belkin F5D7051 > + Linksys WUSB54GSv2 > + Linksys WUSB54GSC > + Asus WL169gE > + Eminent EM4045 > + BT Voyager 1055 > + Linksys WUSB54GSv1 > + U.S. Robotics USR5420 > + BUFFALO WLI-USB-G54 > + > + All of these devices are based on Broadcom 4320 chip which > is the > + only wireless RNDIS chip known to date. > + > + If you choose to build a module, it'll be called > rndis_wlan. > + > diff --git a/drivers/net/wireless/legacy/Makefile > b/drivers/net/wireless/legacy/Makefile > new file mode 100644 > index 000000000000..36878f080bfc > --- /dev/null > +++ b/drivers/net/wireless/legacy/Makefile > @@ -0,0 +1,6 @@ > +# 16-bit wireless PCMCIA client drivers > +obj-$(CONFIG_PCMCIA_RAYCS) += ray_cs.o > +obj-$(CONFIG_PCMCIA_WL3501) += wl3501_cs.o > + > +obj-$(CONFIG_USB_NET_RNDIS_WLAN) += rndis_wlan.o > + > diff --git a/drivers/net/wireless/ray_cs.c > b/drivers/net/wireless/legacy/ray_cs.c > similarity index 100% > rename from drivers/net/wireless/ray_cs.c > rename to drivers/net/wireless/legacy/ray_cs.c > diff --git a/drivers/net/wireless/ray_cs.h > b/drivers/net/wireless/legacy/ray_cs.h > similarity index 100% > rename from drivers/net/wireless/ray_cs.h > rename to drivers/net/wireless/legacy/ray_cs.h > diff --git a/drivers/net/wireless/rayctl.h > b/drivers/net/wireless/legacy/rayctl.h > similarity index 100% > rename from drivers/net/wireless/rayctl.h > rename to drivers/net/wireless/legacy/rayctl.h > diff --git a/drivers/net/wireless/rndis_wlan.c > b/drivers/net/wireless/legacy/rndis_wlan.c > similarity index 100% > rename from drivers/net/wireless/rndis_wlan.c > rename to drivers/net/wireless/legacy/rndis_wlan.c > diff --git a/drivers/net/wireless/wl3501.h > b/drivers/net/wireless/legacy/wl3501.h > similarity index 100% > rename from drivers/net/wireless/wl3501.h > rename to drivers/net/wireless/legacy/wl3501.h > diff --git a/drivers/net/wireless/wl3501_cs.c > b/drivers/net/wireless/legacy/wl3501_cs.c > similarity index 100% > rename from drivers/net/wireless/wl3501_cs.c > rename to drivers/net/wireless/legacy/wl3501_cs.c
Dan Williams <dcbw@redhat.com> writes: > On Mon, 2023-02-27 at 14:17 +0200, Kalle Valo wrote: >> To clean up drivers/net/wireless move the old drivers drivers left in >> the >> directory to a new "legacy" directory. I did consider adding >> CONFIG_WLAN_VENDOR_LEGACY like other vendors have but then dropped >> the idea as >> these are really old drivers and hopefully we get to remove them >> soon. > > Why is rndis_wlan legacy? It supports devices that are way newer than > ray_cs or wl3501... like this Linksys WUSB54GSC from late 2007: > > [1086339.589565] rndis_wlan 1-3:1.0 wlan0: register 'rndis_wlan' at > usb-0000:00:14.0-3, Wireless RNDIS device, BCM4320b based, > 00:1d:7e:9e:2f:bb > [1086339.589961] usbcore: registered new interface driver rndis_wlan So you have this device? Does it work? I think I should make a table somewhere for these old drivers with last success reports :) > Dunno, just seems a completely different class of devices than old > 802.11b-only PCMCIA ones... I was about to say that all drivers using Wireless Extensions are legacy, but to my surprise rndis_wlan actually uses cfg80211 :) I put this to "legacy" as I didn't find any better location and adding a new vendor driver just for rndis_wlan felt like overkill. The directory name "legacy" is just a name, it has no real meaning and users won't see it either. It could be "misc", "old" or something else as well.
kernel test robot <lkp@intel.com> writes: > Hi Kalle, > > I love your patch! Perhaps something to improve: > > [auto build test WARNING on ec52d77d077529f198fd874c550a26b9cc86a331] > > url: > https://github.com/intel-lab-lkp/linux/commits/Kalle-Valo/wifi-move-mac80211_hwsim-and-virt_wifi-to-virtual-directory/20230227-201848 > base: ec52d77d077529f198fd874c550a26b9cc86a331 > patch link: https://lore.kernel.org/r/20230227121732.8967-3-kvalo%40kernel.org > patch subject: [PATCH 2/2] wifi: move raycs, wl3501 and rndis_wlan to legacy directory > config: sparc-allyesconfig > (https://download.01.org/0day-ci/archive/20230228/202302280135.b1uG3bwe-lkp@intel.com/config) > compiler: sparc64-linux-gcc (GCC) 12.1.0 > reproduce (this is a W=1 build): > wget > https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross > -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # > https://github.com/intel-lab-lkp/linux/commit/b3643ed46f437156c43b21bfc61dd622a6d53191 > git remote add linux-review https://github.com/intel-lab-lkp/linux > git fetch --no-tags linux-review > Kalle-Valo/wifi-move-mac80211_hwsim-and-virt_wifi-to-virtual-directory/20230227-201848 > git checkout b3643ed46f437156c43b21bfc61dd622a6d53191 > # save the config file > mkdir build_dir && cp config build_dir/.config > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 > make.cross W=1 O=build_dir ARCH=sparc olddefconfig > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 > make.cross W=1 O=build_dir ARCH=sparc SHELL=/bin/bash drivers/net/ > > If you fix the issue, kindly add following tag where applicable > | Reported-by: kernel test robot <lkp@intel.com> > | Link: https://lore.kernel.org/oe-kbuild-all/202302280135.b1uG3bwe-lkp@intel.com/ > > All warnings (new ones prefixed by >>): > > In function 'init_startup_params', > inlined from 'ray_init' at drivers/net/wireless/legacy/ray_cs.c:506:2: >>> drivers/net/wireless/legacy/ray_cs.c:628:17: warning: 'strncpy' specified bound 32 equals destination size [-Wstringop-truncation] > 628 | strncpy(local->sparm.b4.a_current_ess_id, essid, ESSID_SIZE); > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ I didn't make any changes in the code, just moved files around, so this is an old issue.
On Mon, 2023-02-27 at 21:11 +0200, Kalle Valo wrote: > Dan Williams <dcbw@redhat.com> writes: > > > On Mon, 2023-02-27 at 14:17 +0200, Kalle Valo wrote: > > > To clean up drivers/net/wireless move the old drivers drivers > > > left in > > > the > > > directory to a new "legacy" directory. I did consider adding > > > CONFIG_WLAN_VENDOR_LEGACY like other vendors have but then > > > dropped > > > the idea as > > > these are really old drivers and hopefully we get to remove them > > > soon. > > > > Why is rndis_wlan legacy? It supports devices that are way newer > > than > > ray_cs or wl3501... like this Linksys WUSB54GSC from late 2007: > > > > [1086339.589565] rndis_wlan 1-3:1.0 wlan0: register 'rndis_wlan' at > > usb-0000:00:14.0-3, Wireless RNDIS device, BCM4320b based, > > 00:1d:7e:9e:2f:bb > > [1086339.589961] usbcore: registered new interface driver > > rndis_wlan > > So you have this device? Does it work? I think I should make a table > somewhere for these old drivers with last success reports :) Yep, I have it, it works. Needless to say, I don't *use* it. > > > Dunno, just seems a completely different class of devices than old > > 802.11b-only PCMCIA ones... > > I was about to say that all drivers using Wireless Extensions are > legacy, but to my surprise rndis_wlan actually uses cfg80211 :) > > I put this to "legacy" as I didn't find any better location and > adding a > new vendor driver just for rndis_wlan felt like overkill. The > directory > name "legacy" is just a name, it has no real meaning and users won't > see > it either. It could be "misc", "old" or something else as well. Is the goal just to get all the .c individual drivers out of net/wireless? Also isn't Greg KH out to kill RNDIS too? I don't recall that being a settled question yet, but I lost track. Dan
Dan Williams <dcbw@redhat.com> writes: > On Mon, 2023-02-27 at 21:11 +0200, Kalle Valo wrote: >> Dan Williams <dcbw@redhat.com> writes: >> >> > On Mon, 2023-02-27 at 14:17 +0200, Kalle Valo wrote: >> > > To clean up drivers/net/wireless move the old drivers drivers >> > > left in >> > > the >> > > directory to a new "legacy" directory. I did consider adding >> > > CONFIG_WLAN_VENDOR_LEGACY like other vendors have but then >> > > dropped >> > > the idea as >> > > these are really old drivers and hopefully we get to remove them >> > > soon. >> > >> > Why is rndis_wlan legacy? It supports devices that are way newer >> > than >> > ray_cs or wl3501... like this Linksys WUSB54GSC from late 2007: >> > >> > [1086339.589565] rndis_wlan 1-3:1.0 wlan0: register 'rndis_wlan' at >> > usb-0000:00:14.0-3, Wireless RNDIS device, BCM4320b based, >> > 00:1d:7e:9e:2f:bb >> > [1086339.589961] usbcore: registered new interface driver >> > rndis_wlan >> >> So you have this device? Does it work? I think I should make a table >> somewhere for these old drivers with last success reports :) > > Yep, I have it, it works. Needless to say, I don't *use* it. Yeah, I guessed that part :) But thanks for testing, good to know it works. >> > Dunno, just seems a completely different class of devices than old >> > 802.11b-only PCMCIA ones... >> >> I was about to say that all drivers using Wireless Extensions are >> legacy, but to my surprise rndis_wlan actually uses cfg80211 :) >> >> I put this to "legacy" as I didn't find any better location and >> adding a >> new vendor driver just for rndis_wlan felt like overkill. The >> directory >> name "legacy" is just a name, it has no real meaning and users won't >> see >> it either. It could be "misc", "old" or something else as well. > > Is the goal just to get all the .c individual drivers out of > net/wireless? Yes, exactly. The extra files in drivers/net/wireless annoy me everytime when I'm checking something in the tree. > Also isn't Greg KH out to kill RNDIS too? I don't recall that being a > settled question yet, but I lost track. I haven't heard anything about that recently, I hope we don't have to remove rndis_wlan from the tree. But wext drivers are another thing, we really should get rid of them (or convert to cfg80211).
On Tue, 2023-02-28 at 08:29 +0200, Kalle Valo wrote: > Dan Williams <dcbw@redhat.com> writes: > > > On Mon, 2023-02-27 at 21:11 +0200, Kalle Valo wrote: > > > Dan Williams <dcbw@redhat.com> writes: > > > > > > > On Mon, 2023-02-27 at 14:17 +0200, Kalle Valo wrote: > > > > > To clean up drivers/net/wireless move the old drivers drivers > > > > > left in > > > > > the > > > > > directory to a new "legacy" directory. I did consider adding > > > > > CONFIG_WLAN_VENDOR_LEGACY like other vendors have but then > > > > > dropped > > > > > the idea as > > > > > these are really old drivers and hopefully we get to remove > > > > > them > > > > > soon. > > > > > > > > Why is rndis_wlan legacy? It supports devices that are way > > > > newer > > > > than > > > > ray_cs or wl3501... like this Linksys WUSB54GSC from late 2007: > > > > > > > > [1086339.589565] rndis_wlan 1-3:1.0 wlan0: register > > > > 'rndis_wlan' at > > > > usb-0000:00:14.0-3, Wireless RNDIS device, BCM4320b based, > > > > 00:1d:7e:9e:2f:bb > > > > [1086339.589961] usbcore: registered new interface driver > > > > rndis_wlan > > > > > > So you have this device? Does it work? I think I should make a > > > table > > > somewhere for these old drivers with last success reports :) > > > > Yep, I have it, it works. Needless to say, I don't *use* it. > > Yeah, I guessed that part :) But thanks for testing, good to know it > works. > > > > > Dunno, just seems a completely different class of devices than > > > > old > > > > 802.11b-only PCMCIA ones... > > > > > > I was about to say that all drivers using Wireless Extensions are > > > legacy, but to my surprise rndis_wlan actually uses cfg80211 :) > > > > > > I put this to "legacy" as I didn't find any better location and > > > adding a > > > new vendor driver just for rndis_wlan felt like overkill. The > > > directory > > > name "legacy" is just a name, it has no real meaning and users > > > won't > > > see > > > it either. It could be "misc", "old" or something else as well. > > > > Is the goal just to get all the .c individual drivers out of > > net/wireless? > > Yes, exactly. The extra files in drivers/net/wireless annoy me > everytime > when I'm checking something in the tree. ray_cs and wl3501_cs are also PCMCIA drivers (not CardBus) and isn't Arnd trying to get rid of PCMCIA via "[RFC 0/6] pcmcia: separate 16-bit support from cardbus"? Maybe those two drivers get solved for you :) > > Also isn't Greg KH out to kill RNDIS too? I don't recall that being > > a > > settled question yet, but I lost track. > > I haven't heard anything about that recently, I hope we don't have to > remove rndis_wlan from the tree. But wext drivers are another thing, > we > really should get rid of them (or convert to cfg80211). I have vague, morbid interest in converting atmel and prism54 to cfg80211 but timeline on that would be "this year". Dan
Dan Williams <dcbw@redhat.com> writes: >> > Is the goal just to get all the .c individual drivers out of >> > net/wireless? >> >> Yes, exactly. The extra files in drivers/net/wireless annoy me >> everytime >> when I'm checking something in the tree. > > ray_cs and wl3501_cs are also PCMCIA drivers (not CardBus) and isn't > Arnd trying to get rid of PCMCIA via "[RFC 0/6] pcmcia: separate 16-bit > support from cardbus"? > > Maybe those two drivers get solved for you :) That would be great :) >> > Also isn't Greg KH out to kill RNDIS too? I don't recall that being >> > a >> > settled question yet, but I lost track. >> >> I haven't heard anything about that recently, I hope we don't have to >> remove rndis_wlan from the tree. But wext drivers are another thing, >> we >> really should get rid of them (or convert to cfg80211). > > I have vague, morbid interest in converting atmel and prism54 to > cfg80211 but timeline on that would be "this year". As you already noticed, prism54 is already gone. But I'm keeping fingers crossed that who you would have time to convert atmel :) We really should get rid of wext, at least from drivers/net/wireless. Staging drivers are of course of another thing. Just for fun I decided to grep how many wext drivers and the first match was an ethernet driver, weird: drivers/net/ethernet/toshiba/ps3_gelic_wireless.c:2570: netdev->wireless_handlers = &gelic_wl_wext_handler_def; These are the mainline drivers using wext: drivers/net/wireless/atmel/atmel.c:1574: dev->wireless_handlers = &atmel_handler_def; drivers/net/wireless/cisco/airo.c:2674: dev->wireless_handlers = &airo_handler_def; drivers/net/wireless/cisco/airo.c:2828: dev->wireless_handlers = &airo_handler_def; drivers/net/wireless/intel/ipw2x00/ipw2100.c:6032: dev->wireless_handlers = &ipw2100_wx_handler_def; drivers/net/wireless/intel/ipw2x00/ipw2200.c:11675: net_dev->wireless_handlers = &ipw_wx_handler_def; drivers/net/wireless/intersil/hostap/hostap_main.c:851: dev->wireless_handlers = &hostap_iw_handler_def; drivers/net/wireless/intersil/orinoco/main.c:2251: dev->wireless_handlers = &orinoco_handler_def; drivers/net/wireless/ray_cs.c:303: dev->wireless_handlers = &ray_handler_def; drivers/net/wireless/wl3501_cs.c:1886: dev->wireless_handlers = &wl3501_handler_def; drivers/net/wireless/zydas/zd1201.c:1782: dev->wireless_handlers = &zd1201_iw_handlers; And these are the staging drivers: drivers/staging/ks7010/ks_wlan_net.c:2636: dev->wireless_handlers = &ks_wlan_handler_def; drivers/staging/r8188eu/os_dep/os_intfs.c:358: pnetdev->wireless_handlers = (struct iw_handler_def *)&rtw_handlers_def; drivers/staging/rtl8192e/rtl8192e/rtl_core.c:2334: dev->wireless_handlers = &r8192_wx_handlers_def; drivers/staging/rtl8192u/r8192U_core.c:4532: dev->wireless_handlers = &r8192_wx_handlers_def; drivers/staging/rtl8712/os_intfs.c:214: pnetdev->wireless_handlers = (struct iw_handler_def *)
On Wed, 2023-03-01 at 09:09 +0200, Kalle Valo wrote: > Dan Williams <dcbw@redhat.com> writes: > > > > > Is the goal just to get all the .c individual drivers out of > > > > net/wireless? > > > > > > Yes, exactly. The extra files in drivers/net/wireless annoy me > > > everytime > > > when I'm checking something in the tree. > > > > ray_cs and wl3501_cs are also PCMCIA drivers (not CardBus) and isn't > > Arnd trying to get rid of PCMCIA via "[RFC 0/6] pcmcia: separate 16-bit > > support from cardbus"? > > > > Maybe those two drivers get solved for you :) > > That would be great :) > > > > > Also isn't Greg KH out to kill RNDIS too? I don't recall that being > > > > a > > > > settled question yet, but I lost track. > > > > > > I haven't heard anything about that recently, I hope we don't have to > > > remove rndis_wlan from the tree. But wext drivers are another thing, > > > we > > > really should get rid of them (or convert to cfg80211). > > > > I have vague, morbid interest in converting atmel and prism54 to > > cfg80211 but timeline on that would be "this year". > > As you already noticed, prism54 is already gone. But I'm keeping fingers > crossed that who you would have time to convert atmel :) > > We really should get rid of wext, at least from drivers/net/wireless. > Staging drivers are of course of another thing. Just for fun I decided > to grep how many wext drivers and the first match was an ethernet > driver, weird: > > drivers/net/ethernet/toshiba/ps3_gelic_wireless.c:2570: netdev->wireless_handlers = &gelic_wl_wext_handler_def; Not super odd; that's the PlayStation 3. HW is an odd combo of ethernet and WiFi and the FW interface is bizarre as you can see. At least it supports 802.11g and WPA which is more than we can say for ipw2100. > > These are the mainline drivers using wext: > > drivers/net/wireless/atmel/atmel.c:1574: dev->wireless_handlers = &atmel_handler_def; > drivers/net/wireless/cisco/airo.c:2674: dev->wireless_handlers = &airo_handler_def; > drivers/net/wireless/cisco/airo.c:2828: dev->wireless_handlers = &airo_handler_def; > drivers/net/wireless/intel/ipw2x00/ipw2100.c:6032: dev->wireless_handlers = &ipw2100_wx_handler_def; > drivers/net/wireless/intel/ipw2x00/ipw2200.c:11675: net_dev->wireless_handlers = &ipw_wx_handler_def; > drivers/net/wireless/intersil/hostap/hostap_main.c:851: dev->wireless_handlers = &hostap_iw_handler_def; > drivers/net/wireless/intersil/orinoco/main.c:2251: dev->wireless_handlers = &orinoco_handler_def; > drivers/net/wireless/ray_cs.c:303: dev->wireless_handlers = &ray_handler_def; > drivers/net/wireless/wl3501_cs.c:1886: dev->wireless_handlers = &wl3501_handler_def; > drivers/net/wireless/zydas/zd1201.c:1782: dev->wireless_handlers = &zd1201_iw_handlers; FWIW orinoco and ipw2x00 have some minimal cfg80211 code but IIRC it's pretty limited. Honestly we probably should start deprecation/removal for 802.11b-only hardware which is everything on this list *except* ps3_gelic and ipw2200. Dan > > And these are the staging drivers: > > drivers/staging/ks7010/ks_wlan_net.c:2636: dev->wireless_handlers = &ks_wlan_handler_def; > drivers/staging/r8188eu/os_dep/os_intfs.c:358: pnetdev->wireless_handlers = (struct iw_handler_def *)&rtw_handlers_def; > drivers/staging/rtl8192e/rtl8192e/rtl_core.c:2334: dev->wireless_handlers = &r8192_wx_handlers_def; > drivers/staging/rtl8192u/r8192U_core.c:4532: dev->wireless_handlers = &r8192_wx_handlers_def; > drivers/staging/rtl8712/os_intfs.c:214: pnetdev->wireless_handlers = (struct iw_handler_def *) >
diff --git a/drivers/net/wireless/Kconfig b/drivers/net/wireless/Kconfig index 42b40cc96b21..7555af5195ec 100644 --- a/drivers/net/wireless/Kconfig +++ b/drivers/net/wireless/Kconfig @@ -38,60 +38,7 @@ source "drivers/net/wireless/ti/Kconfig" source "drivers/net/wireless/zydas/Kconfig" source "drivers/net/wireless/quantenna/Kconfig" -config PCMCIA_RAYCS - tristate "Aviator/Raytheon 2.4GHz wireless support" - depends on PCMCIA - select WIRELESS_EXT - select WEXT_SPY - select WEXT_PRIV - help - Say Y here if you intend to attach an Aviator/Raytheon PCMCIA - (PC-card) wireless Ethernet networking card to your computer. - Please read the file - <file:Documentation/networking/device_drivers/wifi/ray_cs.rst> for - details. - - To compile this driver as a module, choose M here: the module will be - called ray_cs. If unsure, say N. - -config PCMCIA_WL3501 - tristate "Planet WL3501 PCMCIA cards" - depends on CFG80211 && PCMCIA - select WIRELESS_EXT - select WEXT_SPY - help - A driver for WL3501 PCMCIA 802.11 wireless cards made by Planet. - It has basic support for Linux wireless extensions and initial - micro support for ethtool. - -config USB_NET_RNDIS_WLAN - tristate "Wireless RNDIS USB support" - depends on USB - depends on CFG80211 - select USB_NET_DRIVERS - select USB_USBNET - select USB_NET_CDCETHER - select USB_NET_RNDIS_HOST - help - This is a driver for wireless RNDIS devices. - These are USB based adapters found in devices such as: - - Buffalo WLI-U2-KG125S - U.S. Robotics USR5421 - Belkin F5D7051 - Linksys WUSB54GSv2 - Linksys WUSB54GSC - Asus WL169gE - Eminent EM4045 - BT Voyager 1055 - Linksys WUSB54GSv1 - U.S. Robotics USR5420 - BUFFALO WLI-USB-G54 - - All of these devices are based on Broadcom 4320 chip which is the - only wireless RNDIS chip known to date. - - If you choose to build a module, it'll be called rndis_wlan. +source "drivers/net/wireless/legacy/Kconfig" source "drivers/net/wireless/virtual/Kconfig" diff --git a/drivers/net/wireless/Makefile b/drivers/net/wireless/Makefile index 1b697cfe0a13..4d7374d567d1 100644 --- a/drivers/net/wireless/Makefile +++ b/drivers/net/wireless/Makefile @@ -23,10 +23,5 @@ obj-$(CONFIG_WLAN_VENDOR_ST) += st/ obj-$(CONFIG_WLAN_VENDOR_TI) += ti/ obj-$(CONFIG_WLAN_VENDOR_ZYDAS) += zydas/ -# 16-bit wireless PCMCIA client drivers -obj-$(CONFIG_PCMCIA_RAYCS) += ray_cs.o -obj-$(CONFIG_PCMCIA_WL3501) += wl3501_cs.o - -obj-$(CONFIG_USB_NET_RNDIS_WLAN) += rndis_wlan.o - +obj-$(CONFIG_WLAN) += legacy/ obj-$(CONFIG_WLAN) += virtual/ diff --git a/drivers/net/wireless/legacy/Kconfig b/drivers/net/wireless/legacy/Kconfig new file mode 100644 index 000000000000..3a5275941212 --- /dev/null +++ b/drivers/net/wireless/legacy/Kconfig @@ -0,0 +1,55 @@ +config PCMCIA_RAYCS + tristate "Aviator/Raytheon 2.4GHz wireless support" + depends on PCMCIA + select WIRELESS_EXT + select WEXT_SPY + select WEXT_PRIV + help + Say Y here if you intend to attach an Aviator/Raytheon PCMCIA + (PC-card) wireless Ethernet networking card to your computer. + Please read the file + <file:Documentation/networking/device_drivers/wifi/ray_cs.rst> for + details. + + To compile this driver as a module, choose M here: the module will be + called ray_cs. If unsure, say N. + +config PCMCIA_WL3501 + tristate "Planet WL3501 PCMCIA cards" + depends on CFG80211 && PCMCIA + select WIRELESS_EXT + select WEXT_SPY + help + A driver for WL3501 PCMCIA 802.11 wireless cards made by Planet. + It has basic support for Linux wireless extensions and initial + micro support for ethtool. + +config USB_NET_RNDIS_WLAN + tristate "Wireless RNDIS USB support" + depends on USB + depends on CFG80211 + select USB_NET_DRIVERS + select USB_USBNET + select USB_NET_CDCETHER + select USB_NET_RNDIS_HOST + help + This is a driver for wireless RNDIS devices. + These are USB based adapters found in devices such as: + + Buffalo WLI-U2-KG125S + U.S. Robotics USR5421 + Belkin F5D7051 + Linksys WUSB54GSv2 + Linksys WUSB54GSC + Asus WL169gE + Eminent EM4045 + BT Voyager 1055 + Linksys WUSB54GSv1 + U.S. Robotics USR5420 + BUFFALO WLI-USB-G54 + + All of these devices are based on Broadcom 4320 chip which is the + only wireless RNDIS chip known to date. + + If you choose to build a module, it'll be called rndis_wlan. + diff --git a/drivers/net/wireless/legacy/Makefile b/drivers/net/wireless/legacy/Makefile new file mode 100644 index 000000000000..36878f080bfc --- /dev/null +++ b/drivers/net/wireless/legacy/Makefile @@ -0,0 +1,6 @@ +# 16-bit wireless PCMCIA client drivers +obj-$(CONFIG_PCMCIA_RAYCS) += ray_cs.o +obj-$(CONFIG_PCMCIA_WL3501) += wl3501_cs.o + +obj-$(CONFIG_USB_NET_RNDIS_WLAN) += rndis_wlan.o + diff --git a/drivers/net/wireless/ray_cs.c b/drivers/net/wireless/legacy/ray_cs.c similarity index 100% rename from drivers/net/wireless/ray_cs.c rename to drivers/net/wireless/legacy/ray_cs.c diff --git a/drivers/net/wireless/ray_cs.h b/drivers/net/wireless/legacy/ray_cs.h similarity index 100% rename from drivers/net/wireless/ray_cs.h rename to drivers/net/wireless/legacy/ray_cs.h diff --git a/drivers/net/wireless/rayctl.h b/drivers/net/wireless/legacy/rayctl.h similarity index 100% rename from drivers/net/wireless/rayctl.h rename to drivers/net/wireless/legacy/rayctl.h diff --git a/drivers/net/wireless/rndis_wlan.c b/drivers/net/wireless/legacy/rndis_wlan.c similarity index 100% rename from drivers/net/wireless/rndis_wlan.c rename to drivers/net/wireless/legacy/rndis_wlan.c diff --git a/drivers/net/wireless/wl3501.h b/drivers/net/wireless/legacy/wl3501.h similarity index 100% rename from drivers/net/wireless/wl3501.h rename to drivers/net/wireless/legacy/wl3501.h diff --git a/drivers/net/wireless/wl3501_cs.c b/drivers/net/wireless/legacy/wl3501_cs.c similarity index 100% rename from drivers/net/wireless/wl3501_cs.c rename to drivers/net/wireless/legacy/wl3501_cs.c
To clean up drivers/net/wireless move the old drivers drivers left in the directory to a new "legacy" directory. I did consider adding CONFIG_WLAN_VENDOR_LEGACY like other vendors have but then dropped the idea as these are really old drivers and hopefully we get to remove them soon. There should be no changes in compilation or in Kconfig options, merely moving files. Signed-off-by: Kalle Valo <kvalo@kernel.org> --- drivers/net/wireless/Kconfig | 55 +------------------ drivers/net/wireless/Makefile | 7 +-- drivers/net/wireless/legacy/Kconfig | 55 +++++++++++++++++++ drivers/net/wireless/legacy/Makefile | 6 ++ drivers/net/wireless/{ => legacy}/ray_cs.c | 0 drivers/net/wireless/{ => legacy}/ray_cs.h | 0 drivers/net/wireless/{ => legacy}/rayctl.h | 0 .../net/wireless/{ => legacy}/rndis_wlan.c | 0 drivers/net/wireless/{ => legacy}/wl3501.h | 0 drivers/net/wireless/{ => legacy}/wl3501_cs.c | 0 10 files changed, 63 insertions(+), 60 deletions(-) create mode 100644 drivers/net/wireless/legacy/Kconfig create mode 100644 drivers/net/wireless/legacy/Makefile rename drivers/net/wireless/{ => legacy}/ray_cs.c (100%) rename drivers/net/wireless/{ => legacy}/ray_cs.h (100%) rename drivers/net/wireless/{ => legacy}/rayctl.h (100%) rename drivers/net/wireless/{ => legacy}/rndis_wlan.c (100%) rename drivers/net/wireless/{ => legacy}/wl3501.h (100%) rename drivers/net/wireless/{ => legacy}/wl3501_cs.c (100%)