[U-Boot] [PATCH 2/5] mmc: sdhci: rework Samsung specfic code

Wolfgang Denk wd at denx.de
Wed Jun 1 21:27:38 CEST 2011


Dear Rob Herring,

In message <1306955427-18940-3-git-send-email-robherring2 at gmail.com> you wrote:
> From: Rob Herring <rob.herring at calxeda.com>
> 
> Move the register definitions into the sdhci.c file. Set the base
> address from the board init code.
> 
> The Samsung SDHCI controller has extra registers. Make them conditional
> on CONFIG_MMC_S5P.
> 
> Signed-off-by: Rob Herring <rob.herring at calxeda.com>

Please run your patches through checkpatch before submittint.  This
should have shown you this:

...
> -		err = s5p_mmc_init(2, 4);
> +		err = sdhci_mmc_init((void *)(samsung_get_base_mmc() + 0x20000), 4);

Line too long.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"I'm growing older, but not up."                      - Jimmy Buffett


More information about the U-Boot mailing list