From: Ville Syrjala <ville.syrjala@linux.intel.com>
To: igt-dev@lists.freedesktop.org
Subject: [PATCH i-g-t 12/27] tools/intel_vbt_decode: Decode blocks 19, 30, 32 (Display Configuration Removal Table)
Date: Fri, 7 Jun 2024 16:57:43 +0300 [thread overview]
Message-ID: <20240607135758.31421-13-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 blocks 19,30,32 (Display Configuration Removal Table).
There are three variants: pre-IVB/IVB/HSW+ eahc of which define
the device handle bitmasks in slightly different ways.
Unexciting block 32 example output from HSW:
BDB block 32 (32 bytes, min 2 bytes) - Display removal table (HSW+):
0000: 20 20 00 0f 02 00 00 00 00 00 00 00 00 00 00 00
0010: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0020: 00 00 00
Num entries: 15
Entry size: 2
Entry #1:
Display select: none (0x0000)
...
Entry #15:
Display select: none (0x0000)
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
tools/intel_vbt_decode.c | 93 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 93 insertions(+)
diff --git a/tools/intel_vbt_decode.c b/tools/intel_vbt_decode.c
index bff6177e27c6..38a7343b0c3b 100644
--- a/tools/intel_vbt_decode.c
+++ b/tools/intel_vbt_decode.c
@@ -342,6 +342,8 @@ static size_t block_min_size(const struct context *context, int section_id)
return sizeof(struct bdb_dot_clock_override);
case BDB_DISPLAY_SELECT_OLD:
return sizeof(struct bdb_display_select_old);
+ case BDB_DISPLAY_REMOVE_OLD:
+ return sizeof(struct bdb_display_remove_old);
case BDB_SDVO_LVDS_OPTIONS:
return sizeof(struct bdb_sdvo_lvds_options);
case BDB_SDVO_LVDS_DTD:
@@ -350,8 +352,12 @@ static size_t block_min_size(const struct context *context, int section_id)
return sizeof(struct bdb_edp);
case BDB_DISPLAY_SELECT_IVB:
return sizeof(struct bdb_display_select_ivb);
+ case BDB_DISPLAY_REMOVE_IVB:
+ return sizeof(struct bdb_display_remove_ivb);
case BDB_DISPLAY_SELECT_HSW:
return sizeof(struct bdb_display_select_hsw);
+ case BDB_DISPLAY_REMOVE_HSW:
+ return sizeof(struct bdb_display_remove_hsw);
case BDB_LFP_OPTIONS:
return sizeof(struct bdb_lfp_options);
case BDB_LFP_DATA_PTRS:
@@ -2133,6 +2139,78 @@ static void dump_display_select_hsw(struct context *context,
}
}
+static void dump_display_remove_old(struct context *context,
+ const struct bdb_block *block)
+{
+ const struct bdb_display_remove_old *r = block_data(block);
+
+ printf("\tNum entries: %d\n", r->num_entries);
+ printf("\tEntry size: %d\n\n", r->entry_size);
+
+ if (sizeof(r->table[0]) != r->entry_size) {
+ printf("\t\tstruct doesn't match (expected %zu, got %u), skipping\n",
+ sizeof(r->table[0]), r->entry_size);
+ return;
+ }
+
+ for (int i = 0 ; i < r->num_entries; i++) {
+ printf("\tEntry #%d:\n", i + 1);
+
+ printf("\t\t\tDisplay select pipe A: %s (0x%02x)\n",
+ child_device_handle(context, r->table[i].display_select_pipe_a),
+ r->table[i].display_select_pipe_a);
+ printf("\t\t\tDisplay select pipe B: %s (0x%02x)\n",
+ child_device_handle(context, r->table[i].display_select_pipe_b),
+ r->table[i].display_select_pipe_b);
+ }
+}
+
+static void dump_display_remove_ivb(struct context *context,
+ const struct bdb_block *block)
+{
+ const struct bdb_display_remove_ivb *r = block_data(block);
+
+ printf("\tNum entries: %d\n", r->num_entries);
+ printf("\tEntry size: %d\n\n", r->entry_size);
+
+ if (sizeof(r->table[0]) != r->entry_size) {
+ printf("\t\tstruct doesn't match (expected %zu, got %u), skipping\n",
+ sizeof(r->table[0]), r->entry_size);
+ return;
+ }
+
+ for (int i = 0 ; i < r->num_entries; i++) {
+ printf("\tEntry #%d:\n", i + 1);
+
+ printf("\t\t\tDisplay select: %s (0x%02x)\n",
+ child_device_handle(context, r->table[i].display_select),
+ r->table[i].display_select);
+ }
+}
+
+static void dump_display_remove_hsw(struct context *context,
+ const struct bdb_block *block)
+{
+ const struct bdb_display_remove_hsw *r = block_data(block);
+
+ printf("\tNum entries: %d\n", r->num_entries);
+ printf("\tEntry size: %d\n\n", r->entry_size);
+
+ if (sizeof(r->table[0]) != r->entry_size) {
+ printf("\t\tstruct doesn't match (expected %zu, got %u), skipping\n",
+ sizeof(r->table[0]), r->entry_size);
+ return;
+ }
+
+ for (int i = 0 ; i < r->num_entries; i++) {
+ printf("\tEntry #%d:\n", i + 1);
+
+ printf("\t\t\tDisplay select: %s (0x%04x)\n",
+ child_device_handle(context, r->table[i].display_select),
+ r->table[i].display_select);
+ }
+}
+
static void dump_edp(struct context *context,
const struct bdb_block *block)
{
@@ -3241,6 +3319,11 @@ struct dumper dumpers[] = {
.name = "Toggle list block (pre-IVB)",
.dump = dump_display_select_old,
},
+ {
+ .id = BDB_DISPLAY_REMOVE_OLD,
+ .name = "Display remove (pre-IVB)",
+ .dump = dump_display_remove_old,
+ },
{
.id = BDB_SDVO_LVDS_OPTIONS,
.name = "SDVO LVDS options block",
@@ -3261,11 +3344,21 @@ struct dumper dumpers[] = {
.name = "Display toggle list (IVB)",
.dump = dump_display_select_ivb,
},
+ {
+ .id = BDB_DISPLAY_REMOVE_IVB,
+ .name = "Display removal table (IVB)",
+ .dump = dump_display_remove_ivb,
+ },
{
.id = BDB_DISPLAY_SELECT_HSW,
.name = "Display toggle list (HSW+)",
.dump = dump_display_select_hsw,
},
+ {
+ .id = BDB_DISPLAY_REMOVE_HSW,
+ .name = "Display removal table (HSW+)",
+ .dump = dump_display_remove_hsw,
+ },
{
.id = BDB_LFP_OPTIONS,
.name = "LFP options block",
--
2.44.2
next prev parent reply other threads:[~2024-06-07 13:58 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 ` Ville Syrjala [this message]
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 ` [PATCH i-g-t 26/27] tools/intel_vbt_decode: Decode block 253 (PRD Table) Ville Syrjala
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-13-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