[Mesa-dev] [PATCH] egl/android: Fix build break with dri2_initialize_android _EGLDisplay parameter

[Eric Engestrom]

On Thursday, 2018-01-04 10:17:49 -0600, Rob Herring wrote:
> On Thu, Jan 4, 2018 at 8:16 AM, Eric Engestrom
> <eric.engestrom at imgtec.com> wrote:
> > On Wednesday, 2018-01-03 10:28:37 -0600, Rob Herring wrote:
> >> Commit 2f421651aca9 ("egl: let each platform decided how to handle
> >> LIBGL_ALWAYS_SOFTWARE") broke the build due to copy-n-paste of misnamed
> >> function parameter.:
> >>
> >> src/egl/drivers/dri2/platform_android.c:1183:8: error: use of undeclared identifier 'disp'
> >>
> >> Rather than just fixing 'disp', rename the function parameter 'dpy' to
> >> 'disp' to align with the other EGL platforms' implementations.
> >>
> >> Fixes: 2f421651aca9 ("egl: let each platform decided how to handle LIBGL_ALWAYS_SOFTWARE")
> >> Cc: Eric Engestrom <eric.engestrom at imgtec.com>
> >> Signed-off-by: Rob Herring <robh at kernel.org>
> >
> > Apologies, I didn't think to check this, and I don't have a setup to
> > build Mesa on Android so it didn't even get build-tested :(
> 
> You just have to push a branch[1].

Oh, I forgot this (:
Thanks for setting this up!

> 
> > Fwiw Mauro also emailed me last week about this, but I only just got
> > back from holiday this morning.
> >
> > Renaming to align with other platforms sounds like the right fix:
> > Acked-by: Eric Engestrom <eric.engestrom at imgtec.com>
> 
> Thanks.
> 
> Rob
> 
> [1] https://lists.freedesktop.org/archives/mesa-dev/2017-October/172604.html