[Mesa-dev] [PATCH 8/8] egl/drm: rename dri2_drm_create_surface()

Eric Engestrom eric.engestrom at imgtec.com
Tue Aug 8 16:34:57 UTC 2017


On Saturday, 2017-08-05 02:30:51 +0100, Emil Velikov wrote:
> On 5 August 2017 at 00:25, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> > From: Emil Velikov <emil.velikov at collabora.com>
> >
> > The function can handle only window surfaces, so let's rename it
> > accordingly, killing the wrapper around it.
> >
> > Suggested-by: Eric Engestrom <eric.engestrom at imgtec.com>
> > Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> > ---
> > New patch
> > ---
> >  src/egl/drivers/dri2/platform_drm.c | 17 ++++-------------
> >  1 file changed, 4 insertions(+), 13 deletions(-)
> >
> > diff --git a/src/egl/drivers/dri2/platform_drm.c b/src/egl/drivers/dri2/platform_drm.c
> > index 8d56fcb7698..89ad9e0d10c 100644
> > --- a/src/egl/drivers/dri2/platform_drm.c
> > +++ b/src/egl/drivers/dri2/platform_drm.c
> > @@ -91,9 +91,9 @@ has_free_buffers(struct gbm_surface *_surf)
> >  }
> >
> >  static _EGLSurface *
> > -dri2_drm_create_surface(_EGLDriver *drv, _EGLDisplay *disp, EGLint type,
> > -                        _EGLConfig *conf, void *native_surface,
> > -                        const EGLint *attrib_list)
> > +dri2_drm_create_window_surface(_EGLDriver *drv, _EGLDisplay *disp,
> > +                               _EGLConfig *conf, void *native_window,
> 
> Fixed this locally to read native_surface, instead of native_window.

With that fixed:
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>


More information about the mesa-dev mailing list