On 08/17/2012 06:50 PM, Inki Dae wrote: > crtc and encoder's dpms callback will be called before connector's dpms > is called so drm_helper_connector_dpms doesn't need to be called.
I can't understand this description. I know crtc and encoder dpms are called by drm_helper_connector_dpms. > Signed-off-by: Inki Dae <inki.dae at samsung.com> > Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com> > --- > drivers/gpu/drm/exynos/exynos_drm_connector.c | 9 ++++++++- > 1 files changed, 8 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_connector.c > b/drivers/gpu/drm/exynos/exynos_drm_connector.c > index d956819..65acf0d 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_connector.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_connector.c > @@ -226,6 +226,13 @@ static struct drm_connector_helper_funcs > exynos_connector_helper_funcs = { > .best_encoder = exynos_drm_best_encoder, > }; > > +static void exynos_drm_connector_dpms(struct drm_connector *connector, int > mode) > +{ > + DRM_DEBUG_KMS("%s\n", __FILE__); > + > + /* drm framework doesn't check NULL. */ > +} > + > static int exynos_drm_connector_fill_modes(struct drm_connector *connector, > unsigned int max_width, unsigned int max_height) > { > @@ -285,7 +292,7 @@ static void exynos_drm_connector_destroy(struct > drm_connector *connector) > } > > static struct drm_connector_funcs exynos_connector_funcs = { > - .dpms = drm_helper_connector_dpms, > + .dpms = exynos_drm_connector_dpms, > .fill_modes = exynos_drm_connector_fill_modes, > .detect = exynos_drm_connector_detect, > .destroy = exynos_drm_connector_destroy,