GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/baytrail/pmutil.c: Constify string arrays 

This reduces the differences between Bay Trail and Braswell.
The resulting binary changes, but it shouldn't matter.

Change-Id: Ic930ab7eee265e86a7cc1095021e3744885f2c25
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43184
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Frans Hendriks <fhendriks@eltan.com>
Reviewed-by: Wim Vervoorn <wvervoorn@eltan.com>
This commit is contained in:
Angel Pons 2020-07-07 18:09:45 +02:00
parent 61dee5c865
commit a5c970d433
1 changed files with 16 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/soc/intel/baytrail/pmutil.c
View File

@ -42,8 +42,7 @@ uint16_t get_pmbase(void)
return pci_read_config16(get_pcu_dev(), ABASE) & 0xfff8; return pci_read_config16(get_pcu_dev(), ABASE) & 0xfff8;
} }
static void print_num_status_bits(int num_bits, uint32_t status, static void print_num_status_bits(int num_bits, uint32_t status, const char *const bit_names[])
const char *bit_names[])
{ {
int i; int i;
@ -62,7 +61,7 @@ static void print_num_status_bits(int num_bits, uint32_t status,
static uint32_t print_smi_status(uint32_t smi_sts) static uint32_t print_smi_status(uint32_t smi_sts)
{ {
static const char *smi_sts_bits[] = { static const char *const smi_sts_bits[] = {
[2] = "BIOS", [2] = "BIOS",
[4] = "SLP_SMI", [4] = "SLP_SMI",
[5] = "APM", [5] = "APM",
@ -147,7 +146,7 @@ static uint16_t reset_pm1_status(void)
static uint16_t print_pm1_status(uint16_t pm1_sts) static uint16_t print_pm1_status(uint16_t pm1_sts)
{ {
static const char *pm1_sts_bits[] = { static const char *const pm1_sts_bits[] = {
[0] = "TMROF", [0] = "TMROF",
[5] = "GBL", [5] = "GBL",
[8] = "PWRBTN", [8] = "PWRBTN",
@ -180,7 +179,7 @@ void enable_pm1(uint16_t events)
static uint32_t print_tco_status(uint32_t tco_sts) static uint32_t print_tco_status(uint32_t tco_sts)
{ {
static const char *tco_sts_bits[] = { static const char *const tco_sts_bits[] = {
[3] = "TIMEOUT", [3] = "TIMEOUT",
[17] = "SECOND_TO", [17] = "SECOND_TO",
}; };
@ -242,7 +241,7 @@ static uint32_t reset_gpe_status(void)
static uint32_t print_gpe_sts(uint32_t gpe_sts) static uint32_t print_gpe_sts(uint32_t gpe_sts)
{ {
static const char *gpe_sts_bits[] = { static const char *const gpe_sts_bits[] = {
[1] = "HOTPLUG", [1] = "HOTPLUG",
[2] = "SWGPE", [2] = "SWGPE",
[3] = "PCIE_WAKE0", [3] = "PCIE_WAKE0",
@ -298,7 +297,7 @@ static uint32_t reset_alt_status(void)
static uint32_t print_alt_sts(uint32_t alt_gpio_smi) static uint32_t print_alt_sts(uint32_t alt_gpio_smi)
{ {
uint32_t alt_gpio_sts; uint32_t alt_gpio_sts;
static const char *alt_gpio_smi_sts_bits[] = { static const char *const alt_gpio_smi_sts_bits[] = {
[0] = "SUS_GPIO_0", [0] = "SUS_GPIO_0",
[1] = "SUS_GPIO_1", [1] = "SUS_GPIO_1",
[2] = "SUS_GPIO_2", [2] = "SUS_GPIO_2",