Hi Rob,
On Tue, Sep 17, 2019 at 2:12 PM Rob Herring <robh+dt@xxxxxxxxxx> wrote:
> On Mon, Sep 16, 2019 at 10:35 AM Simon Horman
> <horms+renesas@xxxxxxxxxxxx> wrote:
> > +examples:
> > + - |
> > + #include <dt-bindings/interrupt-controller/irq.h>
> > +
> > + bsc: bus@fec10000 {
> > + compatible = "simple-pm-bus";
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + ranges = <0 0 0x20000000>;
> > + reg = <0xfec10000 0x400>;
>
> If you have reg, then it shouldn't be "simple-pm-bus" unless you can
> function without accessing the regs.
Typically the registers are set up by the boot loader.
But this is actually the example for renesas,bsc.txt.
Perhaps agnoc@0 in arch/arm64/boot/dts/qcom/msm8996.dtsi would
be more appropriate?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
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
