Re: [PATCH 05/14] drm/i915: Use drm_dbg_kms() for initial FB debugs

[Jani Nikula <jani.nikula@linux.intel.com>]

On Tue, 12 May 2026, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> The initial FB stuff is ultimately about display stuff, so
> use the proper display specific debug level for it.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Reviewed-by: Jani Nikula <jani.nikula@intel.com>

> ---
>  drivers/gpu/drm/i915/i915_initial_plane.c     |  4 ++--
>  drivers/gpu/drm/xe/display/xe_initial_plane.c | 12 ++++++------
>  2 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
> index 56d31f83707b..62365ba1ff97 100644
> --- a/drivers/gpu/drm/i915/i915_initial_plane.c
> +++ b/drivers/gpu/drm/i915/i915_initial_plane.c
> @@ -68,8 +68,8 @@ initial_plane_phys(struct drm_i915_private *i915,
>  		return false;
>  	}
>  
> -	drm_dbg(&i915->drm, "Using dma_addr=%pa, based on initial plane programming\n",
> -		&dma_addr);
> +	drm_dbg_kms(&i915->drm, "Using dma_addr=%pa, based on initial plane programming\n",
> +		    &dma_addr);
>  
>  	*out_phys_base = dma_addr - mem->region.start;
>  	*out_mem = mem;
> diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
> index 1f2919797f6f..37cfc8a55031 100644
> --- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
> +++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
> @@ -59,9 +59,9 @@ initial_plane_bo(struct xe_device *xe,
>  			return NULL;
>  		}
>  
> -		drm_dbg(&xe->drm,
> -			"Using phys_base=%pa, based on initial plane programming\n",
> -			&phys_base);
> +		drm_dbg_kms(&xe->drm,
> +			    "Using phys_base=%pa, based on initial plane programming\n",
> +			    &phys_base);
>  	} else {
>  		struct ttm_resource_manager *stolen = ttm_manager_type(&xe->ttm, XE_PL_STOLEN);
>  
> @@ -78,9 +78,9 @@ initial_plane_bo(struct xe_device *xe,
>  	bo = xe_bo_create_pin_map_at_novm(xe, tile0, size, phys_base,
>  					  ttm_bo_type_kernel, flags, 0, false);
>  	if (IS_ERR(bo)) {
> -		drm_dbg(&xe->drm,
> -			"Failed to create bo phys_base=%pa size %u with flags %x: %li\n",
> -			&phys_base, size, flags, PTR_ERR(bo));
> +		drm_dbg_kms(&xe->drm,
> +			    "Failed to create bo phys_base=%pa size %u with flags %x: %li\n",
> +			    &phys_base, size, flags, PTR_ERR(bo));
>  		return NULL;
>  	}

-- 
Jani Nikula, Intel