diff options
author | Jani Nikula <jani.nikula@intel.com> | 2016-02-04 13:50:50 +0300 |
---|---|---|
committer | Luis Henriques <luis.henriques@canonical.com> | 2016-03-04 13:26:37 +0300 |
commit | 87b3f9084b5b20e1865a5b4c552ad6828e68c1e0 (patch) | |
tree | 15fb51402136de943430a5122e17402170480107 /drivers/gpu/drm | |
parent | 8e3d44f5092d4e847eef0580f1c03b91afdc6265 (diff) | |
download | linux-87b3f9084b5b20e1865a5b4c552ad6828e68c1e0.tar.xz |
drm/i915/dsi: don't pass arbitrary data to sideband
commit 26f6f2d301c1fb46acb1138ee155125815239b0d upstream.
Since sequence block v2 the second byte contains flags other than just
pull up/down. Don't pass arbitrary data to the sideband interface.
The rest may or may not work for sequence block v2, but there should be
no harm done.
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/ebe3c2eee623afc4b3a134533b01f8d591d13f32.1454582914.git.jani.nikula@intel.com
(cherry picked from commit 4e1c63e3761b84ec7d87c75b58bbc8bcf18e98ee)
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Luis Henriques <luis.henriques@canonical.com>
Diffstat (limited to 'drivers/gpu/drm')
-rw-r--r-- | drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c index 9db5dc8f4521..53dbd2e85f06 100644 --- a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c +++ b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c @@ -171,7 +171,7 @@ static u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, u8 *data) gpio = *data++; /* pull up/down */ - action = *data++; + action = *data++ & 1; if (gpio >= ARRAY_SIZE(gtable)) { DRM_DEBUG_KMS("unknown gpio %u\n", gpio); |