drm/i915: remove redundant checks for a null fb pointer
A prior check and return when pointer fb is null makes subsequent null checks on fb redundant. Remove the redundant null checks. Addresses-Coverity: ("Logically dead code") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20191210142349.333171-1-colin.king@canonical.com
This commit is contained in:
parent
7b0bcead18
commit
f3417d703f
|
@ -2582,8 +2582,8 @@ static void intel_plane_hw_info(struct seq_file *m, struct intel_plane *plane)
|
||||||
plane_state->hw.rotation);
|
plane_state->hw.rotation);
|
||||||
|
|
||||||
seq_printf(m, "\t\thw: fb=%d,%s,%dx%d, visible=%s, src=" DRM_RECT_FP_FMT ", dst=" DRM_RECT_FMT ", rotation=%s\n",
|
seq_printf(m, "\t\thw: fb=%d,%s,%dx%d, visible=%s, src=" DRM_RECT_FP_FMT ", dst=" DRM_RECT_FMT ", rotation=%s\n",
|
||||||
fb ? fb->base.id : 0, fb ? format_name.str : "n/a",
|
fb->base.id, format_name.str,
|
||||||
fb ? fb->width : 0, fb ? fb->height : 0,
|
fb->width, fb->height,
|
||||||
yesno(plane_state->uapi.visible),
|
yesno(plane_state->uapi.visible),
|
||||||
DRM_RECT_FP_ARG(&plane_state->uapi.src),
|
DRM_RECT_FP_ARG(&plane_state->uapi.src),
|
||||||
DRM_RECT_ARG(&plane_state->uapi.dst),
|
DRM_RECT_ARG(&plane_state->uapi.dst),
|
||||||
|
|
Loading…
Reference in New Issue