[PATCH v3 2/3] cmd: ini: Fix build warning
Kever Yang
kever.yang at rock-chips.com
Fri Jul 28 11:47:18 CEST 2023
On 2023/7/22 22:02, Jonas Karlman wrote:
> Building U-Boot with CMD_INI=y result in following build warning:
>
> cmd/ini.c: In function 'memgets':
> include/linux/kernel.h:184:24: warning: comparison of distinct pointer types lacks a cast
> 184 | (void) (&_min1 == &_min2); \
> | ^~
> cmd/ini.c:92:15: note: in expansion of macro 'min'
> 92 | len = min((end - *mem) + newline, num);
> | ^~~
>
> Fix this by adding an int cast to the pointer arithmetic result.
>
> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>
Thanks,
- Kever
> ---
> v3:
> - No changes
>
> v2:
> - New patch
>
> cmd/ini.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/cmd/ini.c b/cmd/ini.c
> index 81dfc4c4e83d..35de2373e602 100644
> --- a/cmd/ini.c
> +++ b/cmd/ini.c
> @@ -89,7 +89,7 @@ static char *memgets(char *str, int num, char **mem, size_t *memsize)
> end = *mem + *memsize;
> newline = 0;
> }
> - len = min((end - *mem) + newline, num);
> + len = min((int)(end - *mem) + newline, num);
> memcpy(str, *mem, len);
> if (len < num)
> str[len] = '\0';
More information about the U-Boot
mailing list