The comment "we don't use get_back_bo() since ..." only makes sense in
context on droid_swap_buffers.  Why aren't you calling this helper
in dri2_drm_swap_buffers and dri2_wl_swap_buffers_with_damage?  It seems
like you can.


On Tue, Oct 17, 2017 at 7:38 AM, Emil Velikov <emil.l.veli...@gmail.com>
wrote:

> On 6 October 2017 at 22:38, Gwan-gyeong Mun <elong...@gmail.com> wrote:
> > To share common update buffer age code.
> > This updates old buffer's age and sets current back buffer's age to 1.
> >
> > In preparation to adding of new platform which uses this helper.
> >
> > v2:
> >  - Fixes from Eric's review:
> >    a) Split out series of refactor for helpers to a separate series.
> >    b) Add the new helper function and use them to replace the old code
> in the
> >       same patch.
> >  - Fixes from Rob's review:
> >    Remove unneeded ifdef block
> >
> Reviewed-by: Emil Velikov <emil.veli...@collabora.com>
>
> -Emil
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to