Re: [PATCH v2 1/9] drm/i915: Remove superfluous NULL check

Daniel Vetter
2015-12-04T10:21:28Z

On Tue, Dec 01, 2015 at 05:09:50PM +0100, Takashi Iwai wrote:
> to_intel_crtc() always returns a non-NULL pointer.
>
> Signed-off-by: Takashi Iwai <tiwai@suse.de>

Queued for -next, thanks for the patch.
-Daniel

> ---
> drivers/gpu/drm/i915/intel_audio.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c
> index 4dccd9b003a1..0c38cc6c82ae 100644
> --- a/drivers/gpu/drm/i915/intel_audio.c
> +++ b/drivers/gpu/drm/i915/intel_audio.c
> @@ -656,10 +656,6 @@ static int i915_audio_component_sync_audio_rate(struct device *dev,
> intel_dig_port = enc_to_dig_port(&intel_encoder->base);
> if (port == intel_dig_port->port) {
> crtc = to_intel_crtc(intel_encoder->base.crtc);
> - if (!crtc) {
> - DRM_DEBUG_KMS("%s: crtc is NULL\n", __func__);
> - continue;
> - }
> pipe = crtc->pipe;
> break;
> }
> --
> 2.6.3
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

--
Daniel Vetter
Software Engineer, Intel Corporation