<div dir="ltr">Thanks for your enthusiasm,<div><br></div><div>Let me know when you get around to it!</div><div><br></div><div>-Chris</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 29, 2016 at 8:39 PM, Tanu Kaskinen <span dir="ltr"><<a href="mailto:tanuk@iki.fi" target="_blank">tanuk@iki.fi</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 Fri, 2016-01-29 at 11:11 +1100, Chris Billington wrote:<br>
> Hi all,<br>
><br>
> I was wondering if anyone has had the time to look at my patch?<br>
><br>
> No rush if there are more important things to be doing!<br>
<br>
</span>I haven't had time yet. I will get to it eventually.<br>
<span class=""><br>
> > The other four patches are implementing the setting of individual latency<br>
> > offsets of sinks and sources. This now plays nicely with port latency<br>
> > offsets. The way I've done this is to rename everything pertaining to the<br>
> > latency offsets inherited from ports, so that they are all called<br>
> > sink->port_latency_offset etc. Then I've introduced a new latency offset<br>
> > for sinks and sources, called just sink->latency_offset etc, which is for<br>
> > the latency offset set on that sink or source alone. The total latency<br>
> > reported by the device then includes the sum of both of them.<br>
<br>
</span>Sounds very good.<br>
<span class=""><br>
> > The only thing I didn't do was restoring from disk. Tanu, you said this<br>
> > might be tricky, but I wasn't able to understand module-device-restore<br>
> > enough to even see the trickiness you were talking about, I wasn't able to<br>
> > get that far! I'm not very familiar with the codebase - I'm just here<br>
> > selfishly to try and get my feature in so my application works better :p.<br>
> > So apologies for that, I hope the changes are worthy without it.<br>
<br>
</span>I think it's ok to leave the persistence feature out.<br>
<span class="HOEnZb"><font color="#888888"><br>
-- <br>
Tanu<br>
</font></span></blockquote></div><br></div>