Re: [PATCH] Samsung S3C24xx SD/MMC driver

From: Harald Welte
Date: Wed Dec 19 2007 - 13:00:23 EST


Hi Pierre,

thanks for your comments.

On Wed, Dec 19, 2007 at 05:54:33PM +0100, Pierre Ossman wrote:
> Well, my biggest beef with this driver is the excessive debug code. I
> did a cleanup in an older version of the driver, but it shouldn't be
> to difficult to do the same for this one. I've included my patch for
> reference.

ok, I'll re-work the patch according to your sample and re-submit.

The debugging was needed during development, and might still be needed
to debug some weird issues.

btw: OpenMoko is currently working on an ar6k wifi driver on top of the
new mainline kernel SDIO support using the s3c_mci host controller
driver as basis. So within not-too-distant-time we should also be able
to have SDIO working and tested on s3c_mci.

> > Index: linux-2.6/drivers/mmc/host/s3cmci.c
> > ===================================================================
> > --- /dev/null
> > +++ linux-2.6/drivers/mmc/host/s3cmci.c
>
> > +#include <linux/mmc/mmc.h>
>
> This is always a warning sign. It should never be needed in a host driver.

ok, will see how to remove it.

> > + *words = sg->length >> 2;
> > + *pointer = page_address(sg_page(sg)) + sg->offset;
> > +
>
> The length might not be a multiple of four. And it might also be
> completely unaligned. Make sure you can either handle such requests,
> or fail them with -EINVAL.

ok, will update before submitting the next version.
--
- Harald Welte <laforge@xxxxxxxxxxxx> http://openmoko.org/
============================================================================
Software for the world's first truly open Free Software mobile phone
--
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/