<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Dec 5, 2016 11:43 AM, "Philipp Zabel" <<a href="mailto:p.zabel@pengutronix.de">p.zabel@pengutronix.de</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">Am Montag, den 05.12.2016, 11:19 +0100 schrieb Philipp Zabel:<br>
> Hi Marek,<br>
><br>
> Am Freitag, den 02.12.2016, 23:10 +0100 schrieb Marek Olšák:<br>
> > Shouldn't this be in pipe_context if it does a copy?<br>
<br>
</div>Actually, resource_changed is not supposed to create a copy immediately.<br>
Maybe instead of "trigger recreation of derived internal resources" it<br>
should be described as "mark resource as changed so derived internal<br>
resources will be recreated on next use".<br>
<div class="quoted-text"><br>
> > It's basically the opposite of flush_resource, right?<br>
<br>
</div>Contrary to flush_resource, which has an immediate, externally visible<br>
effect for which a context may be needed, resource_changed can just<br>
update some internal state. The changed resource should be marked as<br>
changed for all contexts, which is why I saw no reason to call<br>
resource_changed with any one specific context.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Sounds good to me.</div><div dir="auto"><br></div><div dir="auto">Marek</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
regards<br>
<font color="#888888">Philipp<br>
<br>
</font></blockquote></div><br></div></div></div>