[Intel-gfx] [PATCH 04/10] drm/crtc: add interface to reinitialise the legacy mode group
Daniel Vetter
daniel at ffwll.ch
Tue May 13 12:15:47 CEST 2014
On Tue, May 13, 2014 at 10:26:56AM +0200, Thierry Reding wrote:
> On Mon, May 12, 2014 at 04:46:41PM +1000, Dave Airlie wrote:
> > From: Dave Airlie <airlied at redhat.com>
> >
> > This can be called to update things after dynamic connectors/encoders
> > are created/deleted.
> >
> > Signed-off-by: Dave Airlie <airlied at redhat.com>
> > ---
> > drivers/gpu/drm/drm_crtc.c | 9 +++++++++
> > include/drm/drm_crtc.h | 1 +
> > 2 files changed, 10 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> > index f1753e6..8bf87a6 100644
> > --- a/drivers/gpu/drm/drm_crtc.c
> > +++ b/drivers/gpu/drm/drm_crtc.c
> > @@ -1421,6 +1421,15 @@ int drm_mode_group_init_legacy_group(struct drm_device *dev,
> > }
> > EXPORT_SYMBOL(drm_mode_group_init_legacy_group);
> >
> > +void drm_reinit_primary_mode_group(struct drm_device *dev)
> > +{
> > + drm_modeset_lock_all(dev);
> > + drm_mode_group_destroy(&dev->primary->mode_group);
> > + drm_mode_group_init_legacy_group(dev, &dev->primary->mode_group);
> > + drm_modeset_unlock_all(dev);
> > +}
> > +EXPORT_SYMBOL(drm_reinit_primary_mode_group);
> > +
> > /**
> > * drm_crtc_convert_to_umode - convert a drm_display_mode into a modeinfo
> > * @out: drm_mode_modeinfo struct to return to the user
> > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > index c6b9e8a..55bc523 100644
> > --- a/include/drm/drm_crtc.h
> > +++ b/include/drm/drm_crtc.h
> > @@ -916,6 +916,7 @@ extern const char *drm_get_tv_select_name(int val);
> > extern void drm_fb_release(struct drm_file *file_priv);
> > extern int drm_mode_group_init_legacy_group(struct drm_device *dev, struct drm_mode_group *group);
> > extern void drm_mode_group_destroy(struct drm_mode_group *group);
> > +extern void drm_reinit_primary_mode_group(struct drm_device *dev);
>
> There was a patch series once to remove all usage of extern from
> prototypes. Was there an official decision against that which I missed,
> or did it just fall off the table?
Iirc I've done that for radeon or so and we decided that it's a bikeshed.
tbh I don't care, but in i915 I tend to rip them out since redundant.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
More information about the Intel-gfx
mailing list