On Wed, Aug 12, 2015 at 5:00 PM, Thierry Reding <thierry.reding at gmail.com> wrote: > From: Thierry Reding <treding at nvidia.com> > > If the DSI output isn't connected, then mdfld_dsi_encoder_get_pipe() > will return -1. The mdfld_dsi_dp_mode_set() function doesn't properly > check for this condition and causes the following compiler warnings: > > CC drivers/gpu/drm/gma500/mdfld_dsi_dpi.o > drivers/gpu/drm/gma500/mdfld_dsi_dpi.c: In function > âmdfld_dsi_dpi_mode_setâ: > drivers/gpu/drm/gma500/mdfld_dsi_dpi.c:828:35: warning: array > subscript is below array bounds [-Warray-bounds] > u32 pipeconf = dev_priv->pipeconf[pipe]; > ^ > drivers/gpu/drm/gma500/mdfld_dsi_dpi.c:829:33: warning: array > subscript is below array bounds [-Warray-bounds] > u32 dspcntr = dev_priv->dspcntr[pipe]; > ^ > > Fix this by checking for a valid pipe before indexing the pipeconf and > dspcntr arrays. > > Cc: Patrik Jakobsson <patrik.r.jakobsson at gmail.com> > Signed-off-by: Thierry Reding <treding at nvidia.com>
Reviewed-by: Patrik Jakobsson <patrik.r.jakobsson at gmail.com>