[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