Hi!

> From: Dinh Nguyen <dingu...@altera.com>
> 
> Add functionality in the System Manager to set the SDR settings for the
> SD/MMC IP.
> 
> Signed-off-by: Dinh Nguyen <dingu...@altera.com>
> Cc: Pavel Machek <pa...@denx.de>

> +void socfpga_sysmgr_set_dwmmc_drvsel_smpsel(void)
> +{
> +     struct device_node *np;
> +     u32 timing[2];
> +     u32 hs_timing;
> +
> +     np = of_find_compatible_node(NULL, NULL, "altr,socfpga-dw-mshc");
> +     of_property_read_u32_array(np, "samsung,dw-mshc-sdr-timing", timing, 2);
> +     hs_timing = SYSMGR_SDMMC_CTRL_SET(timing[0], timing[1]);
> +     writel(hs_timing, sys_manager_base_addr + SYSMGR_SDMMCGRP_CTRL_OFFSET);
> +}
> +EXPORT_SYMBOL(socfpga_sysmgr_set_dwmmc_drvsel_smpsel);

To get the abstraction right, would it make sense to have timing
parameters as arguments to socfpga_sysmgr_set_dwmmc_drvsel_smpsel(),
so that sysmgr code is not walking MMC's device tree directly?

Thanks,
                                                                        Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to