diff mbox series

edid-decode: Fix decoing of DispID physical enclosure bit

Message ID 20200313183924.10820-1-ville.syrjala@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series edid-decode: Fix decoing of DispID physical enclosure bit | expand

Commit Message

Ville Syrjälä March 13, 2020, 6:39 p.m. UTC
From: Ville Syrjälä <ville.syrjala@linux.intel.com>

The interpretation of the physucal enclosure bit is backwards.
Fix it to match the spec.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 parse-displayid-block.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Hans Verkuil March 15, 2020, 7:39 a.m. UTC | #1
On 3/13/20 7:39 PM, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> The interpretation of the physucal enclosure bit is backwards.
> Fix it to match the spec.

Thank you, it's been applied.

Regards,

	Hans

> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
>  parse-displayid-block.cpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/parse-displayid-block.cpp b/parse-displayid-block.cpp
> index e6c8d95dcaa1..b436f343fa63 100644
> --- a/parse-displayid-block.cpp
> +++ b/parse-displayid-block.cpp
> @@ -913,9 +913,9 @@ static void parse_displayid_tiled_display_topology(const unsigned char *x, bool
>  	default: printf("Reserved\n"); break;
>  	}
>  	if (caps & 0x80)
> -		printf("    Tiled display consists of multiple physical display enclosures\n");
> -	else
>  		printf("    Tiled display consists of a single physical display enclosure\n");
> +	else
> +		printf("    Tiled display consists of multiple physical display enclosures\n");
>  	printf("    Num horizontal tiles: %u Num vertical tiles: %u\n",
>  	       num_h_tile + 1, num_v_tile + 1);
>  	printf("    Tile location: %u, %u\n", tile_h_location, tile_v_location);
>
diff mbox series

Patch

diff --git a/parse-displayid-block.cpp b/parse-displayid-block.cpp
index e6c8d95dcaa1..b436f343fa63 100644
--- a/parse-displayid-block.cpp
+++ b/parse-displayid-block.cpp
@@ -913,9 +913,9 @@  static void parse_displayid_tiled_display_topology(const unsigned char *x, bool
 	default: printf("Reserved\n"); break;
 	}
 	if (caps & 0x80)
-		printf("    Tiled display consists of multiple physical display enclosures\n");
-	else
 		printf("    Tiled display consists of a single physical display enclosure\n");
+	else
+		printf("    Tiled display consists of multiple physical display enclosures\n");
 	printf("    Num horizontal tiles: %u Num vertical tiles: %u\n",
 	       num_h_tile + 1, num_v_tile + 1);
 	printf("    Tile location: %u, %u\n", tile_h_location, tile_v_location);