On Tue, 3 Dec 2019 at 11:06, Emil Velikov <emil.l.veli...@gmail.com> wrote:
>
> Hi Thomas,
>
> On Tue, 3 Dec 2019 at 10:04, Thomas Zimmermann <tzimmerm...@suse.de> wrote:
> >
> > Declarations of drm_legacy_pci_{init,exit}() are being moved to
> > drm_legacy.h. CONFIG_DRM_LEGACY protects the implementation.
> >
> > Signed-off-by: Thomas Zimmermann <tzimmerm...@suse.de>
> > ---
> >  drivers/gpu/drm/drm_pci.c |  4 ++++
> >  include/drm/drm_legacy.h  | 29 ++++++++++++++++++++++++++++-
> >  include/drm/drm_pci.h     | 15 ---------------
> >  3 files changed, 32 insertions(+), 16 deletions(-)
> >
> How about we make this patch 01/12, this way we can avoid the current
> case of - add code only to be moved in next patch.
>
> As always - couple of ideas for follow-up cleanups.
>  - drm_irq_by_busid() is legacy codebase (see the DRIVER_LEGACY check
> at the top). How about we give it a _legacy_ name?
>  - WE HAVE A FEW driver_features

Pardon there, the last one should read:
- We have a few instances that read driver_features directly, instead
of using the drm_core_check_feature() helper.
Might be a good idea to address those.

Just some nitpicks noticed while skimming through. It's not a
requirement by any means.

HTH
Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to