Hi! On Wed, Oct 16, 2013 at 07:32:37PM -0500, dinguyen@xxxxxxxxxx wrote: > From: Dinh Nguyen <dinguyen@xxxxxxxxxx> > > Hi, > > This patch series enables support for the Synopsys SD/MMC driver that is > on the Altera SOCFPGA platform. The reason why this series has 4 patches > is to implement Arnd's suggestion: > > http://lists.infradead.org/pipermail/linux-arm-kernel/2013-October/204982.html > > To summarize Arnd's suggestion: > > 1. Create a backend syscon driver to control the system manager. > 2. Create a clock driver independent of the SOCFPGA clock driver that > uses syscon as the low-level interface. > 3. Make the sdmmc driver use the normal clock API and link its clock to > the driver step 2 in the device tree. > > The end approach is a bit different because I did not find the need for a > syscon driver for the system manager. Since the system manager had already > been iomap already in the SOCFPGA platform code, I just reused it in the > new clock driver. > What about the usbgrp, pinmuxgrp, nandgrp, etc. ? Those are all registers for different subsystems, that are located/controlled by the systemmgr. With Arnd's approach they all could call that low-level interface, instead of having a special case for every IP core. Isn't that the whole idea behind syscon? Regards, Steffen -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- 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