Hi Dinh, On Sat, Sep 14 2013, Pavel Machek wrote: >> 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?
I think this review comment from Pavel is still open, please reply. Thanks, - Chris. -- Chris Ball <c...@laptop.org> <http://printf.net/> -- 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