On Thu, 04 Feb 2016, Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > On Thu, Feb 04, 2016 at 06:52:47PM +0200, Jani Nikula wrote: >> Skip v3 gpio element because the support is not there, and skip gpio >> element on non-vlv because the sideband code is vlv specific. >> >> v2: the gpio stuff is currently only supported on vlv (Ville) >> >> Cc: drm-intel-fixes@xxxxxxxxxxxxxxxxxxxxx >> Fixes: 2a33d93486f2 ("drm/i915/bios: add support for MIPI sequence block v3") >> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> And pushed to dinq, thanks for the review. BR, Jani. > >> --- >> drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 13 +++++++++++++ >> 1 file changed, 13 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c >> index f4d303ee538b..bcc083db7632 100644 >> --- a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c >> +++ b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c >> @@ -205,6 +205,9 @@ static const u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, const u8 *data) >> struct drm_device *dev = intel_dsi->base.base.dev; >> struct drm_i915_private *dev_priv = dev->dev_private; >> >> + if (dev_priv->vbt.dsi.seq_version >= 3) >> + data++; >> + >> gpio = *data++; >> >> /* pull up/down */ >> @@ -215,6 +218,16 @@ static const u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, const u8 *data) >> goto out; >> } >> >> + if (!IS_VALLEYVIEW(dev_priv)) { >> + DRM_DEBUG_KMS("GPIO element not supported on this platform\n"); >> + goto out; >> + } >> + >> + if (dev_priv->vbt.dsi.seq_version >= 3) { >> + DRM_DEBUG_KMS("GPIO element v3 not supported\n"); >> + goto out; >> + } >> + >> function = gtable[gpio].function_reg; >> pad = gtable[gpio].pad_reg; >> >> -- >> 2.1.4 >> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx