From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: jani.nikula@intel.com
Subject: [PATCH i-g-t 8/8] tools/intel_vbt_decode: abstract child devices printing more
Date: Thu, 19 Oct 2017 18:22:59 +0300 [thread overview]
Message-ID: <20171019152259.31615-9-jani.nikula@intel.com> (raw)
In-Reply-To: <20171019152259.31615-1-jani.nikula@intel.com>
Unify the common code for current and legacy blocks.
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
tools/intel_vbt_decode.c | 70 +++++++++++++++++++++---------------------------
1 file changed, 31 insertions(+), 39 deletions(-)
diff --git a/tools/intel_vbt_decode.c b/tools/intel_vbt_decode.c
index ebc65b82bbde..04f733c8cb06 100644
--- a/tools/intel_vbt_decode.c
+++ b/tools/intel_vbt_decode.c
@@ -483,25 +483,12 @@ static void dump_child_device(struct context *context,
}
}
-static void dump_general_definitions(struct context *context,
- const struct bdb_block *block)
+
+static void dump_child_devices(struct context *context, const uint8_t *devices,
+ uint8_t child_dev_num, uint8_t child_dev_size)
{
- const struct bdb_general_definitions *defs = block->data;
struct child_device_config *child;
int i;
- int child_device_num;
-
- printf("\tCRT DDC GMBUS addr: 0x%02x\n", defs->crt_ddc_gmbus_pin);
- printf("\tUse ACPI DPMS CRT power states: %s\n",
- YESNO(defs->dpms_acpi));
- printf("\tSkip CRT detect at boot: %s\n",
- YESNO(defs->skip_boot_crt_detect));
- printf("\tUse DPMS on AIM devices: %s\n", YESNO(defs->dpms_aim));
- printf("\tBoot display type: 0x%02x%02x\n", defs->boot_display[1],
- defs->boot_display[0]);
- printf("\tChild device size: %d\n", defs->child_dev_size);
- child_device_num = (block->size - sizeof(*defs)) /
- defs->child_dev_size;
/*
* Use a temp buffer so dump_child_device() doesn't have to worry about
@@ -510,9 +497,9 @@ static void dump_general_definitions(struct context *context,
*/
child = calloc(1, sizeof(*child));
- for (i = 0; i < child_device_num; i++) {
- memcpy(child, &defs->devices[i * defs->child_dev_size],
- min(sizeof(*child), defs->child_dev_size));
+ for (i = 0; i < child_dev_num; i++) {
+ memcpy(child, devices + i * child_dev_size,
+ min(sizeof(*child), child_dev_size));
dump_child_device(context, child);
}
@@ -520,33 +507,38 @@ static void dump_general_definitions(struct context *context,
free(child);
}
-static void dump_legacy_child_devices(struct context *context,
- const struct bdb_block *block)
+static void dump_general_definitions(struct context *context,
+ const struct bdb_block *block)
{
- const struct bdb_legacy_child_devices *defs = block->data;
- struct child_device_config *child;
- int i;
- int child_device_num;
+ const struct bdb_general_definitions *defs = block->data;
+ int child_dev_num;
+ printf("\tCRT DDC GMBUS addr: 0x%02x\n", defs->crt_ddc_gmbus_pin);
+ printf("\tUse ACPI DPMS CRT power states: %s\n",
+ YESNO(defs->dpms_acpi));
+ printf("\tSkip CRT detect at boot: %s\n",
+ YESNO(defs->skip_boot_crt_detect));
+ printf("\tUse DPMS on AIM devices: %s\n", YESNO(defs->dpms_aim));
+ printf("\tBoot display type: 0x%02x%02x\n", defs->boot_display[1],
+ defs->boot_display[0]);
printf("\tChild device size: %d\n", defs->child_dev_size);
- child_device_num = (block->size - sizeof(*defs)) /
- defs->child_dev_size;
- /*
- * Use a temp buffer so dump_child_device() doesn't have to worry about
- * accessing the struct beyond child_dev_size. The tail, if any, remains
- * initialized to zero.
- */
- child = calloc(1, sizeof(*child));
+ child_dev_num = (block->size - sizeof(*defs)) / defs->child_dev_size;
+ dump_child_devices(context, defs->devices,
+ child_dev_num, defs->child_dev_size);
+}
- for (i = 0; i < child_device_num; i++) {
- memcpy(child, &defs->devices[i * defs->child_dev_size],
- min(sizeof(*child), defs->child_dev_size));
+static void dump_legacy_child_devices(struct context *context,
+ const struct bdb_block *block)
+{
+ const struct bdb_legacy_child_devices *defs = block->data;
+ int child_dev_num;
- dump_child_device(context, child);
- }
+ printf("\tChild device size: %d\n", defs->child_dev_size);
- free(child);
+ child_dev_num = (block->size - sizeof(*defs)) / defs->child_dev_size;
+ dump_child_devices(context, defs->devices,
+ child_dev_num, defs->child_dev_size);
}
static void dump_lvds_options(struct context *context,
--
2.11.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2017-10-19 15:22 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-19 15:22 [PATCH i-g-t 0/8] tools/intel_vbt_decode: refactoring and cleanups Jani Nikula
2017-10-19 15:22 ` [PATCH i-g-t 1/8] tools/intel_vbt_decode: make a copy of child devices before dumping Jani Nikula
2017-10-19 15:22 ` [PATCH i-g-t 2/8] tools/intel_vbt_decode: update dvo port name dumping Jani Nikula
2017-10-19 15:22 ` [PATCH i-g-t 3/8] tools/intel_vbt_decode: use %.*s instead of duplicating a string Jani Nikula
2017-10-19 15:22 ` [PATCH i-g-t 4/8] tools/intel_vbt_decode: abstract DSI bridge type dump Jani Nikula
2017-10-19 15:22 ` [PATCH i-g-t 5/8] tools/intel_vbt_decode: unify child device printing across versions Jani Nikula
2017-10-19 17:24 ` Ville Syrjälä
2017-10-20 10:56 ` Jani Nikula
2017-10-20 11:33 ` Ville Syrjälä
2017-10-20 13:17 ` Jani Nikula
2017-10-20 13:49 ` Ville Syrjälä
2017-10-19 15:22 ` [PATCH i-g-t 6/8] tools/intel_vbt_decode: unify legacy child device block dumping Jani Nikula
2017-10-19 15:22 ` [PATCH i-g-t 7/8] tools/intel_vbt_decode: dump more child device data for version < 152 Jani Nikula
2017-10-19 15:22 ` Jani Nikula [this message]
2017-10-19 16:52 ` ✗ Fi.CI.BAT: failure for tools/intel_vbt_decode: refactoring and cleanups Patchwork
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=20171019152259.31615-9-jani.nikula@intel.com \
--to=jani.nikula@intel.com \
--cc=intel-gfx@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