Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Ville Syrjala <ville.syrjala@linux.intel.com>
To: igt-dev@lists.freedesktop.org
Subject: [PATCH i-g-t 26/27] tools/intel_vbt_decode: Decode block 253 (PRD Table)
Date: Fri,  7 Jun 2024 16:57:57 +0300	[thread overview]
Message-ID: <20240607135758.31421-27-ville.syrjala@linux.intel.com> (raw)
In-Reply-To: <20240607135758.31421-1-ville.syrjala@linux.intel.com>

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Decode VBT block 253 (PRD Table). The layout changes in a funky
way so it looks some guesswork is needed to determine which
way to go :(

Example output from ADL:
Block 253 min size 2 less than block size 66
BDB block 253 (66 bytes, min 2 bytes) - PRD table:
        0000: fd 42 00 10 00 08 00 40  00 08 00 20 00 08 00 00
        0010: 00 40 00 00 00 20 00 00  00 00 00 00 00 00 00 00
        0020: 00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00
        0030: 00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00
        0040: 00 00 00 00 00

        Num entries: 16
        Entry #1:
                Primary display: LFP1 (0x8)
                Secondary display: EFP2 (0x40)
        ...
        Entry #16:
                Primary display: none (0x0)
                Secondary display: none (0x0)

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 tools/intel_vbt_decode.c | 70 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 70 insertions(+)

diff --git a/tools/intel_vbt_decode.c b/tools/intel_vbt_decode.c
index 83c90083e1b9..9d9c96f1c264 100644
--- a/tools/intel_vbt_decode.c
+++ b/tools/intel_vbt_decode.c
@@ -422,6 +422,9 @@ static size_t block_min_size(const struct context *context, int section_id)
 		return sizeof(struct bdb_vswing_preemph);
 	case BDB_GENERIC_DTD:
 		return sizeof(struct bdb_generic_dtd);
+	case BDB_PRD_TABLE:
+		return max(sizeof(struct bdb_prd_table_old),
+			   sizeof(struct bdb_prd_table_new));
 	default:
 		return 0;
 	}
@@ -3494,6 +3497,68 @@ static void dump_generic_dtd(struct context *context,
 	}
 }
 
+static void dump_prd_table_old(struct context *context,
+			       const struct bdb_block *block)
+{
+	const struct bdb_prd_table_old *prd =
+		block_data(block) + block->size - sizeof(*prd);
+	const struct prd_entry_old *list = block_data(block);
+
+	for (int i = 0; i < prd->num_entries; i++) {
+		printf("\tEntry #%d:\n", i + 1);
+		printf("\t\tDisplays attached: %s (0x%x)\n",
+		       child_device_handle(context, list[i].displays_attached),
+		       list[i].displays_attached);
+		printf("\t\tDisplays in pipe A: %s (0x%x)\n",
+		       child_device_handle(context, list[i].display_in_pipe_a),
+		       list[i].display_in_pipe_a);
+		printf("\t\tDisplays in pipe B: %s (0x%x)\n",
+		       child_device_handle(context, list[i].display_in_pipe_b),
+		       list[i].display_in_pipe_b);
+	}
+
+	printf("\tNum entries: %d\n", prd->num_entries);
+}
+
+static void dump_prd_table_new(struct context *context,
+			       const struct bdb_block *block)
+{
+	const struct bdb_prd_table_new *prd = block_data(block);
+	const struct prd_entry_new *list = prd->list;
+
+	printf("\tNum entries: %d\n", prd->num_entries);
+
+	for (int i = 0; i < prd->num_entries; i++) {
+		printf("\tEntry #%d:\n", i + 1);
+		printf("\t\tPrimary display: %s (0x%x)\n",
+		       child_device_handle(context, list[i].primary_display),
+		       list[i].primary_display);
+		printf("\t\tSecondary display: %s (0x%x)\n",
+		       child_device_handle(context, list[i].secondary_display),
+		       list[i].secondary_display);
+	}
+}
+
+static void dump_prd_table(struct context *context,
+			   const struct bdb_block *block)
+{
+	const struct bdb_prd_table_old *old =
+		block_data(block) + block->size - sizeof(*old);
+	const struct bdb_prd_table_new *new =
+		block_data(block);
+	int num_entries_old = (block->size - sizeof(*old)) / sizeof(*old->list);
+	int num_entries_new = (block->size - sizeof(*new)) / sizeof(*new->list);
+
+	/*
+	 * The cutoff seems to be TGL+ w/ GOP rather than a specific
+	 * BDB version number. Just guess based on the actual data.
+	 */
+	if (num_entries_old == old->num_entries)
+		dump_prd_table_old(context, block);
+	else if (num_entries_new == new->num_entries)
+		dump_prd_table_new(context, block);
+}
+
 static int get_panel_type_pnpid(const struct context *context,
 				const char *edid_file)
 {
@@ -3852,6 +3917,11 @@ struct dumper dumpers[] = {
 		.name = "Generic DTD",
 		.dump = dump_generic_dtd,
 	},
+	{
+		.id = BDB_PRD_TABLE,
+		.name = "PRD table",
+		.dump = dump_prd_table,
+	},
 };
 
 static void hex_dump_block(const struct bdb_block *block)
