drm_hwcomposer: Remove unnecessary warning
When mirroring, this message spams the logs since we loop over the
device planes for multiple CRTCs and print this message for the primary
planes of each active CRTC.
The case this is avoiding (avoid sharing primary planes) is still valid,
afaict, so leave the return statement.
Change-Id: I14afce4918baa5652ec16a3610fe374cccf72422
Signed-off-by: Sean Paul <seanpaul@chromium.org>
diff --git a/drm/DrmPlane.cpp b/drm/DrmPlane.cpp
index 310e67b..dbb5ad6 100644
--- a/drm/DrmPlane.cpp
+++ b/drm/DrmPlane.cpp
@@ -155,14 +155,11 @@
if (crtc_prop_val != 0 && crtc_prop_val != crtc.GetId() &&
GetType() == DRM_PLANE_TYPE_PRIMARY) {
// Some DRM driver such as omap_drm allows sharing primary plane between
- // CRTCs, but the primay plane could not be shared if it has been used by
+ // CRTCs, but the primary plane could not be shared if it has been used by
// any CRTC already, which is protected by the plane_switching_crtc function
// in the kernel drivers/gpu/drm/drm_atomic.c file.
// The current drm_hwc design is not ready to support such scenario yet,
// so adding the CRTC status check here to workaorund for now.
- ALOGW("%s: This Plane(id=%d) is activated for Crtc(id=%" PRIu64
- "), could not be used for Crtc (id=%d)",
- __FUNCTION__, GetId(), crtc_prop_val, crtc.GetId());
return false;
}