On Wed, Sep 23, 2015 at 11:30:43PM +0530, Uma Shankar wrote: > DSI backlight support for bxt is added. > > TODO: There is no support for backlight control in drm panel > framework. This will be added as part of VBT version patches > fixing the backlight sequence. > > v2: Fixed Jani's review comments from previous patch. Added the > BXT DSI backlight code in this patch. Backlight setup and > enable/disable code for backlight is added in intel_dsi.c. > > v3: Rebased on latest drm-nightly. Fixed Jani's review comments. > > v4: Making backlight calls generic as per Jani's suggestion. > > Signed-off-by: Uma Shankar <[email protected]> > --- > drivers/gpu/drm/i915/intel_dsi.c | 20 +++++++++++++++++++- > 1 file changed, 19 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_dsi.c > b/drivers/gpu/drm/i915/intel_dsi.c > index 08bade2..3e7f796 100644 > --- a/drivers/gpu/drm/i915/intel_dsi.c > +++ b/drivers/gpu/drm/i915/intel_dsi.c > @@ -438,6 +438,7 @@ static void intel_dsi_enable(struct intel_encoder > *encoder) > struct drm_device *dev = encoder->base.dev; > struct drm_i915_private *dev_priv = dev->dev_private; > struct intel_dsi *intel_dsi = enc_to_intel_dsi(&encoder->base); > + struct intel_connector *intel_connector = intel_dsi->attached_connector; > enum port port; > > DRM_DEBUG_KMS("\n"); > @@ -458,6 +459,9 @@ static void intel_dsi_enable(struct intel_encoder > *encoder) > > intel_dsi_port_enable(encoder); > } > + > + msleep(intel_dsi->backlight_on_delay); > + intel_panel_enable_backlight(intel_connector); > } > > static void intel_dsi_pre_enable(struct intel_encoder *encoder) > @@ -623,10 +627,14 @@ static void intel_dsi_post_disable(struct intel_encoder > *encoder) > { > struct drm_i915_private *dev_priv = encoder->base.dev->dev_private; > struct intel_dsi *intel_dsi = enc_to_intel_dsi(&encoder->base); > + struct intel_connector *intel_connector = intel_dsi->attached_connector; > u32 val; > > DRM_DEBUG_KMS("\n"); > > + intel_panel_disable_backlight(intel_connector); > + msleep(intel_dsi->backlight_off_delay); > + > intel_dsi_disable(encoder); > > intel_dsi_clear_device_ready(encoder); > @@ -1226,8 +1234,18 @@ void intel_dsi_init(struct drm_device *dev) > > intel_panel_init(&intel_connector->panel, fixed_mode, NULL); > > - return; > + /* > + * Pipe parameter is not used for BXT. > + * Passing INVALID_PIPE to adher to API requirement. > + */ > + if (IS_BROXTON(dev)) > + intel_panel_setup_backlight(connector, INVALID_PIPE); > + else > + intel_panel_setup_backlight(connector, > + intel_encoder->crtc_mask == (1 << PIPE_A) ? > + PIPE_A : PIPE_B);
Is this aganst some ancient kernel version? We have backlight calls in the dsi code already. > > + return; > err: > drm_encoder_cleanup(&intel_encoder->base); > kfree(intel_dsi); > -- > 1.7.9.5 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
