ACK: [PATCH] lib: fwts_devicetree: constify a few function arguments

ivanhu ivan.hu at canonical.com
Thu Sep 21 09:16:47 UTC 2017



On 09/08/2017 05:40 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Code cleanup, constify some function arguments
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/lib/src/fwts_devicetree.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/lib/src/fwts_devicetree.c b/src/lib/src/fwts_devicetree.c
> index 2fea3c8a..9db63de6 100644
> --- a/src/lib/src/fwts_devicetree.c
> +++ b/src/lib/src/fwts_devicetree.c
> @@ -182,7 +182,7 @@ char *hidewhitespace(char *name)
>   
>   int fwts_dt_property_read_u32(
>   	void *fdt,
> -	int offset,
> +	const int offset,
>   	const char *pname,
>   	int *value)
>   {
> @@ -207,7 +207,7 @@ int fwts_dt_property_read_u32(
>   
>   int fwts_dt_property_read_u32_arr(
>   	void *fdt,
> -	int offset,
> +	const int offset,
>   	const char *pname,
>   	int *value,
>   	int *len)
> @@ -236,7 +236,7 @@ int fwts_dt_property_read_u32_arr(
>   
>   int fwts_dt_property_read_u64_arr(
>   	void *fdt,
> -	int offset,
> +	const int offset,
>   	const char *pname,
>   	uint64_t *value,
>   	int *len)
> @@ -261,7 +261,7 @@ int fwts_dt_property_read_u64_arr(
>   int fwts_dt_stringlist_count(
>   	fwts_framework *fw,
>   	const void *fdt,
> -	int nodeoffset,
> +	const int nodeoffset,
>   	const char *property)
>   {
>   	const char *list, *end;
> 


Acked-by: Ivan Hu <ivan.hu at canonical.com>



More information about the fwts-devel mailing list