On Thu, 24 Mar 2016, Mika Kahola <[email protected]> wrote:
> Housekeeping stuff.
>
> Reviewed-by: Mika Kahola <[email protected]>

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 <[email protected]>
>> ---
>>  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
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to