<div dir="ltr">Hi all,<div><br></div><div>I was wondering if anyone has had the time to look at my patch?</div><div><br></div><div>No rush if there are more important things to be doing!</div><div><br></div><div>Regards,</div><div><br></div><div>Chris</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 23, 2016 at 12:49 PM, Chris Billington <span dir="ltr"><<a href="mailto:chrisjbillington@gmail.com" target="_blank">chrisjbillington@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Tanu, Peter,<div><br></div><div>The preceding patches are for implementing setting latency offsets of individual sources and sinks, as discussed after the previous patch I made a few weeks ago.</div><div><br></div><div>Two of the patches are trivial bugfixes:<br><br>patch 5/6 is using int64s for latency offsets in pactl and pacmd instead of int32s, as pointed out by Peter.<br><br>Patch 1/6 is having sources update the latency offsets they inherit from a port when the active port changes - previously they didn't do so. I assume this is a bug, but perhaps there is some reason this was not done that I am not aware of.</div><div><br></div><div>The other four patches are implementing the setting of individual latency offsets of sinks and sources. This now plays nicely with port latency offsets. The way I've done this is to rename everything pertaining to the latency offsets inherited from ports, so that they are all called sink->port_latency_offset etc. Then I've introduced a new latency offset for sinks and sources, called just sink->latency_offset etc, which is for the latency offset set on that sink or source alone. The total latency reported by the device then includes the sum of both of them.</div><div><br></div><div>As suggested by Tanu, shell completion for pacmd and pactl has been included, and the protocol change is in a separate commit.</div><div><br></div><div>The only thing I didn't do was restoring from disk. Tanu, you said this might be tricky, but I wasn't able to understand module-device-restore enough to even see the trickiness you were talking about, I wasn't able to get that far! I'm not very familiar with the codebase - I'm just here selfishly to try and get my feature in so my application works better :p. So apologies for that, I hope the changes are worthy without it.</div><div><br></div><div>Please advise if there's anything else I ought to do with this, I can refactor commits if they're split up too much or little, or whatever.</div><div><br></div><div>Cheers,</div><div><br></div><div>Chris</div></div>
</blockquote></div><br></div>