Re: [PATCH] ARM: Make sure CPU_ARM940T kernel can be built.

["Arnd Bergmann" <arnd@arndb.de>]

On Thu, Oct 2, 2025, at 17:34, Franz-Josef Haider wrote:
> On 10/1/25 23:52, Arnd Bergmann wrote:
>
> Ok if the support for ARMv4T NOMMU has been removed, then maybe we can 
> leave this as resolved.
>
> It's actually just for a hobby project, nothing important. I just 
> thought these two patches might be helpful to other people who 
> encounter a similar issue on similar cpus, but if the support for 
> ARMv4T NOMMU has been dropped then there is no point maybe.
>
> Here are the patches for 6.2 in any case but it's very much a hacky 
> WIP, and there is some stuff in there that's not needed anymore on top 
> of later commits. https://github.com/krnlyng/linux-ipod/commits/n2g-wip
> I also have a version on top of a recent (~29 SEPT 2025) commit locally 
> if that is interesting too.

Looking at your tree, I find that the ARMv4T NOMMU support is not
even that bad to put back, especially since commit 84fc86360623
("ARM: make ARCH_MULTIPLATFORM user-visible") made it possible to
add a 'depends on !ARCH_MULTIPLATFORM' in the platform specific
code.

I still think that this specific oneline patch is not helpful,
but the other one you sent is technically still a correct fix
and we should either merge that or drop the remnants of the
ARM740T/940T/946E code now that we know it's broken.

Similarly, the later ARM1176 based code that you based your
patches on does look worth cleaning up and merging if there are
still users.

For the specific s5l870x (arm940) SoC support, I see what you
mean with missing cleanups, but more importantly I think that
we will want to drop nommu support from arch/arm/ in a few years
as the remaining users all tend to fall into a similar class
as yours where the developers work on them for the fun of it
but without anyone else depending on it to run on the latest
kernels. Since there is a significant cost in continuing to
test and fix the special cases for nommu support overall, I
would prefer to draw the line at S5L8720 support and not merge
S5L8702 or earlier.

      Arnd