1. Patchew
  2. linux
  3. View series

[PATCH] drm/amd/display: Clean up errors in bios_parser2.c

Ran Sun posted 1 patch 2 years, 6 months ago 
.../drm/amd/display/dc/bios/bios_parser2.c    | 32 +++++++++----------
1 file changed, 16 insertions(+), 16 deletions(-)
[PATCH] drm/amd/display: Clean up errors in bios_parser2.c
Posted by Ran Sun 2 years, 6 months ago 
Fix the following errors reported by checkpatch:

ERROR: switch and case should be at the same indent
ERROR: code indent should use tabs where possible

Signed-off-by: Ran Sun <sunran001@208suo.com>
---
 .../drm/amd/display/dc/bios/bios_parser2.c    | 32 +++++++++----------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
index 540d19efad8f..033ce2638eb2 100644
--- a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
+++ b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
@@ -772,20 +772,20 @@ static enum bp_result bios_parser_get_device_tag(
 		return BP_RESULT_BADINPUT;
 
 	switch (bp->object_info_tbl.revision.minor) {
-	    case 4:
-	    default:
+	case 4:
+	default:
 	        /* getBiosObject will return MXM object */
-	        object = get_bios_object(bp, connector_object_id);
+		object = get_bios_object(bp, connector_object_id);
 
 		if (!object) {
 			BREAK_TO_DEBUGGER(); /* Invalid object id */
 			return BP_RESULT_BADINPUT;
 		}
 
-	        info->acpi_device = 0; /* BIOS no longer provides this */
-	        info->dev_id = device_type_from_device_id(object->device_tag);
-	        break;
-	    case 5:
+		info->acpi_device = 0; /* BIOS no longer provides this */
+		info->dev_id = device_type_from_device_id(object->device_tag);
+		break;
+	case 5:
 		object_path_v3 = get_bios_object_from_path_v3(bp, connector_object_id);
 
 		if (!object_path_v3) {
@@ -1580,13 +1580,13 @@ static bool bios_parser_is_device_id_supported(
 	uint32_t mask = get_support_mask_for_device_id(id);
 
 	switch (bp->object_info_tbl.revision.minor) {
-	    case 4:
-	    default:
-	        return (le16_to_cpu(bp->object_info_tbl.v1_4->supporteddevices) & mask) != 0;
-			break;
-	    case 5:
-			return (le16_to_cpu(bp->object_info_tbl.v1_5->supporteddevices) & mask) != 0;
-			break;
+	case 4:
+	default:
+		return (le16_to_cpu(bp->object_info_tbl.v1_4->supporteddevices) & mask) != 0;
+		break;
+	case 5:
+		return (le16_to_cpu(bp->object_info_tbl.v1_5->supporteddevices) & mask) != 0;
+		break;
 	}
 
 	return false;
@@ -1755,7 +1755,7 @@ static enum bp_result bios_parser_get_firmware_info(
 			case 2:
 			case 3:
 				result = get_firmware_info_v3_2(bp, info);
-                                break;
+			break;
 			case 4:
 				result = get_firmware_info_v3_4(bp, info);
 				break;
@@ -2225,7 +2225,7 @@ static enum bp_result bios_parser_get_disp_connector_caps_info(
 		return BP_RESULT_BADINPUT;
 
 	switch (bp->object_info_tbl.revision.minor) {
-	    case 4:
+	case 4:
 	    default:
 		    object = get_bios_object(bp, object_id);
 
-- 
2.17.1
Re: [PATCH] drm/amd/display: Clean up errors in bios_parser2.c
Posted by Alex Deucher 2 years, 6 months ago 
Applied.  Thanks!

As a follow up patch, care to drop the break statements after a return?

On Tue, Aug 1, 2023 at 11:23 PM Ran Sun <sunran001@208suo.com> wrote:
>
> Fix the following errors reported by checkpatch:
>
> ERROR: switch and case should be at the same indent
> ERROR: code indent should use tabs where possible
>
> Signed-off-by: Ran Sun <sunran001@208suo.com>
> ---
>  .../drm/amd/display/dc/bios/bios_parser2.c    | 32 +++++++++----------
>  1 file changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
> index 540d19efad8f..033ce2638eb2 100644
> --- a/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
> +++ b/drivers/gpu/drm/amd/display/dc/bios/bios_parser2.c
> @@ -772,20 +772,20 @@ static enum bp_result bios_parser_get_device_tag(
>                 return BP_RESULT_BADINPUT;
>
>         switch (bp->object_info_tbl.revision.minor) {
> -           case 4:
> -           default:
> +       case 4:
> +       default:
>                 /* getBiosObject will return MXM object */
> -               object = get_bios_object(bp, connector_object_id);
> +               object = get_bios_object(bp, connector_object_id);
>
>                 if (!object) {
>                         BREAK_TO_DEBUGGER(); /* Invalid object id */
>                         return BP_RESULT_BADINPUT;
>                 }
>
> -               info->acpi_device = 0; /* BIOS no longer provides this */
> -               info->dev_id = device_type_from_device_id(object->device_tag);
> -               break;
> -           case 5:
> +               info->acpi_device = 0; /* BIOS no longer provides this */
> +               info->dev_id = device_type_from_device_id(object->device_tag);
> +               break;
> +       case 5:
>                 object_path_v3 = get_bios_object_from_path_v3(bp, connector_object_id);
>
>                 if (!object_path_v3) {
> @@ -1580,13 +1580,13 @@ static bool bios_parser_is_device_id_supported(
>         uint32_t mask = get_support_mask_for_device_id(id);
>
>         switch (bp->object_info_tbl.revision.minor) {
> -           case 4:
> -           default:
> -               return (le16_to_cpu(bp->object_info_tbl.v1_4->supporteddevices) & mask) != 0;
> -                       break;
> -           case 5:
> -                       return (le16_to_cpu(bp->object_info_tbl.v1_5->supporteddevices) & mask) != 0;
> -                       break;
> +       case 4:
> +       default:
> +               return (le16_to_cpu(bp->object_info_tbl.v1_4->supporteddevices) & mask) != 0;
> +               break;
> +       case 5:
> +               return (le16_to_cpu(bp->object_info_tbl.v1_5->supporteddevices) & mask) != 0;
> +               break;
>         }
>
>         return false;
> @@ -1755,7 +1755,7 @@ static enum bp_result bios_parser_get_firmware_info(
>                         case 2:
>                         case 3:
>                                 result = get_firmware_info_v3_2(bp, info);
> -                                break;
> +                       break;
>                         case 4:
>                                 result = get_firmware_info_v3_4(bp, info);
>                                 break;
> @@ -2225,7 +2225,7 @@ static enum bp_result bios_parser_get_disp_connector_caps_info(
>                 return BP_RESULT_BADINPUT;
>
>         switch (bp->object_info_tbl.revision.minor) {
> -           case 4:
> +       case 4:
>             default:
>                     object = get_bios_object(bp, object_id);
>
> --
> 2.17.1
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.