Hi,
On 25-07-14 20:19, Stefan Roese wrote:
On 25.07.2014 19:07, Gerald Kerma wrote:
This patch add Marvell kirkwood MVSDIO/MMC driver
and enable it for Sheevaplugs and OpenRD boards.
Signed-off-by: Gerald Kerma <[email protected]>
---
Changes in v6:
- rename driver to mvebu_mmc
- renames constants and functions
- move mmc_init to kirkwood/cpu
- code cleaning
<snip>
+
+#define mvebu_mmc_write(offs, val) writel(val, CONFIG_SYS_MMC_BASE +
(offs))
+#define mvebu_mmc_read(offs) readl(CONFIG_SYS_MMC_BASE + (offs))
Wouldn't it be better to use functions instead of macros here?
or use structs?
Regards,
Jeroen
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot