On Sat, 30 May 2020 at 14:18, Sam Ravnborg <s...@ravnborg.org> wrote: > > Hi Emil. > On Sat, May 30, 2020 at 01:46:40PM +0100, Emil Velikov wrote: > > Currently the ret handling is all over the place - with two redundant > > assignments and another one addressed earlier. > > > > Use the exact same flow in both functions. > > > > v2: straighten the code flow, instead of just removing the assignments > Now even I should be able to follow the flow - thanks :-) > Fwiw reading at the code the first couple of times, did confuse the hell out of me. So "there is nothing wrong with your television set" :-P
> > > > Cc: David Airlie <airl...@linux.ie> > > Cc: Daniel Vetter <dan...@ffwll.ch> > > Cc: Sam Ravnborg <s...@ravnborg.org> > > Cc: Colin Ian King <colin.k...@canonical.com> > > Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> > Reviewed-by: Sam Ravnborg <s...@ravnborg.org> Thanks. If you're up for a few more neat patches - check these out [1]. -Emil [1] https://patchwork.freedesktop.org/series/76967/ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel