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

acpi/acpigen: Constify CST functions' pointers

Browse Source 
The `acpigen_write_CST_package` and `acpigen_write_CST_package_entry`
functions don't modify the provided C-state information. So, make the
pointer parameters read-only to enforce this. Also constify arguments
where possible.

Change-Id: I9e18d82ee6c16e4435b8fad6d467e58c33194cf4
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/58391
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
This commit is contained in:
Angel Pons
2021-10-17 12:59:43 +02:00
committed by Felix Held
parent 14643b33b0
commit d2794cea12
6 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/acpi/acpigen.c
View File

@@ -994,7 +994,7 @@ void acpigen_write_PSD_package(u32 domain, u32 numprocs, PSD_coord coordtype)
acpigen_pop_len();
}
void acpigen_write_CST_package_entry(acpi_cstate_t *cstate)
void acpigen_write_CST_package_entry(const acpi_cstate_t *cstate)
{
acpigen_write_package(4);
acpigen_write_register_resource(&cstate->resource);
@@ -1004,7 +1004,7 @@ void acpigen_write_CST_package_entry(acpi_cstate_t *cstate)
acpigen_pop_len();
}
void acpigen_write_CST_package(acpi_cstate_t *cstate, int nentries)
void acpigen_write_CST_package(const acpi_cstate_t *cstate, int nentries)
{
int i;
acpigen_write_name("_CST");