Tomasz Figa wrote: > > Hi, > > On Sunday 03 of March 2013 12:17:29 Sylwester Nawrocki wrote: > > Hi, > > > > On 03/03/2013 02:08 AM, Heiko Stübner wrote: > > > But is there an easy way to define more than one alias? On the s3c2416 > > > for example the hsmmc hclk is the "hsmmc" io-clock, as well as the > > > source for the "mmc_busclk.0". Same for the "uart" pclk, that is also > > > a baud clock source. > > This driver currently provides for only one additional clkdev lookup > > entry per a platform clock. I pointed out this desing issue in the > > early version of the patch set. It's because a machine clock definition > > is coupled with a clock consumer definition. And IMO various > > samsung_clock_register_* functions should not have > > clk_register_clkdev() inside them. I.e. first step could be registering > > all machine clocks and in the second one clkdev lookup entries could be > > created. This is how most (all?) existing SoC clock drivers are > > working. > > > > But those multiple aliases are important only for machines with device > > tree support, aren't they ? > > I suppose you meant _without_ device tree support, right? > > > I hope this patch series gets merged early to linux-next in the 3.10 > > cycle so the multiple accumulated fixup patches for this clock driver > > can be merged as well and issues like that you pointed out can be > > resolved with incremental patches. > > Yes, I hope so too. > Yes, I will in this weekend. Note, I will modify [07/16] for universal_c210 as per Kyungmin's request, I'm not sure about universal_c210 though. Thanks. - Kukjin -- 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
 |