Hi Simon,
On 8 July 2015 at 09:57, Peter Griffin <[email protected]> wrote: > > This patch adds the glue code for hi6220 SoC which has 2x synopsis > > dw_mmc controllers. This will be used by the hikey board support > > in subsequent patches. > > > > Signed-off-by: Peter Griffin <[email protected]> > > --- > > Reviewed-by: Simon Glass <[email protected]> > > (see nit below - this could be moved to device tree once you have it) > Thanks for reviewing :) My plan regarding DT was to wait for it to land in the kernel, and then convert over to using it in u-boot. I saw some of your patches converting Raspberry Pi to DM and DT which should make migrating HiKey board fairly straight forward, as it shares the same Synopsis IP for MMC and USB :) > > arch/arm/include/asm/arch-hi6220/dwmmc.h | 8 +++++ > > drivers/mmc/Makefile | 1 + > > drivers/mmc/hi6220_dw_mmc.c | 56 > ++++++++++++++++++++++++++++++++ > > 3 files changed, 65 insertions(+) > > create mode 100644 arch/arm/include/asm/arch-hi6220/dwmmc.h > > create mode 100644 drivers/mmc/hi6220_dw_mmc.c > > > > diff --git a/arch/arm/include/asm/arch-hi6220/dwmmc.h > b/arch/arm/include/asm/arch-hi6220/dwmmc.h > > new file mode 100644 > > index 0000000..c747383 > > --- /dev/null > > +++ b/arch/arm/include/asm/arch-hi6220/dwmmc.h > > @@ -0,0 +1,8 @@ > > +/* > > + * (C) Copyright 2015 Linaro > > + * Peter Griffin <[email protected]> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +int hi6220_dwmci_add_port(int index, u32 regbase, int bus_width); > > diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile > > index ed73687..81a1a8f 100644 > > --- a/drivers/mmc/Makefile > > +++ b/drivers/mmc/Makefile > > @@ -11,6 +11,7 @@ obj-$(CONFIG_BFIN_SDH) += bfin_sdh.o > > obj-$(CONFIG_DAVINCI_MMC) += davinci_mmc.o > > obj-$(CONFIG_DWMMC) += dw_mmc.o > > obj-$(CONFIG_EXYNOS_DWMMC) += exynos_dw_mmc.o > > +obj-$(CONFIG_HIKEY_DWMMC) += hi6220_dw_mmc.o > > obj-$(CONFIG_FSL_ESDHC) += fsl_esdhc.o > > obj-$(CONFIG_FTSDC010) += ftsdc010_mci.o > > obj-$(CONFIG_FTSDC021) += ftsdc021_sdhci.o > > diff --git a/drivers/mmc/hi6220_dw_mmc.c b/drivers/mmc/hi6220_dw_mmc.c > > new file mode 100644 > > index 0000000..106f673 > > --- /dev/null > > +++ b/drivers/mmc/hi6220_dw_mmc.c > > @@ -0,0 +1,56 @@ > > +/* > > + * (C) Copyright 2015 Linaro > > + * peter.griffin <[email protected]> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <dwmmc.h> > > +#include <malloc.h> > > +#include <asm-generic/errno.h> > > + > > +#define DWMMC_MAX_CH_NUM 4 > > + > > +#define DWMMC_MAX_FREQ 50000000 > > +#define DWMMC_MIN_FREQ 378000 > > + > > +/* Source clock is configured to 100Mhz by ATF bl1*/ > > nit: MHz > Fixed in v3. regards, Peter. _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
