PATCH: Config option to pad SDIO sizes

Hans J. Koch hjk at linutronix.de
Mon Jun 30 06:52:51 EDT 2008


On Fri, Jun 27, 2008 at 11:37:53AM -0400, Dan Williams wrote:
> On Fri, 2008-06-27 at 15:22 +0200, Johan Adolfsson wrote:
> > Hi,
> > Attached is a patch agains 2.6.25 that adds a CONFIG_LIBERTAS_SDIO_PAD_SIZE
> > option to control padding of sdio packet sizes to work around buggy SDIO 
> > controllers - they are known to exist.. and not always possible to fix in
> > the
> > SDIO driver it seems.
> > 
> > Perhaps the patch could be trimmed a little - any sane compiler would
> > make it a noop if the PAD_SIZE is 0.
> 
> Pending a look from Pierre, but I think this would probably _also_ be
> better as a module parameter rather than a CONFIG option.  Since the
> addition is done anyway, there shouldn't be a real speed hit.  The
> problem is that in the general world, we have no idea what hardware
> you'll be running this on, and it's better to have one kernel that can
> run on lots of hardware rather than having to flip config options and
> recompile.  I don't think a module parameter would be an issue in
> embedded-land either, right?

In embedded-land you sometimes wish to build a kernel with everything
compiled in, without support for module loading.

hjk




More information about the libertas-dev mailing list