From: Olivier Blin To: Forest Bond Cc: Gábor Stefanik , Greg KH , Larry Finger , "John W. Linville" , linux-wireless@vger.kernel.org, Dan Williams Bcc: Subject: Re: [PATCH] Add vt6656 driver to drivers/staging. Reply-To: In-Reply-To: <20090601165207.GA12481@alittletooquiet.net> X-Spam-Status: No, score=-4.6 required=4.8 tests=AWL,BAYES_00 autolearn=ham version=3.2.5 Forest Bond writes: > Hi, > > On Mon, Jun 01, 2009 at 06:35:35PM +0200, Gábor Stefanik wrote: >> On Tue, Apr 14, 2009 at 12:52 PM, Forest Bond >> wrote: >> > This patch adds the vt6656 driver to drivers/staging.  This code was literally >> > copied from the upstream source archive, available here: >> > >> >  http://www.viaarena.com/Driver/VT6656_Linux_src_v1.19_12_x86.zip >> > >> > Copyright: VIA Networking Technologies, Inc. >> > License: GPL >> > Authors: Jerry Chen, Kyle Hsu, Lucas Lin, Lyndon Chen, Tevin Chen, Warren Hsu, >> >         Yiching Chen >> > Signed-off-by: Forest Bond >> > >> > Patch: http://www.alittletooquiet.net/media/code/0002-Add-vt6656-driver-to-drivers-staging.patch.gz >> > Signature: http://www.alittletooquiet.net/media/code/0002-Add-vt6656-driver-to-drivers-staging.patch.gz.sig >> >> Did this patch (along with the vt6655 one) somehow get lost? I can't >> see it in drivers/staging. > > vt6655 is in drivers/staging on linux-next, although I have three more patches > that I'll be sending to Greg sometime tonight. He has been applying them for > me. > > vt6656 is currently delayed a bit. I may be able to get a start on that > tonight, as well, but there is likelihood of further delay. By the way, are you sure about the hook below about wpactl.c in commit 8f2c32b86b3ac16ba4c93bf0c2b766089644d9d1 ? If I do the same in vt6656, it is freezing the kernel, or causing random crashes. Removing the netdev priv assignation makes the driver ok. I don't think the wpa priv was really useful, but I fail to see where it was used, and why it caused the hang. Don't you have the same issue with vt6655? BTW, I have some patches to send for vt6656, what is the proper place to submit them since vt6656 is not in staging for now? diff --git a/drivers/staging/vt6655/wpactl.c b/drivers/staging/vt6655/wpactl.c index ee7109d..cc4f0ad 100644 (file) --- a/drivers/staging/vt6655/wpactl.c +++ b/drivers/staging/vt6655/wpactl.c @@ -112,14 +112,17 @@ static void wpadev_setup(struct net_device *dev) static int wpa_init_wpadev(PSDevice pDevice) { + PSDevice wpadev_priv; struct net_device *dev = pDevice->dev; int ret=0; - pDevice->wpadev = alloc_netdev(0, "vntwpa", wpadev_setup); + pDevice->wpadev = alloc_netdev(sizeof(PSDevice), "vntwpa", wpadev_setup); if (pDevice->wpadev == NULL) return -ENOMEM; - pDevice->wpadev->priv = pDevice; + wpadev_priv = netdev_priv(pDevice->wpadev); + *wpadev_priv = *pDevice; + memcpy(pDevice->wpadev->dev_addr, dev->dev_addr, U_ETHER_ADDR_LEN); pDevice->wpadev->base_addr = dev->base_addr; pDevice->wpadev->irq = dev->irq; -- Olivier Blin (blino) - Mandriva