Commit 7cb017db authored by Stephen Boyd's avatar Stephen Boyd Committed by Rob Clark
Browse files

drm/msm: Move FB debug prints to drm_dbg_state() 



These are verbose prints that tell us about the framebuffer state. Let's
move them to drm_dbg_state() so that they're only printed if we're
interested in verbose state logging while drm debugging.

Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Abhinav Kumar <abhinavk@codeaurora.org>
Cc: Kuogee Hsieh <khsieh@codeaurora.org>
Cc: aravindh@codeaurora.org
Cc: Sean Paul <sean@poorly.run>
Signed-off-by: default avatarStephen Boyd <swboyd@chromium.org>
Link: https://lore.kernel.org/r/20210430193104.1770538-5-swboyd@chromium.org


Reviewed-by: default avatarAbhinav Kumar <abhinavk@codeaurora.org>
Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
parent e45b40ab

drivers/gpu/drm/msm/msm_fb.c

+4 −4
Original line number Diff line number Diff line
@@ -61,7 +61,7 @@ int msm_framebuffer_prepare(struct drm_framebuffer *fb, 


	for (i = 0; i < n; i++) {

		ret = msm_gem_get_and_pin_iova(fb->obj[i], aspace, &iova);

		DBG("FB[%u]: iova[%d]: %08llx (%d)", fb->base.id, i, iova, ret);

		drm_dbg_state(fb->dev, "FB[%u]: iova[%d]: %08llx (%d)", fb->base.id, i, iova, ret);

		if (ret)

			return ret;

	}
@@ -140,8 +140,8 @@ static struct drm_framebuffer *msm_framebuffer_init(struct drm_device *dev, 
	const struct msm_format *format;

	int ret, i, n;



	DBG("create framebuffer: dev=%p, mode_cmd=%p (%dx%d@%4.4s)",

			dev, mode_cmd, mode_cmd->width, mode_cmd->height,

	drm_dbg_state(dev, "create framebuffer: mode_cmd=%p (%dx%d@%4.4s)",

			mode_cmd, mode_cmd->width, mode_cmd->height,

			(char *)&mode_cmd->pixel_format);



	n = info->num_planes;
@@ -194,7 +194,7 @@ static struct drm_framebuffer *msm_framebuffer_init(struct drm_device *dev, 
		goto fail;

	}



	DBG("create: FB ID: %d (%p)", fb->base.id, fb);

	drm_dbg_state(dev, "create: FB ID: %d (%p)", fb->base.id, fb);



	return fb;