diff options
| author | Yidi Lin <yidilin@chromium.org> | 2024-11-02 11:10:26 +0800 |
|---|---|---|
| committer | Yu-Ping Wu <yupingso@google.com> | 2024-11-07 09:38:31 +0000 |
| commit | d9b0f5a5779ab040599113601a9971ceb36c399b (patch) | |
| tree | 3b62dafab66b1f214d7e667ba34f5ce95f507dfc /src/soc/mediatek/common/tracker.c | |
| parent | 6efc32b6c610bcbec4125a3219a1e23d3a8a34e1 (diff) | |
soc/mediatek/common: Refactor `struct tracker`
Rather than using a static array size for the `offset` variable, use a
pointer named `offsets` that points to a dynamically allocated array. A
separate variable called `offset_size` stores the size of this array.
TEST=emerge-corsola coreboot && emerge-geralt coreboot
Change-Id: I4b89c27fd693ee08e670c1a9ab4cbdbec220bee7
Signed-off-by: Yidi Lin <yidilin@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84958
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Yu-Ping Wu <yupingso@google.com>
Diffstat (limited to 'src/soc/mediatek/common/tracker.c')
| -rw-r--r-- | src/soc/mediatek/common/tracker.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/mediatek/common/tracker.c b/src/soc/mediatek/common/tracker.c index 3485449b6b..1341d5a134 100644 --- a/src/soc/mediatek/common/tracker.c +++ b/src/soc/mediatek/common/tracker.c @@ -25,10 +25,10 @@ static void tracker_dump_data(void) continue; printk(BIOS_INFO, "**Dump %s debug register start**\n", tra->str); - for (k = 0; k < 2; k++) { + for (k = 0; k < tra->offsets_size; k++) { size = 2 * tra->entry; for (i = 0; i < size; i++) { - reg = tra->base_addr + tra->offset[k] + i * 4; + reg = tra->base_addr + tra->offsets[k] + i * 4; printk(BIOS_INFO, "%#lx:%#x,", reg, read32((void *)reg)); if (i % 4 == 3 || i == size - 1) |