ACK: [PATCH] acpi: dmar: constify a pointer argument in acpi_parse_dev_scope

Alex Hung alex.hung at canonical.com
Wed Sep 22 19:08:09 UTC 2021


On 2021-09-22 4:15 a.m., Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> One of the pointer arguments can be const, clean up a cppcheck
> style warning.
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpi/dmar/dmar.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/acpi/dmar/dmar.c b/src/acpi/dmar/dmar.c
> index 5b06c8c7..91721229 100644
> --- a/src/acpi/dmar/dmar.c
> +++ b/src/acpi/dmar/dmar.c
> @@ -190,7 +190,7 @@ error:
>  }
>  
>  static int acpi_parse_dev_scope(fwts_framework *fw,
> -	uint8_t *start, uint8_t *end, const uint16_t seg)
> +	uint8_t *start, const uint8_t *end, const uint16_t seg)
>  {
>  	while (start < end) {
>  		struct acpi_dev_scope *scope = (struct acpi_dev_scope *)start;
> 


Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list