Re: [PATCH] ARM: dts: renesas: Add boot phase tags marking to Renesas RZ/A1

[Geert Uytterhoeven <geert@linux-m68k.org>]

Hi Marek,

On Wed, 6 Aug 2025 at 17:06, Marek Vasut <marek.vasut@mailbox.org> wrote:
> On 8/6/25 11:11 AM, Geert Uytterhoeven wrote:
> >> @@ -639,6 +643,7 @@ ostm0: timer@fcfec000 {
> >>                          interrupts = <GIC_SPI 102 IRQ_TYPE_EDGE_RISING>;
> >>                          clocks = <&mstp5_clks R7S72100_CLK_OSTM0>;
> >>                          power-domains = <&cpg_clocks>;
> >> +                       bootph-all;
> >
> > Please move this to the board-specific .dts files, where this node
> > is enabled.
> Shouldn't we rather enable OSTM0 by default , it is enabled in all RZA1
> upstream board DTs it seems.

Perhaps.  We used to have issues with enabling all timers by default,
as some may be used by (secure) firmware or another OS.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

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