[PATCH v2 2/3] drm: Add ASPEED GFX driver

Joel Stanley joel at jms.id.au
Wed Apr 3 00:14:43 UTC 2019


On Tue, 2 Apr 2019 at 06:26, Sam Ravnborg <sam at ravnborg.org> wrote:
>
> Hi Joel
>
> > index 000000000000..fb56e425bd48
> > --- /dev/null
> > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h
> > @@ -0,0 +1,104 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +// Copyright 2018 IBM Corporation
> > +
> > +#include <drm/drmP.h>
> > +#include <drm/drm_simple_kms_helper.h>
>
> A drmP.h include was left here, can we have this removed too.
>
> > +
> > +static struct drm_driver aspeed_gfx_driver = {
> > +     .driver_features        = DRIVER_GEM | DRIVER_MODESET |
> > +                             DRIVER_PRIME | DRIVER_ATOMIC |
> > +                             DRIVER_HAVE_IRQ,
> DRIVER_HAVE_IRQ is obsolete and not needed anymore.
> See drm_drv.h for details.
>
> With these few things fixed:
> Reviewed-by: Sam Ravnborg <sam at ravnborg.org>

Thanks for the review. I've fixed those up now, and this time I'm sure
there's no drmP.h!

Cheers,

Joe


More information about the dri-devel mailing list