sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/mediatek/mt8188: Reduce lastbus configuration size by 1280 bytes

Browse Source 
Original lastbus configuration consumes constant memory size by
allocating 16 and 8 members arrays and the utilization is bad. Refactor
the lastbus structs to save memory usage.

BRANCH=none
BUG=none
TEST=bootblock.raw.bin size is reduced from 60328 bytes to 59048 bytes.

Change-Id: I07ff9ff7c75f03219e1792b92b62814293ef43fe
Signed-off-by: Yidi Lin <yidilin@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74061
Reviewed-by: Yidi Lin <yidilin@google.com>
Reviewed-by: Rex-BC Chen <rex-bc.chen@mediatek.com>
Reviewed-by: Yu-Ping Wu <yupingso@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Yidi Lin
2023-03-28 10:47:27 +08:00
committed by Rex-BC Chen
parent 47a9797100
commit 9fbdb2b192
2 changed files with 77 additions and 71 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/soc/mediatek/common/include/soc/lastbus_v2.h
View File

@ -3,8 +3,6 @@
#ifndef SOC_MEDIATEK_COMMON_LASTBUS_V2_H #ifndef SOC_MEDIATEK_COMMON_LASTBUS_V2_H
#define SOC_MEDIATEK_COMMON_LASTBUS_V2_H #define SOC_MEDIATEK_COMMON_LASTBUS_V2_H
#define NR_MAX_LASTBUS_IDLE_MASK 8
#define NR_MAX_LASTBUS_MONITOR 16
#define TIMEOUT_THRES_SHIFT 16 #define TIMEOUT_THRES_SHIFT 16
#define TIMEOUT_TYPE_SHIFT 1 #define TIMEOUT_TYPE_SHIFT 1
#define LASTBUS_TIMEOUT_CLR 0x0200 #define LASTBUS_TIMEOUT_CLR 0x0200
@ -23,7 +21,7 @@ struct lastbus_monitor {
size_t num_ports; size_t num_ports;
u16 bus_freq_mhz; u16 bus_freq_mhz;
size_t num_idle_mask; size_t num_idle_mask;
struct lastbus_idle_mask idle_masks[NR_MAX_LASTBUS_IDLE_MASK]; const struct lastbus_idle_mask *idle_masks;
}; };
struct lastbus_config { struct lastbus_config {
@ -31,7 +29,7 @@ struct lastbus_config {
unsigned int timeout_ms; unsigned int timeout_ms;
unsigned int timeout_type; unsigned int timeout_type;
unsigned int num_used_monitors; unsigned int num_used_monitors;
struct lastbus_monitor monitors[NR_MAX_LASTBUS_MONITOR]; const struct lastbus_monitor *monitors;
}; };
void lastbus_init(void); void lastbus_init(void);

142
src/soc/mediatek/mt8188/lastbus.c
View File

@ -5,75 +5,83 @@
#include <soc/addressmap.h> #include <soc/addressmap.h>
#include <soc/lastbus_v2.h> #include <soc/lastbus_v2.h>
static const struct lastbus_idle_mask infra_ao_mask[] = {
{
.reg_offset = 0x04,
.reg_value = 0x2,
},
{
.reg_offset = 0x08,
.reg_value = 0x10000,
},
};
static const struct lastbus_idle_mask peri_ao_mask[] = {
{
.reg_offset = 0x04,
.reg_value = 0x20000,
},
};
static const struct lastbus_idle_mask fmem_ao_mask[] = {
{
.reg_offset = 0x14,
.reg_value = 0x204,
},
};
static const struct lastbus_monitor monitors[] = {
{
.name = "debug_ctrl_ao_INFRA_AO",
.base = INFRA_AO_DBUG_BASE,
.num_ports = 34,
.num_idle_mask = ARRAY_SIZE(infra_ao_mask),
.idle_masks = infra_ao_mask,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_INFRA2_AO",
.base = INFRA2_AO_DBUG_BASE,
.num_ports = 9,
.num_idle_mask = 0,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_PERI_AO",
.base = PERI_AO_BASE,
.num_ports = 25,
.num_idle_mask = ARRAY_SIZE(peri_ao_mask),
.idle_masks = peri_ao_mask,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_PERI_AO2",
.base = PERI_AO2_BASE,
.num_ports = 20,
.num_idle_mask = 0,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_PERI_PAR_AO",
.base = PERI_PAR_AO_BASE,
.num_ports = 18,
.num_idle_mask = 0,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_FMEM_AO",
.base = FMEM_AO_BASE,
.num_ports = 28,
.num_idle_mask = ARRAY_SIZE(fmem_ao_mask),
.idle_masks = fmem_ao_mask,
.bus_freq_mhz = 78,
},
};
const struct lastbus_config lastbus_cfg = { const struct lastbus_config lastbus_cfg = {
.latch_platform = "MT8188", .latch_platform = "MT8188",
.timeout_ms = 200, .timeout_ms = 200,
.timeout_type = 0, .timeout_type = 0,
.num_used_monitors = 6, .num_used_monitors = ARRAY_SIZE(monitors),
.monitors = { .monitors = monitors,
{
.name = "debug_ctrl_ao_INFRA_AO",
.base = INFRA_AO_DBUG_BASE,
.num_ports = 34,
.num_idle_mask = 2,
.idle_masks = {
{
.reg_offset = 0x04,
.reg_value = 0x2,
},
{
.reg_offset = 0x08,
.reg_value = 0x10000,
},
},
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_INFRA2_AO",
.base = INFRA2_AO_DBUG_BASE,
.num_ports = 9,
.num_idle_mask = 0,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_PERI_AO",
.base = PERI_AO_BASE,
.num_ports = 25,
.num_idle_mask = 1,
.idle_masks = {
{
.reg_offset = 0x04,
.reg_value = 0x20000,
},
},
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_PERI_AO2",
.base = PERI_AO2_BASE,
.num_ports = 20,
.num_idle_mask = 0,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_PERI_PAR_AO",
.base = PERI_PAR_AO_BASE,
.num_ports = 18,
.num_idle_mask = 0,
.bus_freq_mhz = 78,
},
{
.name = "debug_ctrl_ao_FMEM_AO",
.base = FMEM_AO_BASE,
.num_ports = 28,
.num_idle_mask = 1,
.idle_masks = {
{
.reg_offset = 0x14,
.reg_value = 0x204,
},
},
.bus_freq_mhz = 78,
},
},
}; };