<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 23, 2016 at 5:54 PM, Daniel Vetter <span dir="ltr"><<a href="mailto:daniel@ffwll.ch" target="_blank">daniel@ffwll.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Tue, Aug 23, 2016 at 05:39:36PM +0200, Andrea Merello wrote:<br>
> On Tue, Aug 23, 2016 at 5:20 PM, Daniel Vetter <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>> wrote:<br>
><br>
> > On Tue, Aug 23, 2016 at 04:08:04PM +0200, Andrea Merello wrote:<br>
> > > Introduce drm_simple_display_pipe_<wbr>attach_bridge() in order<br>
> > > to make it possible to use drm encoders with the simple display<br>
> > > pipes managed by simple_kms_helpers<br>
> > ><br>
> > > Suggested-by: Daniel Vetter <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>><br>
> > > Signed-off-by: Andrea Merello <<a href="mailto:andrea.merello@gmail.com">andrea.merello@gmail.com</a>><br>
> > > Cc: Noralf Trønnes <<a href="mailto:noralf@tronnes.org">noralf@tronnes.org</a>><br>
> > > Cc: Daniel Vetter <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>><br>
> > > Cc: David Airlie <<a href="mailto:airlied@linux.ie">airlied@linux.ie</a>><br>
> ><br>
> > Threading of your patch series is somehow broken, usually that should all<br>
> > work nicely if you've set up git send-email.<br>
> ><br>
> > One question: Should we ahve a drm_simple_display_pipe_<wbr>detach_bridge (for<br>
> > cleanup) too?<br>
> ><br>
><br>
> Unsure if it worths. May be nice to have a balanced pair, but it would<br>
> probably end up in<br>
> a quite redundant one-line func, that only calls drm_bridge_detach with the<br>
> very same argument.<br>
><br>
> ..But of course if you want I can add it in v2 series.<br>
<br>
</span>Yes it's just going to be a one-line, but it'll do a typecast and so<br>
better encapsulate the internals of the simple pipe helper. </blockquote><div><br></div><div>I'm unsure about what do you mean here. Why a typecast?<br><br>Wouldn't it be simply <span class="">drm_simple_display_pipe_<wbr>detach_bridge(struct drm_bridge *bridge) calling in turn drm_detach_bridge(struct drm_bridge *bridge) with the very same argument?<br><br>Or</span> if you want to stay behind the pipe, then it could be <span class="">drm_simple_display_pipe_<wbr>detach_bridge(</span><span class="">struct drm_simple_display_pipe *pipe)</span>, but I would say it just does something like drm_bridge_detach(pipe->encoder.bridge), so I don't really get your point about the cast, sorry..<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I think that's<br>
worth it - we have piles&piles of functions and #defines just to make<br>
little one-line bits of code a notch more readable.<br>
<br>
And yes the symmetry is nice too ;-)<br></blockquote><div><br></div><div>OK, then I'll add it :)<br><br></div><div>Andrea<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><font color="#888888">-Daniel<br>
</font></span><div class=""><div class="h5">--<br>
Daniel Vetter<br>
Software Engineer, Intel Corporation<br>
<a href="http://blog.ffwll.ch" rel="noreferrer" target="_blank">http://blog.ffwll.ch</a><br>
</div></div></blockquote></div><br></div></div>