
1 Jun
2011
1 Jun
'11
9:27 p.m.
Dear Rob Herring,
In message 1306955427-18940-3-git-send-email-robherring2@gmail.com you wrote:
From: Rob Herring rob.herring@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@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@denx.de
"I'm growing older, but not up." - Jimmy Buffett