Re: [PATCH v8 04/07] arm64: renesas: r8a7795 dtsi: Add all SCIF nodes

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

Hi Magnus,

On Thu, Sep 3, 2015 at 10:06 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
>> Practically, I need a simple way to identify the clock used by the GIC.
>> Using the "intc-sys"/"intc-ap" name looked like a good idea.
>> While it's always MSTP408, MSTP408 is used for other purposes on some SoCs.
>>
>> Let's give DT scanning a try...
>
> Uhm, sounds a bit overly complicated to me. This has probably been
> discussed wildly before, but please remind me:
>
> What is wrong again with being standard and adding "clocks" and
> "clock-names" to the GIC node?

That needs to be done anyway (still have to dive into GIC architecture
and come up with bindings the ARM people like).

But it's not sufficient as:
  - The GIC driver is not PM Runtime aware. It doesn't even use a platform
    device,
  - The GIC is instantiated from IRQCHIP_DECLARE(), which is way before
    clocks are ready.

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