Re: [RFC PATCH 0/15] x86: Remove support for TSC-less and CX8-less CPUs

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

On Fri, Apr 25, 2025, at 09:40, Ingo Molnar wrote:
> * Arnd Bergmann <arnd@arndb.de> wrote:
>
>> >       x86/platform: Remove CONFIG_X86_RDC321X support
>> >       arch/x86, gpio: Remove GPIO_RDC321X support
>> >       arch/x86, watchdog: Remove the RDC321X_WDT watchdog driver
>> >       arch/x86, mfd: Remove MFD_RDC321X support
>> >       x86/reboot: Remove the RDC321X reboot quirk
>> 
>> I'm not sure about the RDC321X bits. Obviously the original
>> 321x/861x/vortex86sx chips are obsolete and can be removed,
>> but the product line is still actively developed by RDC and 
>> DM&P, and I suspect that some of the drivers are still used
>> on 586tsc-class (vortex86dx, vortex86mx) and 686-class
>> (vortex86dx3, vortex86ex) SoCs that do run modern kernels and
>> get updates.
>
> So CONFIG_X86_RDC321X actively selects M486:
>
> +++ b/arch/x86/Kconfig
> 
>  config X86_RDC321X
>         bool "RDC R-321x SoC"
>         depends on X86_32
>         depends on X86_EXTENDED_PLATFORM
>         select M486
>         ^^^^^^^^^^^
>         select X86_REBOOTFIXUPS

Right, when the code got added, it was certainly for that
specific chip, which I think is 486SX compatible.

The 'select M486' here doesn't actually do anything because
Kconfig silently ignores 'select' for 'choice' symbols.

> But indeed the other drivers are not dependent on M486, at least 
> overtly:
>
>   arch/x86, mfd: Remove MFD_RDC321X support
>   arch/x86, watchdog: Remove the RDC321X_WDT watchdog driver
>   arch/x86, gpio: Remove GPIO_RDC321X support
>
> Although the watchdog driver has this indirect dependency:
>
>    drivers/watchdog/Kconfig:       depends on X86_RDC321X || COMPILE_TEST

> Anyway, I've dropped the mfd/watchdog/gpio removal patches, no harm in 
> keeping these drivers.

Thanks. We should still revisit all these separately and see which
ones are used on more modern RDC/Vortex86 chips, as the relation
between the brands isn't well documented.

I found an older lspci output from Xcore86MX/Vortex86MX showing
that is uses an RDC R6021/R6036 bridge instead of R6020/R6030
on the RDC321x:

https://lore.kernel.org/all/4CC80AF3.9040708@croler.net/

The Vortex86DX (585tsc compatible) datasheet in turn lists
an R6021/R6031, which means the driver won't work out of the box,
but it's probably not far off either if someone just adds
the PCI ID.

Clearly nobody has done that so far, which would indicate that
not a lot of people run vortex86 /and/ realize it's related
to rdc321x.

> and I've switched the watchdog driver over to X86_32:
>
>   config RDC321X_WDT
>         tristate "RDC R-321x SoC watchdog"
>         depends on X86_32 || COMPILE_TEST

How about 'CPU_SUP_VORTEX_32 || COMPILE_TEST'?

> There's also no harm in keeping the southbridge reboot quirk I suppose, 
> so I've dropped this as well:
>
>   x86/reboot: Remove the RDC321X reboot quirk

Right. Same thing here: the code probably still works on later
R603x south bridges, but only triggers on the R6030 PCI ID
that is not used on supported chips. Most likely nothing
needs it.

    Arnd