<div class="gmail_quote">2011/4/20 Thomas Gleixner <span dir="ltr"><<a href="mailto:tglx@linutronix.de">tglx@linutronix.de</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On Wed, 20 Apr 2011, Uwe Kleine-König wrote:<br>
> I declare this to be stable, so assuming people are OK with it, you can<br>
> use that as a base to convert your platforms.<br>
<br>
</div>You declare that stable? Interesting.<br>
<div class="im"><br>
> There is a 2nd branch on that repository that also contains my RFC patch<br>
> just sent to this list in case you want to test it.<br>
<br>
</div>Which is utter crap as I pointed out a few minutes ago.<br>
<br>
Also that clk thing is neither stable nor complete. It's just designed<br>
wrong.<br>
<br>
As long as it does not handle nested clocks proper and by default w/o<br>
your tasteless add ons, it's just moving the status quo of ARM into a<br>
common infrastructure file.<br>
<br>
Yes, that's probably better than not having common infrastructure at<br>
all, but trying to build up conversions on that lot would be a<br>
complete waste of time and resources. Simply because you need to<br>
convert the already converted stuff another time.<br></blockquote><div><br>Hey, guys,<br><br>Thomas's words might seem a bit harsh, but he really does understand what Linus's reaction will be. And Thomas's words are much less harsh than the ones that Linus would likely choose. I wish we had a good example of a common-clock structure elsewhere in the kernel, but the fact is that ARM has a much wider variety of SoCs than pretty much any other architecture. So we might as well get something put in place to start with that will have a decent chance of standing the test of time (and therefore being something that Linus is likely to approve of).<br>
<br>Could you please take a look at what Thomas suggested (<a href="http://lists.infradead.org/pipermail/linux-arm-kernel/2011-April/049013.html">http://lists.infradead.org/pipermail/linux-arm-kernel/2011-April/049013.html</a>) and give it a try?<br>
<br> Thanx, Paul<br></div></div>