On Sun, 22 Sept 2024 at 02:32, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
>
> kernel/resource.c: In function ‘gfr_start’:
> ./include/linux/minmax.h:93:30: error: conversion from ‘long long
> unsigned int’ to ‘resource_size_t’ {aka ‘unsigned int’} changes value
> from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
>
> Due to
>
> #define PHYSMEM_END (-1ULL)
>
> not being correct on 32-bit without LPAE.
Hmm. Can you check if making it be
#define PHYSMEM_END ((phys_addr_t)-1)
fixes things for you?
That said, it would probably be even better if we got rid of these
games entirely, and m68k just defined MAX_PHYSMEM_BITS instead. Maybe
as a config option, since I assume it's going to be either 24 or 32
depending on CPU (or are there other choices? I used to know the old
m68k, but...)
Linus