-- 
2.44.2


  parent reply	other threads:[~2024-06-07 13:59 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-07 13:57 [PATCH i-g-t 00/27] tools/intel_vbt_decode: Decode almost everything Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 01/27] tools/intel_vbt_decode: sync intel_vbt_defs.h with kernel commit 80c414772d93 Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 02/27] tools/intel_vbt_decode: sync intel_vbt_defs.h with kernel commit 3e8daf14c47d Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 03/27] tools/intel_vbt_decode: Dump MIPI config for the correct panel Ville Syrjala
2024-06-13  9:13   ` Jani Nikula
2024-06-07 13:57 ` [PATCH i-g-t 04/27] tools/intel_vbt_decode: Decode block 3 (Display Toggle Option) Ville Syrjala
2024-06-13  9:14   ` Jani Nikula
2024-06-13 17:19     ` Ville Syrjälä
2024-06-07 13:57 ` [PATCH i-g-t 05/27] tools/intel_vbt_decode: Decode block 4 (Mode Support List) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 06/27] tools/intel_vbt_decode: Decode block 5 (Generic Mode Table) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 07/27] tools/intel_vbt_decode: Decode blocks 6, 7, 8 (register tables) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 08/27] tools/intel_vbt_decode: Decode block 10 (Mode Removal Table) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 09/27] tools/intel_vbt_decode: Decode block 12 (Driver Persistent Algorithm) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 10/27] tools/intel_vbt_decode: Decode block 15 (Dot Clock Override Table) and block 9 (ALM only) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 11/27] tools/intel_vbt_decode: Decode blocks 16, 29, 31 (Toggle List) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 12/27] tools/intel_vbt_decode: Decode blocks 19, 30, 32 (Display Configuration Removal Table) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 13/27] tools/intel_vbt_decode: Decode block 18 (Driver Rotation) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 14/27] tools/intel_vbt_decode: Decode block 20 (OEM Customizable Modes) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 15/27] tools/intel_vbt_decode: Decode block 21 (EFP List) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 16/27] tools/intel_vbt_decode: Decode block 24 (SDVO LVDS PnP ID) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 17/27] tools/intel_vbt_decode: Decode block 25 (SDVO LVDS PPS) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 18/27] tools/intel_vbt_decode: Decode block 26 (TV Options) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 19/27] tools/intel_vbt_decode: Decode block 28 (EFP DTD) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 20/27] tools/intel_vbt_decode: Decode block 45 (eDP BFI) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 21/27] tools/intel_vbt_decode: Decode block 46 (Chromaticity For Narrow Gamut Panel) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 22/27] tools/intel_vbt_decode: Decode block 51 (Fixed Set Mode Table) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 23/27] tools/intel_vbt_decode: Decode block 55 (RGB Palette Table) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 24/27] tools/intel_vbt_decode: Decode block 57 (Vswing PreEmphasis Table) Ville Syrjala
2024-06-07 13:57 ` [PATCH i-g-t 25/27] tools/intel_vbt_decode: Decode block 58 (Generic DTD Block) Ville Syrjala
2024-06-07 13:57 ` Ville Syrjala [this message]
2024-06-07 13:57 ` [PATCH i-g-t 27/27] tools/intel_vbt_decode: Name a few more VBT blocks Ville Syrjala
2024-06-07 22:08 ` ✓ CI.xeBAT: success for tools/intel_vbt_decode: Decode almost everything Patchwork
2024-06-07 22:13 ` ✓ Fi.CI.BAT: " Patchwork
2024-06-08 12:30 ` ✗ CI.xeFULL: failure " Patchwork
2024-06-11 15:37   ` Kamil Konieczny
2024-06-08 13:08 ` ✗ Fi.CI.IGT: " Patchwork
2024-06-11 15:36   ` Kamil Konieczny
2024-06-13  9:17 ` [PATCH i-g-t 00/27] " Jani Nikula

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240607135758.31421-27-ville.syrjala@linux.intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox