diff mbox series

[v2,1/6] drm/i915/bios: Use drm_dbg_kms() consistently

Message ID 20240923152453.11230-2-ville.syrjala@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series drm/i915/bios: Refactor ROM access | expand

Commit Message

Ville Syrjälä Sept. 23, 2024, 3:24 p.m. UTC
From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Replace the few oddball drm_dbg() calls in VBT related code
with drm_dbg_kms() as that is what we generally use for all
display code.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_bios.c | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

Comments

Jani Nikula Sept. 24, 2024, 1:52 p.m. UTC | #1
On Mon, 23 Sep 2024, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Replace the few oddball drm_dbg() calls in VBT related code
> with drm_dbg_kms() as that is what we generally use for all
> display code.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

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

> ---
>  drivers/gpu/drm/i915/display/intel_bios.c | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
> index daa4b9535123..b00aad23d6c2 100644
> --- a/drivers/gpu/drm/i915/display/intel_bios.c
> +++ b/drivers/gpu/drm/i915/display/intel_bios.c
> @@ -1706,8 +1706,8 @@ parse_mipi_config(struct intel_display *display,
>  		return;
>  	}
>  
> -	drm_dbg(display->drm, "Found MIPI Config block, panel index = %d\n",
> -		panel_type);
> +	drm_dbg_kms(display->drm, "Found MIPI Config block, panel index = %d\n",
> +		    panel_type);
>  
>  	/*
>  	 * get hold of the correct configuration block and pps data as per
> @@ -2067,8 +2067,8 @@ parse_mipi_sequence(struct intel_display *display,
>  		return;
>  	}
>  
> -	drm_dbg(display->drm, "Found MIPI sequence block v%u\n",
> -		sequence->version);
> +	drm_dbg_kms(display->drm, "Found MIPI sequence block v%u\n",
> +		    sequence->version);
>  
>  	seq_data = find_panel_sequence_block(display, sequence, panel_type, &seq_size);
>  	if (!seq_data)
> @@ -2114,7 +2114,7 @@ parse_mipi_sequence(struct intel_display *display,
>  
>  	fixup_mipi_sequences(display, panel);
>  
> -	drm_dbg(display->drm, "MIPI related VBT parsing complete\n");
> +	drm_dbg_kms(display->drm, "MIPI related VBT parsing complete\n");
>  	return;
>  
>  err:
> @@ -2771,9 +2771,9 @@ static bool child_device_size_valid(struct intel_display *display, int size)
>  	expected_size = child_device_expected_size(display->vbt.version);
>  	if (expected_size < 0) {
>  		expected_size = sizeof(struct child_device_config);
> -		drm_dbg(display->drm,
> -			"Expected child device config size for VBT version %u not known; assuming %d\n",
> -			display->vbt.version, expected_size);
> +		drm_dbg_kms(display->drm,
> +			    "Expected child device config size for VBT version %u not known; assuming %d\n",
> +			    display->vbt.version, expected_size);
>  	}
>  
>  	/* Flag an error for unexpected size, but continue anyway. */
> @@ -3143,14 +3143,14 @@ static struct vbt_header *oprom_get_vbt(struct intel_display *display,
>  		goto err_unmap_oprom;
>  
>  	if (sizeof(struct vbt_header) > size) {
> -		drm_dbg(display->drm, "VBT header incomplete\n");
> +		drm_dbg_kms(display->drm, "VBT header incomplete\n");
>  		goto err_unmap_oprom;
>  	}
>  
>  	vbt_size = ioread16(p + offsetof(struct vbt_header, vbt_size));
>  	if (vbt_size > size) {
> -		drm_dbg(display->drm,
> -			"VBT incomplete (vbt_size overflows)\n");
> +		drm_dbg_kms(display->drm,
> +			    "VBT incomplete (vbt_size overflows)\n");
>  		goto err_unmap_oprom;
>  	}
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
index daa4b9535123..b00aad23d6c2 100644
--- a/drivers/gpu/drm/i915/display/intel_bios.c
+++ b/drivers/gpu/drm/i915/display/intel_bios.c
@@ -1706,8 +1706,8 @@  parse_mipi_config(struct intel_display *display,
 		return;
 	}
 
-	drm_dbg(display->drm, "Found MIPI Config block, panel index = %d\n",
-		panel_type);
+	drm_dbg_kms(display->drm, "Found MIPI Config block, panel index = %d\n",
+		    panel_type);
 
 	/*
 	 * get hold of the correct configuration block and pps data as per
@@ -2067,8 +2067,8 @@  parse_mipi_sequence(struct intel_display *display,
 		return;
 	}
 
-	drm_dbg(display->drm, "Found MIPI sequence block v%u\n",
-		sequence->version);
+	drm_dbg_kms(display->drm, "Found MIPI sequence block v%u\n",
+		    sequence->version);
 
 	seq_data = find_panel_sequence_block(display, sequence, panel_type, &seq_size);
 	if (!seq_data)
@@ -2114,7 +2114,7 @@  parse_mipi_sequence(struct intel_display *display,
 
 	fixup_mipi_sequences(display, panel);
 
-	drm_dbg(display->drm, "MIPI related VBT parsing complete\n");
+	drm_dbg_kms(display->drm, "MIPI related VBT parsing complete\n");
 	return;
 
 err:
@@ -2771,9 +2771,9 @@  static bool child_device_size_valid(struct intel_display *display, int size)
 	expected_size = child_device_expected_size(display->vbt.version);
 	if (expected_size < 0) {
 		expected_size = sizeof(struct child_device_config);
-		drm_dbg(display->drm,
-			"Expected child device config size for VBT version %u not known; assuming %d\n",
-			display->vbt.version, expected_size);
+		drm_dbg_kms(display->drm,
+			    "Expected child device config size for VBT version %u not known; assuming %d\n",
+			    display->vbt.version, expected_size);
 	}
 
 	/* Flag an error for unexpected size, but continue anyway. */
@@ -3143,14 +3143,14 @@  static struct vbt_header *oprom_get_vbt(struct intel_display *display,
 		goto err_unmap_oprom;
 
 	if (sizeof(struct vbt_header) > size) {
-		drm_dbg(display->drm, "VBT header incomplete\n");
+		drm_dbg_kms(display->drm, "VBT header incomplete\n");
 		goto err_unmap_oprom;
 	}
 
 	vbt_size = ioread16(p + offsetof(struct vbt_header, vbt_size));
 	if (vbt_size > size) {
-		drm_dbg(display->drm,
-			"VBT incomplete (vbt_size overflows)\n");
+		drm_dbg_kms(display->drm,
+			    "VBT incomplete (vbt_size overflows)\n");
 		goto err_unmap_oprom;
 	}