On 12/12/14 19:53, Mark Brown wrote: > On Thu, Dec 11, 2014 at 06:45:49PM +0100, Sylwester Nawrocki wrote: >> > If the codec control it's master clock provider by the I2S module >> > we should not be touching it with set_sysclk() calls. >> > So skip the set_sysclk() call in the machine driver if "clocks" >> > property is found in the codec device node. > > Please clarify in the changelog here that this is a transition measure > to support old DTs, it's not clear why we might want to do things for > the I2S controller by fishing around in the CODEC DT otherwise. Indeed, there is some room for improvement. I'll add such note to the commit description. Presumably this whole driver could be removed after a kernel release or two, since we switched Odroid X2/U3 to the simple-card. -- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html