Re: [PATCH] arm64: dts: renesas: r9a07g044: Add missing GICv3 node properties

Hi Prabhakar,

On Tue, Jul 13, 2021 at 10:22 AM Lad, Prabhakar
[off-list ref] wrote:

On Tue, Jul 13, 2021 at 9:08 AM Geert Uytterhoeven [off-list ref] wrote:

quoted

On Mon, Jun 14, 2021 at 2:48 PM Geert Uytterhoeven [off-list ref] wrote:

quoted

On Fri, Jun 11, 2021 at 5:21 PM Lad Prabhakar
[off-list ref] wrote:

quoted

Add the below missing properties into GIC node,
- clocks
- clock-names
- power-domains
- resets

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Queueing pending on[1].

quoted

[1] https://lore.kernel.org/linux-devicetree/
    20210609155108.16590-1-prabhakar.mahadev-lad.rj@bp.renesas.com/

The dependency has been accepted, but this patch needs a respin
for the changed clocks.

Thank you for pointing this out. wrt resets the GIC has two signals
(which I learnt lately when the dependency path was accepted). Earlier
discussion in irc with Sudeep pointed out that there wouldn't be any
use case of having GIC resets in DTSI, so either we drop the resets
property in DT binding doc or correct it.

Let me know your thoughts on this and how we proceed further.

DT Rule #1: DT describes hardware not software policy.

And a possible use case: the RT CPU core may want to reset the AP GIC.

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