diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2021-07-12 22:21:36 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-07-14 02:20:22 +0000 |
| commit | c0a6a0efc4be6c09c2181528d34af1deeda81817 (patch) | |
| tree | 4dfcf6b205db1ae300d2ff93c43d80343991241c /src/soc | |
| parent | ceb2fbb9203c66478f1566d7fcfebc5d807bdb32 (diff) | |
soc/amd/picasso,stoneyridge/mca: mark num_banks as constant
Change-Id: I23aa4d36d4e6d4c7ed66800c2e7963c4ed03c393
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56236
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc')
| -rw-r--r-- | src/soc/amd/picasso/mca.c | 4 | ||||
| -rw-r--r-- | src/soc/amd/stoneyridge/mca.c | 4 |
2 files changed, 2 insertions, 6 deletions
diff --git a/src/soc/amd/picasso/mca.c b/src/soc/amd/picasso/mca.c index 65b1916112..3c5dd10c2c 100644 --- a/src/soc/amd/picasso/mca.c +++ b/src/soc/amd/picasso/mca.c @@ -154,9 +154,7 @@ void check_mca(void) { int i; struct mca_bank mci; - unsigned int num_banks; - - num_banks = mca_get_bank_count(); + const unsigned int num_banks = mca_get_bank_count(); for (i = 0 ; i < num_banks ; i++) { mci.sts = rdmsr(MCAX_STATUS_MSR(i)); diff --git a/src/soc/amd/stoneyridge/mca.c b/src/soc/amd/stoneyridge/mca.c index c2ec402bd3..6432ff433e 100644 --- a/src/soc/amd/stoneyridge/mca.c +++ b/src/soc/amd/stoneyridge/mca.c @@ -153,9 +153,7 @@ void check_mca(void) { int i; struct mca_bank mci; - unsigned int num_banks; - - num_banks = mca_get_bank_count(); + const unsigned int num_banks = mca_get_bank_count(); if (is_warm_reset()) { for (i = 0 ; i < num_banks ; i++) { |