Hi Mike, On Tue, Oct 27, 2020 at 12:31 PM Mike Rapoport <rppt@xxxxxxxxxx> wrote: > From: Mike Rapoport <rppt@xxxxxxxxxxxxx> > > The pg_data_t node structures and their initialization currently depends on > !CONFIG_SINGLE_MEMORY_CHUNK. Since they are required only for DISCONTIGMEM > make this dependency explicit and replace usage of > CONFIG_SINGLE_MEMORY_CHUNK with CONFIG_DISCONTIGMEM where appropriate. > > The CONFIG_SINGLE_MEMORY_CHUNK was implicitly disabled on the ColdFire MMU > variant, although it always presumed a single memory bank. As there is no > actual need for DISCONTIGMEM in this case, make sure that ColdFire MMU > systems set CONFIG_SINGLE_MEMORY_CHUNK to 'y'. > > Signed-off-by: Mike Rapoport <rppt@xxxxxxxxxxxxx> Thanks for your patch! > --- > arch/m68k/Kconfig.cpu | 6 +++--- > arch/m68k/include/asm/page_mm.h | 2 +- > arch/m68k/mm/init.c | 4 ++-- > 3 files changed, 6 insertions(+), 6 deletions(-) Is there any specific reason you didn't convert the checks for CONFIG_SINGLE_MEMORY_CHUNK in arch/m68k/kernel/setup_mm.c and arch/m68k/include/asm/virtconvert.h? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds