Alternative Libertas SDIO driver seen

[Tang, Feng]

Hi, Dan, Hlger,

Yes, moblin.org has some drivers for Marvell SDIO 868x devices. One reason for that is moblin.org need suppport some Marvell devices which are brand new and not covered by current Libertas driver.

Also Marvell driver on moblin.org is updated frequently as it keeps getting new developing versons from Marvell, which make it difficult to clean up the code to open-source format for each new version.

Thanks,
Feng
-----Original Message-----
From: Dan Williams [mailto:dcbw at redhat.com] 
Sent: 2008年4月14日 23:20
To: Holger Schurig
Cc: libertas-dev at lists.infradead.org; claud_yu; Harvey, Timothy; Tang, Feng; linux-wireless at vger.kernel.org
Subject: Re: Alternative Libertas SDIO driver seen

On Mon, 2008-04-14 at 12:11 +0200, Holger Schurig wrote:
> On Monday 14 April 2008 11:12:38 claud_yu wrote:
> > Hi ,
> >
> > Currently I use wi2wi W2SW0001, I send Murata module
> > back. I still try to port the libertas driver to mx31.
> > Wi2Wi provide a SD card development card on wi2wi
> > website. I also try another 88w8686 driver from
> > project moblin (http://www.moblin.org/). The driver
> > should based on marvell SDIO driver. But the driver
> > still have some problem while firmware download.
> 
> Okay, I find a kernel tree there which, in 
> debian/patches/0019-marvell_8686_8688.patch contains another 
> libertas driver.

Yeah, this is based on the V8/V9 Marvell vendor driver, which is a
continuation of the driver that got dropped on OLPC.  So you'll notice
quite a few similarities to early versions of the upstream libertas
driver.

I'd like to think we've improved on the vendor driver greatly, from a
cleanup standpoint (getting rid of the SBI abstraction stuff), from a
Wireless Extensions standpoint, from a readability/maintainability
standpoint, etc.

Unfortunately we never got any buy-in from Marvell themselves on
developing on the upstream driver together, though we did get some
patches from their engineers that were against upstream.

We can only really pull stuff from the moblin/vendor driver since if we
wanted to use that code, we'd have to re-do all the work we've done over
the past 2 years cleaning up the existing libertas driver but on the
moblin/vendor driver.  The vendor driver probably has better support for
things like WMM, power management, etc in it that are useful to mobile
stuff but haven't yet been a focus of the existing libertas driver since
it too so long just to get it to work reliably with a lot of different
hardware than just the 868x.

Dan

> Unfortunately, the (Intel created) moblin.org webpage only 
> supports the "http://" access protocol to the git tree, which is 
> rather inefficent. I'm currently trying to get this tree 
> with "git clone --reference /usr/src/linux-git 
> http://www.moblin.org/repos/projects/kernel-mid-2.6.24.git/", 
> and takes ages. Maybe they also made this tree from scratch, so 
> that the reference to Linus' git tree won't help the 
> downloading.
> 
> 
> 
> Okay, to the driver:
> 
> * it's GPL, so we can copy from it
> * it's an if_sdio.c only-driver (and also contains something
>    with bluetooth stuff)
> * it's weird, as this patch contains two wlan_11.c files
>   which differ. This is true for effectively every wlan_XXX.c
>   file.
> * it looks similar to what Marvell donated originally to OLPC
> * compared to the current in-kernel libertas driver, it has
>   some more code for 802.11a frequencies
> * older Marvell drivers contained firmware as *.h files, this
>   driver now uses #include <linux/firmware.h> as well
> 
> As this driver is sdio-only, and we have already other libertas 
> drivers supported in the kernel, it would never accepted as-is 
> in mainland kernel. Therefore I invite Marvell and Intel to 
> participate with the in-kernel libertas driver. I understand 
> that for Intel this might not be possible immediately, because 
> the current libertas driver might not work on their 
> Atom-reference platform, but I guess in the long-run could be 
> beneficial.
> 
> I CC'ed feng tang from "git log 
> debian/patches/0019-marvell_8686_8688.patch" therefore :-)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html