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

Ville Syrjälä
2015-12-04T12:16:59Z

On Tue, Dec 01, 2015 at 05:09:50PM +0100, Takashi Iwai wrote:
> to_intel_crtc() always returns a non-NULL pointer.

Eh? to_intel_crtc(NULL) should return NULL or we might have tons of
breakage on our hands. Or maybe the atomic work has gotten rid of that
assumption, but at least we used to depend on that heavily.

>
> Signed-off-by: Takashi Iwai <tiwai@suse.de>
> ---
> 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

--
Ville Syrjälä
Intel OTC