Hi Dinh, On Sat, Sep 14 2013, Pavel Machek wrote: >> From: Dinh Nguyen <dinguyen@xxxxxxxxxx> >> >> Add functionality in the System Manager to set the SDR settings for the >> SD/MMC IP. >> >> Signed-off-by: Dinh Nguyen <dinguyen@xxxxxxxxxx> >> Cc: Pavel Machek <pavel@xxxxxxx> > >> +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 <cjb@xxxxxxxxxx> <http://printf.net/> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html