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

ivanhu ivan.hu at canonical.com
Thu Aug 16 09:19:35 UTC 2018



On 08/15/2018 09:11 PM, 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: Ivan Hu <ivan.hu at canonical.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20180816/afda1448/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20180816/afda1448/attachment-0001.sig>


More information about the fwts-devel mailing list