Re: [PATCH] ARM: PXA: remove remnants of PXA93x support

On Tue, Jun 9, 2026 at 4:55 AM Ethan Nelson-Moore
<[email protected]> wrote:
>
> Support for PXA93x chips was removed in commit d711b8a2987a ("ARM: pxa:
> remove pxa93x support"), but some code to handle them remains. Remove
> it.
>
> Discovered while searching for CONFIG_* symbols referenced in code but
> not defined in any Kconfig file.
>
> Signed-off-by: Ethan Nelson-Moore <[email protected]>

Applied for next, thanks!

Kind regards
Uffe


> ---
>  drivers/mmc/host/pxamci.c   |  3 +-
>  include/linux/soc/pxa/cpu.h | 56 -------------------------------------
>  2 files changed, 1 insertion(+), 58 deletions(-)
>
> diff --git a/drivers/mmc/host/pxamci.c b/drivers/mmc/host/pxamci.c
> index b5ea058ed467..f8427f071c00 100644
> --- a/drivers/mmc/host/pxamci.c
> +++ b/drivers/mmc/host/pxamci.c
> @@ -43,8 +43,7 @@
>  #define NR_SG  1
>  #define CLKRT_OFF      (~0)
>
> -#define mmc_has_26MHz()                (cpu_is_pxa300() || cpu_is_pxa310() \
> -                               || cpu_is_pxa935())
> +#define mmc_has_26MHz()                (cpu_is_pxa300() || cpu_is_pxa310())
>
>  struct pxamci_host {
>         struct mmc_host         *mmc;
> diff --git a/include/linux/soc/pxa/cpu.h b/include/linux/soc/pxa/cpu.h
> index 5782450ee45c..38bacdae684f 100644
> --- a/include/linux/soc/pxa/cpu.h
> +++ b/include/linux/soc/pxa/cpu.h
> @@ -46,14 +46,6 @@
>   *  PXA31x     A2      0x69056892      0x2E649013
>   *  PXA32x     B1      0x69056825      0x5E642013
>   *  PXA32x     B2      0x69056826      0x6E642013
> - *
> - *  PXA930     B0      0x69056835      0x5E643013
> - *  PXA930     B1      0x69056837      0x7E643013
> - *  PXA930     B2      0x69056838      0x8E643013
> - *
> - *  PXA935     A0      0x56056931      0x1E653013
> - *  PXA935     B0      0x56056936      0x6E653013
> - *  PXA935     B1      0x56056938      0x8E653013
>   */
>  #ifdef CONFIG_PXA25x
>  #define __cpu_is_pxa210(id)                            \
> @@ -126,26 +118,6 @@
>  #define __cpu_is_pxa320(id)    (0)
>  #endif
>
> -#ifdef CONFIG_CPU_PXA930
> -#define __cpu_is_pxa930(id)                            \
> -       ({                                              \
> -               unsigned int _id = (id) >> 4 & 0xfff;   \
> -               _id == 0x683;                           \
> -        })
> -#else
> -#define __cpu_is_pxa930(id)    (0)
> -#endif
> -
> -#ifdef CONFIG_CPU_PXA935
> -#define __cpu_is_pxa935(id)                            \
> -       ({                                              \
> -               unsigned int _id = (id) >> 4 & 0xfff;   \
> -               _id == 0x693;                           \
> -        })
> -#else
> -#define __cpu_is_pxa935(id)    (0)
> -#endif
> -
>  #define cpu_is_pxa210()                                        \
>         ({                                              \
>                 __cpu_is_pxa210(read_cpuid_id());       \
> @@ -186,18 +158,6 @@
>                 __cpu_is_pxa320(read_cpuid_id());       \
>          })
>
> -#define cpu_is_pxa930()                                        \
> -       ({                                              \
> -               __cpu_is_pxa930(read_cpuid_id());       \
> -        })
> -
> -#define cpu_is_pxa935()                                        \
> -       ({                                              \
> -               __cpu_is_pxa935(read_cpuid_id());       \
> -        })
> -
> -
> -
>  /*
>   * CPUID Core Generation Bit
>   * <= 0x2 for pxa21x/pxa25x/pxa26x/pxa27x
> @@ -218,22 +178,11 @@
>                 __cpu_is_pxa300(id)                     \
>                         || __cpu_is_pxa310(id)          \
>                         || __cpu_is_pxa320(id)          \
> -                       || __cpu_is_pxa93x(id);         \
>          })
>  #else
>  #define __cpu_is_pxa3xx(id)    (0)
>  #endif
>
> -#if defined(CONFIG_CPU_PXA930) || defined(CONFIG_CPU_PXA935)
> -#define __cpu_is_pxa93x(id)                            \
> -       ({                                              \
> -               __cpu_is_pxa930(id)                     \
> -                       || __cpu_is_pxa935(id);         \
> -        })
> -#else
> -#define __cpu_is_pxa93x(id)    (0)
> -#endif
> -
>  #define cpu_is_pxa2xx()                                        \
>         ({                                              \
>                 __cpu_is_pxa2xx(read_cpuid_id());       \
> @@ -244,9 +193,4 @@
>                 __cpu_is_pxa3xx(read_cpuid_id());       \
>          })
>
> -#define cpu_is_pxa93x()                                        \
> -       ({                                              \
> -               __cpu_is_pxa93x(read_cpuid_id());       \
> -        })
> -
>  #endif
> --
> 2.43.0
>