Re: MMC Driver RFC

[Richard Purdie]

Russell King:
> > 2. Card Initialisation Problems
> >
> > One of my cards works fine. The other works when I enable debug and 
> > doesn't when I don't. I suspect the delay while it does a printk gives 
> > something time to happen that doesn't normally when running at full 
> > speed!
>
> Different cards behave differently.  I suspect you have yet another
> quirky card.

For reference, I got the 512MB SD card working by adding an mdelay(3) into 
the middle of mmc_send_op_cond(). Anything shorter and it marks the card as 
bad...

http://www.rpsys.net/openzaurus/2.6.11-rc1/mmc_sd-r2.patch shows the updated 
SD code that works for me on the Sharp SL-C760 with the mdelay included.

Pierre Ossman:
> The patch can be found at:
> http://projects.drzeus.cx/wbsd/sd.php

Now I've got both my cards working, I plan to test this code and compare 
soon. The added features look good.

> That page also contains the legal issues as I've understood them.

I don't see the problems you mention there. Yes companies may have signed 
agreements with the SD card association and yes, that may possibly mean they 
can't distribute Linux with SD support but that would be *their* problem and 
not a reason against including SD support in the kernel. If they remove any 
SD code from Linux then they would still be able to distribute it. I suspect 
that they would be able to distribute SD code already in the public domain 
anyway.

On the subject of patents, the whole idea behind SD is that there aren't 
patents as for a patent to exist, we'd have some publicly available 
information on how SD works. We're not breaking any copyrights as I nobody 
involved with this code has see any code to copy from.

So in short, I can't see any reason we can't put the code we have into the 
kernel...

Richard 

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