Hi Heinrich,
On Wed, Feb 25, 2026 at 1:19 AM Heinrich Schuchardt
<[email protected]> wrote:
>
> The symbol CONFIG_GIC_600_CLEAR_RDPD does not exist.
> Don't select it.
>
> Signed-off-by: Heinrich Schuchardt <[email protected]>
> ---
> arch/arm/mach-sc5xx/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/mach-sc5xx/Kconfig b/arch/arm/mach-sc5xx/Kconfig
> index ec1d5539dc9..e781603a8cf 100644
> --- a/arch/arm/mach-sc5xx/Kconfig
> +++ b/arch/arm/mach-sc5xx/Kconfig
> @@ -44,7 +44,6 @@ config SC59X_64
> select COMMON_CLK_ADI_SC598
> select GICV3
> select GICV3_SUPPORT_GIC600
> - select GIC_600_CLEAR_RDPD
> select MMC_SDHCI_ADMA_FORCE_32BIT
> select NOP_PHY if PHY
>
> --
> 2.51.0
>
Thanks for this fix. For posterity, the logic originally expected for
"GIC_600_CLEAR_RDPD" is available in arch/arm/lib/gic_64.S:
#if defined(CONFIG_GICV3_SUPPORT_GIC600)
mov w10, #0x0 /* Power on redistributor */
str w10, [x9, GICR_PWRR]
5: ldr w10, [x9, GICR_PWRR] /* Wait until the power on
state is reflected */
tbnz w10, #1, 5b /* If RDPD == 0 then powered on */
#endif
and the symbol was just a leftover from a patch we had made internally
to add the same thing before upstreaming and then dropped.
GICV3_SUPPORT_GIC600 is the correct symbol for us to use here.
Reviewed-by: Greg Malysa <[email protected]>
Thanks,
Greg
