Hi! > 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? 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 devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html