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
next prev 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