<div dir="ltr">I don't believe there are any bindings changes or a need to revert.  Stephen's suggested changes are small improvements that make the driver suitable for non-DT platforms (not an issue anytime soon) and help with a planned refactor of the clk_hw structure.  I'll send out follow-up changes when Stephen and I have reached consensus on how to resolve the latter improvement.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 8, 2017 at 9:15 AM, Jonathan Cameron <span dir="ltr"><<a href="mailto:jic23@kernel.org" target="_blank">jic23@kernel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 05/04/17 22:50, Stephen Boyd wrote:<br>
> On 04/01, Jonathan Cameron wrote:<br>
>> On 28/03/17 22:52, Rick Altherr wrote:<br>
>>> Aspeed BMC SoCs include a 16 channel, 10-bit ADC. Low and high threshold<br>
>>> interrupts are supported by the hardware but are not currently implemented.<br>
>>><br>
>>> Signed-off-by: Rick Altherr <<a href="mailto:raltherr@google.com">raltherr@google.com</a>><br>
>> Two really trivial things inline. I'll fix them whilst applying rather than<br>
>> having you do a v6 - please do sanity check I haven't messed it up though!<br>
>><br>
>> Applied to the togreg branch of iio.git and pushed out as testing for<br>
>> the autobuilders to play with it.<br>
>><br>
><br>
> Oh I was too late. Blame work. Anyway, I made some comments on<br>
> v4. If they're fixed in a later patch or discussed on list that's<br>
> fine. No worries on my end.<br>
><br>
</span>Oops. I went a bit quick on this one I guess.<br>
<br>
Anyhow, from a quick read of your comments I'm not sure if any of<br>
them are such that we should revert.  Perhaps we Rick can prepare<br>
a follow up patch covering them?  I got a little lost so am not<br>
sure but if there is anything effecting bindings Rick, please<br>
get this through asap or let me know if we need to revert for now.<br>
Even if it is a case of adding a bit of binding and following up<br>
with the code using it a bit later.<br>
<br>
I'd asked for input on the clock stuff then forgot all about it on<br>
the later version. Sorry about that!<br>
<span class="HOEnZb"><font color="#888888"><br>
Jonathan<br>
</font></span></blockquote></div><br></div>