| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20081001113458.GC6126@tesla>
Date: Wed, 1 Oct 2008 04:34:58 -0700
From: "Luis R. Rodriguez" <lrodriguez@...eros.com>
To: Pavel Machek <pavel@...e.cz>
CC: kernel list <linux-kernel@...r.kernel.org>,
Greg KH <greg@...ah.com>,
"kaszak@...il.com" <kaszak@...il.com>,
"lcostantino@...il.com" <lcostantino@...il.com>,
"linux-wireless@...r.kernel.org" <linux-wireless@...r.kernel.org>
Subject: Re: w35und wifi driver for linux-staging
On Wed, Oct 01, 2008 at 05:36:56AM -0700, Pavel Machek wrote:
> Hi!
>
> This is driver for w35und usb wifi -- also in kohjinsha
> subnotebook. It should work well enough to associate and ping, but it
> obviously needs to be rewritten two more times...
>
> OTOH worst horrors (like embedded wifi stack) should have been fixed
> already...
>
> (against 2.6.27-rc8).
>
> Signed-off-by: Pavel Machek <pavel@...e.cz>
Quick review just looking at only a few files.
This can be removed:
> diff --git a/drivers/net/wireless/winbond/winbondport/SVN_CHANGELOG b/drivers/net/wireless/winbond/winbondport/SVN_CHANGELOG
> diff --git a/drivers/net/wireless/winbond/winbondport/VersionNote.txt
> diff --git a/drivers/net/wireless/winbond/winbondport/VersionNote.txt b/drivers/net/wireless/winbond/winbondport/VersionNote.txt
> +++ b/drivers/net/wireless/winbond/winbondport/adapter.h
> @@ -0,0 +1,23 @@
> +//
> +// ADAPTER.H -
> +// Windows NDIS global variable 'Adapter' typedef
> +//
Better kdoc or /* use this style of comments */
> +#define MAX_ANSI_STRING 40
> +typedef struct WB32_ADAPTER
> +{
Typedefs need to be removed.
> +void CreateWpaIE(PWB32_ADAPTER Adapter, u16* iFildOffset, PUCHAR msg, struct Management_Frame* msgHeader,
> + struct Association_Request_Frame_Body* msgBody, u16 iMSindex); //added by WS 05/14/05
> +
Please run patch against checkpatch.pl.
> +#ifdef _WPA2_
> +void CreateRsnIE(PWB32_ADAPTER Adapter, u16* iFildOffset, PUCHAR msg, struct Management_Frame* msgHeader,
> + struct Association_Request_Frame_Body* msgBody, u16 iMSindex);//added by WS 05/14/05
> +
> +u16 SearchPmkid(PWB32_ADAPTER Adapter, struct Management_Frame* msgHeader,
> + struct PMKID_Information_Element * AssoReq_PMKID );
> +#endif
> +//=============================================================================
> +// IOCTLS defined for DUT (Device Under Test)
> +
> +// IOCTL_WB_802_11_DUT_MAC_ADDRESS
> +// Query: Return the dot11StationID
> +// Set : Set the dot11StationID. Demo only.
> +//
> +#define IOCTL_WB_802_11_DUT_MAC_ADDRESS CTL_CODE( \
> + FILE_DEVICE_UNKNOWN, \
> + WB_IOCTL_DUT_INDEX + 1, \
> + METHOD_BUFFERED, \
> + FILE_ANY_ACCESS)
> +
IOCTL stuff? What? This unused anyway.
> +// IOCTL_WB_802_11_DUT_BSS_DESCRIPTION
> +// Query: Return the info. of the current connected BSS.
> +// Set : None.
> +//
> +#define IOCTL_WB_802_11_DUT_BSS_DESCRIPTION CTL_CODE( \
> + FILE_DEVICE_UNKNOWN, \
> + WB_IOCTL_DUT_INDEX + 2, \
> + METHOD_BUFFERED, \
> + FILE_ANY_ACCESS)
> +
This too.
> +#define IOCTL_WB_802_11_DUT_TX_RATE_MODE CTL_CODE( \
> + FILE_DEVICE_UNKNOWN, \
> + WB_IOCTL_DUT_INDEX + 30, \
> + METHOD_BUFFERED, \
> + FILE_ANY_ACCESS)
And this guy.. etc...
> +++ b/drivers/net/wireless/winbond/winbondport/linux/Makefile
> @@ -0,0 +1,37 @@
> +#=========================================================================
> +#
> +# Module Name:
> +# w35und.ko Makefile for the ISSC wireless networking driver in linux kernel 2.6.x
> +# Port from Original
> +
> +# Abstract:
> +# Linux 2.6 driver for IS89C35D 802.11 a/b/g Wireless USB
> +#
> +#=========================================================================
> +
> +
> +# This is not used for build when the wifi module is build into kernel
> +#
> + KERNELDIR := /lib/modules/$(shell uname -r)/build
> + PWD := $(shell pwd)
> + WPA_DIR=../wpa
> +
> +obj-m := w35und.o
> +
> +w35und-objs := wbusb.o wb35reg.o wb35rx.o wb35tx.o\
> + ../bssdbase.o ../ds_tkip.o ../knl.o ../mds.o ../mlme_assoc.o ../mlme_auth.o ../mlmetask.o ../mlmetxrx.o \
> + ../mto.o ../phy_calibration.o ../reg.o ../roamtask.o ../rxisr.o ../scan_probe.o ../scantask.o \
> + ../sme_api.o ../smetask.o ../wbhal.o ../wblinux.o \
> + $(WPA_DIR)/eap.o $(WPA_DIR)/4Way.o $(WPA_DIR)/Adapter_Bridge.o $(WPA_DIR)/STLFunction.o \
> + $(WPA_DIR)/passphrase.o $(WPA_DIR)/prf.o \
> + $(WPA_DIR)/rc4.o $(WPA_DIR)/sha1.o $(WPA_DIR)/hmac_sha1.o \
> + $(WPA_DIR)/hmac_md5.o $(WPA_DIR)/md5.o $(WPA_DIR)/aes.o $(WPA_DIR)/aes_wrap.o
The kernel has most of these if not all.
> +++ b/drivers/net/wireless/winbond/winbondport/linux/Makefile.debug
Make this a CONFIG option instead of using another file.
> diff --git a/drivers/net/wireless/winbond/winbondport/linux/README.PORT b/drivers/net/wireless/winbond/winbondport/linux/README.PORT
Can be removed or moved to Documentation/networking/ somewhere if its really useful.
> +++ b/drivers/net/wireless/winbond/winbondport/linux/README.RXART
This can be removed.
> diff --git a/drivers/net/wireless/winbond/winbondport/linux/README.txt b/drivers/net/wireless/winbond/winbondport/linux/README.txt
Documentation/networking/ somewhere, but this documentation is not
relevant for the kernel.
> +0. To build driver
> + #cd linux/
> + #make
> +
> + ( For debug mode )
> + #cp Makefile.debug Makefile
> + #make
> +
> + #make install ( if want it on /lib/modules..)
> +
> + w35und.ko should be created
Like this.
> +
> +1. Installation
> +
> + # insmod w35und.ko or #modprobe w35und ( if "make install", ( check depmod too.. ) )
> +
> + Notes:
> + After your inserted this module to system, your can use special commands
> + of user guider to connect your desired AP or IBSS station. After connected,
> + you can use this command to assign an IP address and up it.
> +
> + # ifconfig wlan0 xxx.xxx.xxx.xxx. up
> +
> +
> +
> +2. Un-Installation
> +
> + # rmmod w35und
> +
> + Notes:
> + If you have up our driver, you must down your dirver with this command
> + before you remove this module:
> +
> + #ifconfig wlan0 down
> +
> +
> +
> +3. User Guide
> +
> + 3.0.1 <Set/Get region>
> +
> + # iwpriv wlan0 set_region XXX
> + # iwpriv wlan0 get_region
> +
> + Notes:
> + XXX is a string, includes : Europe, Japan, USA, Reg_Unknown
Please consider using cfg80211 and new regulatory framework for this.
Then you can refer to iw:
http://wireless.kernel.org/en/users/Documentation/iw
Regulatory stuff:
http://wireless.kernel.org/en/developers/Regulatory
Also please read Documentation/networking/regulatory.txt
> diff --git a/drivers/net/wireless/winbond/winbondport/linux/common.h b/drivers/net/wireless/winbond/winbondport/linux/common.h
Please keep files in its own directory.
> +unsigned char
> +Wb35Reg_Read(phw_data_t pHwData, u16 RegisterNo, PULONG pRegisterValue )
> +{
Please install sparse and then compile the driver using C=1, and correct
the warnings.
Ok well that should take a while..
Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists