Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandi...@intel.com>

On Thu, 2016-09-15 at 16:28 +0300, Jani Nikula wrote:
> Fixes sparse warning:
> 
> drivers/gpu/drm/i915/intel_dpll_mgr.c:1712:24: warning: Using plain
> integer as NULL pointer
> 
> Fixes: a277ca7dc01d ("drm/i915: Split bxt_ddi_pll_select()")
> Cc: Manasi Navare <manasi.d.nav...@intel.com>
> Cc: Ander Conselvan de Oliveira <ander.conselvan.de.olive...@intel.com>
> Cc: Durgadoss R <durgados...@intel.com>
> Cc: Rodrigo Vivi <rodrigo.v...@intel.com>
> Signed-off-by: Jani Nikula <jani.nik...@intel.com>
> ---
>  drivers/gpu/drm/i915/intel_dpll_mgr.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_dpll_mgr.c 
> b/drivers/gpu/drm/i915/intel_dpll_mgr.c
> index 4b067accd5cd..c26d18a574b6 100644
> --- a/drivers/gpu/drm/i915/intel_dpll_mgr.c
> +++ b/drivers/gpu/drm/i915/intel_dpll_mgr.c
> @@ -1709,12 +1709,12 @@ bxt_get_dpll(struct intel_crtc *crtc,
>       if (encoder->type == INTEL_OUTPUT_HDMI
>           && !bxt_ddi_hdmi_pll_dividers(crtc, crtc_state,
>                                         clock, &clk_div))
> -             return false;
> +             return NULL;
>  
>       if ((encoder->type == INTEL_OUTPUT_DP ||
>            encoder->type == INTEL_OUTPUT_EDP) &&
>           !bxt_ddi_dp_set_dpll_hw_state(clock, &dpll_hw_state))
> -             return false;
> +             return NULL;
>  
>       memset(&crtc_state->dpll_hw_state, 0,
>              sizeof(crtc_state->dpll_hw_state));

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to