On Thu, 24 Mar 2016, Mika Kahola <mika.kahola@xxxxxxxxx> wrote: > Housekeeping stuff. > > Reviewed-by: Mika Kahola <mika.kahola@xxxxxxxxx> Thanks, pushed this one patch to drm-intel-next-queued. BR, Jani. > > On Fri, 2016-03-18 at 13:11 +0200, Jani Nikula wrote: >> The DSI sequence blocks contain gpio index references, not actual gpio >> numbers. No functional changes. >> >> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> >> --- >> drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 16 ++++++++-------- >> 1 file changed, 8 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c >> index 8302a972d2d4..f687b2e9d8ca 100644 >> --- a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c >> +++ b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c >> @@ -198,7 +198,7 @@ static const u8 *mipi_exec_delay(struct intel_dsi *intel_dsi, const u8 *data) >> >> static const u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, const u8 *data) >> { >> - u8 gpio, action; >> + u8 gpio_index, action; >> u16 function, pad; >> u32 val; >> struct drm_device *dev = intel_dsi->base.base.dev; >> @@ -207,13 +207,13 @@ static const u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, const u8 *data) >> if (dev_priv->vbt.dsi.seq_version >= 3) >> data++; >> >> - gpio = *data++; >> + gpio_index = *data++; >> >> /* pull up/down */ >> action = *data++ & 1; >> >> - if (gpio >= ARRAY_SIZE(gtable)) { >> - DRM_DEBUG_KMS("unknown gpio %u\n", gpio); >> + if (gpio_index >= ARRAY_SIZE(gtable)) { >> + DRM_DEBUG_KMS("unknown gpio index %u\n", gpio_index); >> goto out; >> } >> >> @@ -227,16 +227,16 @@ static const u8 *mipi_exec_gpio(struct intel_dsi *intel_dsi, const u8 *data) >> goto out; >> } >> >> - function = gtable[gpio].function_reg; >> - pad = gtable[gpio].pad_reg; >> + function = gtable[gpio_index].function_reg; >> + pad = gtable[gpio_index].pad_reg; >> >> mutex_lock(&dev_priv->sb_lock); >> - if (!gtable[gpio].init) { >> + if (!gtable[gpio_index].init) { >> /* program the function */ >> /* FIXME: remove constant below */ >> vlv_iosf_sb_write(dev_priv, IOSF_PORT_GPIO_NC, function, >> 0x2000CC00); >> - gtable[gpio].init = 1; >> + gtable[gpio_index].init = 1; >> } >> >> val = 0x4 | action; > > -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx