[Intel-gfx] [PATCH 2/6] drm/i915: do not use 'false' as a NULL pointer

Pandiyan, Dhinakaran dhinakaran.pandiyan at intel.com
Thu Sep 15 17:13:08 UTC 2016


Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at 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.navare at intel.com>
> Cc: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira at intel.com>
> Cc: Durgadoss R <durgadoss.r at intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Signed-off-by: Jani Nikula <jani.nikula at 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));



More information about the Intel-gfx mailing list