[U-Boot] fsl_esdhc.c bug
Graeme Russ
graeme.russ at gmail.com
Fri Dec 16 15:17:10 CET 2011
Hi Jim
On Dec 17, 2011 1:11 AM, "Jim Lentz" <JLentz at zhone.com> wrote:
>
> All,
>
> There is a bug rendering SDCard support useless on Freescale esdhc
controller.
>
> File drivers/mmc/fsl_esdhc.c, function esdhc_pio_read_write
>
> *change: *
>
> struct fsl_esdhc *regs = mmc->priv;
>
> *to:*
>
> struct fsl_esdhc_cfg *cfg = (struct fsl_esdhc_cfg *)mmc->priv;
> volatile struct fsl_esdhc *regs = (struct fsl_esdhc *)cfg->esdhc_base;
use of volatile is highly likely to be wrong
>
> How do I/we get this into the mainstream?
First read:
http://www.denx.de/wiki/U-Boot/CodingStyle
And:
http://www.denx.de/wiki/U-Boot/Patches
>
> I cloned the repository, and have it fixed. First time using git, and
trying to contribute to OpenSource too.
git format-patch and git send-email
Regards,
Graeme
More information about the U-Boot
mailing list