Hi Mark, On Wed, Mar 27, 2013 at 12:37:00PM +0000, Mark Brown wrote: > On Wed, Mar 27, 2013 at 12:57:00PM +0100, Philipp Zabel wrote: > > Am Mittwoch, den 27.03.2013, 12:22 +0100 schrieb Markus Pargmann: > > > imx-ssi needs admux_gate clock to work on imx35. This patch registers > > > this clock and enables it in the imx-ssi driver. > > > I have no idea what the admux_gate really is, but the ssi driver doesn't > > seem like the right place. Isn't this more closely related to the > > imx-audmux.c driver? > > This is a patch to the ASoC driver? I just ignored it since it said it > was an ARM patch... Anyway, what I'd expect should be happening here is > that the clock API walks up the tree enabling supply clocks - that would > mean that either that code isn't there or there's a missing parent child > relationship in the data. It changes some things in imx-ssi, sorry, perhaps I should have splitted it into two patches. admux clock does not seem to be a physical parent of the ssi clock, so I don't think it should be a parent in the clock tree. But imx-ssi (or some other evice) needs this clock to work. As mentioned in the other response, I will put it directly into clk-imx35.c so this won't be a patch for ASoC in the next version. Regards, Markus -- 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 stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html