ACK: [PATCH 22/27] opal: reserv_mem: add in missing const to fix build warnings

Alex Hung alex.hung at canonical.com
Wed Aug 15 18:11:48 UTC 2018


On 2018-08-15 06:11 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Add in missing const, cleans up warnings.
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/opal/reserv_mem.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/opal/reserv_mem.c b/src/opal/reserv_mem.c
> index 4683f73c..b42b959b 100644
> --- a/src/opal/reserv_mem.c
> +++ b/src/opal/reserv_mem.c
> @@ -125,7 +125,7 @@ static int reserv_mem_init(fwts_framework *fw)
>   static int reserv_mem_limits_test(fwts_framework *fw)
>   {
>   	bool ok = true;
> -	char *region_names;
> +	const char *region_names;
>   	const uint64_t *ranges;
>   	reserve_region_t *regions;
>   	int  offset, len, nr_regions, rc, j;
> @@ -145,7 +145,7 @@ static int reserv_mem_limits_test(fwts_framework *fw)
>   				"reserved-names");
>   
>   	/* Check for the reservd-names property */
> -	region_names = (char *)fdt_getprop(fw->fdt, offset,
> +	region_names = (const char *)fdt_getprop(fw->fdt, offset,
>   					"reserved-names", &len);
>   	if (!region_names) {
>   		fwts_failed(fw, LOG_LEVEL_MEDIUM, "DTPropertyMissing",
> 



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



More information about the fwts-devel mailing list