* Re: [PATCH 2/9] ARM: SPMP8000: Add machine base files [not found] ` <20111016205221.GL1459@game.jcrosoft.org> @ 2011-10-17 11:44 ` Arnd Bergmann 0 siblings, 0 replies; 3+ messages in thread From: Arnd Bergmann @ 2011-10-17 11:44 UTC (permalink / raw) To: Jean-Christophe PLAGNIOL-VILLARD, devicetree-discuss Cc: linux-arm-kernel, Zoltan Devai On Sunday 16 October 2011, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 22:59 Sun 16 Oct , Arnd Bergmann wrote: > > On Sunday 16 October 2011, Zoltan Devai wrote: > >> BTW, the static mappings are only needed to be able to set up the > >> clk stuff in init_early. That in turn is only needed for the timer code to > >> determinate its input clock. > >> If I would hard-code the timer clock rate in its driver (the bootloader is > >> very unlikely to change), then I could init the clk driver from > >> machine_init and get rid of these mappings. > >> How about that ? > > > > Maybe you can just pass the clock rate in the device tree then. If the boot > > loader changes it, it can still pass the correct rate and you don't > > need the early mappings for functional reasons any more. > > please don't update the DT from the bootloader, I'll recommand you to limit > your bootloader to just update the strict minimun (same as ATAG) and touch > noting else Why? Do you think it's better to hardcode it and have no way to update it? Note that the more likely scenario even with clock rate in the device tree would be that the value is hardcoded in the board's file and would not be touched by the boot loader. Only if the boot loader actually changes the physical clock rate (which Zoltan described as very unlikely), it would also have to go the full length and update the device tree as well. Arnd ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <1319040118-29773-1-git-send-email-zoss@devai.org>]
* [PATCH v2 4/5] ARM: SPMP8000: Add SPMP8000 SoC and Letcool board dts descriptions [not found] ` <1319040118-29773-1-git-send-email-zoss@devai.org> @ 2011-10-19 16:01 ` Zoltan Devai 2011-10-24 12:47 ` Rob Herring 0 siblings, 1 reply; 3+ messages in thread From: Zoltan Devai @ 2011-10-19 16:01 UTC (permalink / raw) To: linux-arm-kernel Cc: Grant Likely, devicetree-discuss, Russell King, Arnd Bergmann This adds the DT description of the SPMP8000 SoC, the Letcool N350JP board and documentation of required bindings Cc: Grant Likely <grant.likely@secretlab.ca> CC: devicetree-discuss@lists.ozlabs.org Signed-off-by: Zoltan Devai <zoss@devai.org> --- Documentation/devicetree/bindings/arm/spmp8000.txt | 25 +++++ arch/arm/boot/dts/spmp8000-letcool.dts | 20 ++++ arch/arm/boot/dts/spmp8000.dtsi | 93 ++++++++++++++++++++ 3 files changed, 138 insertions(+), 0 deletions(-) create mode 100644 Documentation/devicetree/bindings/arm/spmp8000.txt create mode 100644 arch/arm/boot/dts/spmp8000-letcool.dts create mode 100644 arch/arm/boot/dts/spmp8000.dtsi diff --git a/Documentation/devicetree/bindings/arm/spmp8000.txt b/Documentation/devicetree/bindings/arm/spmp8000.txt new file mode 100644 index 0000000..c392b09 --- /dev/null +++ b/Documentation/devicetree/bindings/arm/spmp8000.txt @@ -0,0 +1,25 @@ +Sunplus SPMP8000 device tree bindings +===================================== + +Required root node properties: + - compatible: + - "gameware,letcool" : Letcool N350JO handheld game console board + - "sunplus,spmp8000" : A board based on the SPMP8000 SoC + +Timer required properties: + - compatible = "sunplus,spmp8000-timer" + - interrupt-parent : The interrupt controller node this timer belongs to + - interrupts : IRQs of the timer + - clock-freq : The frequency in HZ of the timer. + - reg : The register bank for the timer. + +Watchdog required properties: + - compatible = "sunplus,spmp8000-watchdof" + - interrupt-parent : The interrupt controller node this timer belongs to + - interrupts : IRQs of the watchdog + - reg : The register bank for the watchdog controller + +SCU required properties: + - compatible: "sunplus,spmp8000-scua" or "sunplus,spmp8000-scuab" or + "sunplus,spmp8000-scuc" + - reg: The register bank of the System Control Unit diff --git a/arch/arm/boot/dts/spmp8000-letcool.dts b/arch/arm/boot/dts/spmp8000-letcool.dts new file mode 100644 index 0000000..424f2aa --- /dev/null +++ b/arch/arm/boot/dts/spmp8000-letcool.dts @@ -0,0 +1,20 @@ +/dts-v1/; +/include/ "spmp8000.dtsi" +/ { + model = "Letcool N350JP handheld game console"; + compatible = "gameware,letcool", "sunplus,spmp8000"; + + memory { + reg = <0x00000000 0x02000000>; + }; + + chosen { + linux,stdout-path = &uart0; + }; + + armapb { + timer@90000000 { + clock-freq = <15187500>; + }; + }; +}; diff --git a/arch/arm/boot/dts/spmp8000.dtsi b/arch/arm/boot/dts/spmp8000.dtsi new file mode 100644 index 0000000..15b25fa --- /dev/null +++ b/arch/arm/boot/dts/spmp8000.dtsi @@ -0,0 +1,93 @@ +/ { + compatible = "sunplus,spmp8000"; + #address-cells = <1>; + #size-cells = <1>; + + armapb { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0x90000000 0x10000>; + + timer@90000000 { + compatible = "sunplus,spmp8000-timer"; + reg = <0x0 0x1000>; + interrupt-parent = <&vic0>; + interrupts = <7 8 9>; + }; + + watchdog@90001000 { + compatible = "sunplus,spmp8000-wdt"; + reg = <0x1000 0x1000>; + interrupt-parent = <&vic0>; + interrupts = <10 11 12>; + }; + + scu-b@90005000 { + compatible = "sunplus,spmp8000-scub"; + reg = <0x5000 0x1000>; + }; + }; + + armahb { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0x90010000 0x20000>; + + vic0: interrupt-controller@90010000 { + compatible = "arm,pl192"; + interrupt-controller; + #interrupt-cells = <1>; + reg = <0x0 0x1000>; + }; + + vic1: interrupt-controller@900020000 { + compatible = "arm,pl192"; + interrupt-controller; + #interrupt-cells = <1>; + reg = <0x10000 0x1000>; + }; + }; + + plat { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0x92000000 0x1000000>; + + scu-c@92005000 { + compatible = "sunplus,spmp8000-scuc"; + reg = <0x5000 0x1000>; + }; + + plat-apb { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0xB00000 0x10000>; + + uart0: uart-c0@92B04000 { + compatible = "ns16550a"; + reg = <0x4000 0x1000>; + clock-frequency = <2076923>; + interrupt-parent = <&vic1>; + interrupts = <24>; + current-speed = <115200>; + reg-shift = <2>; + }; + }; + }; + + axi { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0x93000000 0x20000>; + + scu-a@93007000 { + compatible = "sunplus,spmp8000-scua"; + reg = <0x7000 0x1000>; + }; + }; +}; -- 1.7.4.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2 4/5] ARM: SPMP8000: Add SPMP8000 SoC and Letcool board dts descriptions 2011-10-19 16:01 ` [PATCH v2 4/5] ARM: SPMP8000: Add SPMP8000 SoC and Letcool board dts descriptions Zoltan Devai @ 2011-10-24 12:47 ` Rob Herring 0 siblings, 0 replies; 3+ messages in thread From: Rob Herring @ 2011-10-24 12:47 UTC (permalink / raw) To: Zoltan Devai Cc: Grant Likely, devicetree-discuss, Russell King, Arnd Bergmann, linux-arm-kernel On 10/19/2011 11:01 AM, Zoltan Devai wrote: > This adds the DT description of the SPMP8000 SoC, the Letcool > N350JP board and documentation of required bindings > > Cc: Grant Likely <grant.likely@secretlab.ca> > CC: devicetree-discuss@lists.ozlabs.org > Signed-off-by: Zoltan Devai <zoss@devai.org> > --- Only a couple of minor things below. Otherwise looks good. Acked-by: Rob Herring <rob.herring@calxeda.com> > Documentation/devicetree/bindings/arm/spmp8000.txt | 25 +++++ > arch/arm/boot/dts/spmp8000-letcool.dts | 20 ++++ > arch/arm/boot/dts/spmp8000.dtsi | 93 ++++++++++++++++++++ > 3 files changed, 138 insertions(+), 0 deletions(-) > create mode 100644 Documentation/devicetree/bindings/arm/spmp8000.txt > create mode 100644 arch/arm/boot/dts/spmp8000-letcool.dts > create mode 100644 arch/arm/boot/dts/spmp8000.dtsi > > diff --git a/Documentation/devicetree/bindings/arm/spmp8000.txt b/Documentation/devicetree/bindings/arm/spmp8000.txt > new file mode 100644 > index 0000000..c392b09 > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/spmp8000.txt > @@ -0,0 +1,25 @@ > +Sunplus SPMP8000 device tree bindings > +===================================== > + > +Required root node properties: > + - compatible: > + - "gameware,letcool" : Letcool N350JO handheld game console board > + - "sunplus,spmp8000" : A board based on the SPMP8000 SoC > + > +Timer required properties: > + - compatible = "sunplus,spmp8000-timer" > + - interrupt-parent : The interrupt controller node this timer belongs to > + - interrupts : IRQs of the timer > + - clock-freq : The frequency in HZ of the timer. "clock-frequency" is what is commonly used. > + - reg : The register bank for the timer. > + > +Watchdog required properties: > + - compatible = "sunplus,spmp8000-watchdof" s/watchdof/watchdog/ > + - interrupt-parent : The interrupt controller node this timer belongs to > + - interrupts : IRQs of the watchdog > + - reg : The register bank for the watchdog controller > + > +SCU required properties: > + - compatible: "sunplus,spmp8000-scua" or "sunplus,spmp8000-scuab" or > + "sunplus,spmp8000-scuc" > + - reg: The register bank of the System Control Unit > diff --git a/arch/arm/boot/dts/spmp8000-letcool.dts b/arch/arm/boot/dts/spmp8000-letcool.dts > new file mode 100644 > index 0000000..424f2aa > --- /dev/null > +++ b/arch/arm/boot/dts/spmp8000-letcool.dts > @@ -0,0 +1,20 @@ > +/dts-v1/; > +/include/ "spmp8000.dtsi" > +/ { > + model = "Letcool N350JP handheld game console"; > + compatible = "gameware,letcool", "sunplus,spmp8000"; > + > + memory { > + reg = <0x00000000 0x02000000>; > + }; > + > + chosen { > + linux,stdout-path = &uart0; > + }; > + > + armapb { > + timer@90000000 { > + clock-freq = <15187500>; > + }; > + }; > +}; > diff --git a/arch/arm/boot/dts/spmp8000.dtsi b/arch/arm/boot/dts/spmp8000.dtsi > new file mode 100644 > index 0000000..15b25fa > --- /dev/null > +++ b/arch/arm/boot/dts/spmp8000.dtsi > @@ -0,0 +1,93 @@ > +/ { > + compatible = "sunplus,spmp8000"; > + #address-cells = <1>; > + #size-cells = <1>; > + > + armapb { > + compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0x90000000 0x10000>; > + > + timer@90000000 { > + compatible = "sunplus,spmp8000-timer"; > + reg = <0x0 0x1000>; > + interrupt-parent = <&vic0>; > + interrupts = <7 8 9>; > + }; > + > + watchdog@90001000 { > + compatible = "sunplus,spmp8000-wdt"; > + reg = <0x1000 0x1000>; > + interrupt-parent = <&vic0>; > + interrupts = <10 11 12>; > + }; > + > + scu-b@90005000 { > + compatible = "sunplus,spmp8000-scub"; > + reg = <0x5000 0x1000>; > + }; > + }; > + > + armahb { > + compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0x90010000 0x20000>; > + > + vic0: interrupt-controller@90010000 { > + compatible = "arm,pl192"; > + interrupt-controller; > + #interrupt-cells = <1>; > + reg = <0x0 0x1000>; > + }; > + > + vic1: interrupt-controller@900020000 { > + compatible = "arm,pl192"; > + interrupt-controller; > + #interrupt-cells = <1>; > + reg = <0x10000 0x1000>; > + }; > + }; > + > + plat { > + compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0x92000000 0x1000000>; > + > + scu-c@92005000 { > + compatible = "sunplus,spmp8000-scuc"; > + reg = <0x5000 0x1000>; > + }; > + > + plat-apb { > + compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0xB00000 0x10000>; lower case for hex is preferred. > + > + uart0: uart-c0@92B04000 { > + compatible = "ns16550a"; > + reg = <0x4000 0x1000>; > + clock-frequency = <2076923>; > + interrupt-parent = <&vic1>; > + interrupts = <24>; > + current-speed = <115200>; > + reg-shift = <2>; > + }; > + }; > + }; > + > + axi { > + compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0x93000000 0x20000>; > + > + scu-a@93007000 { > + compatible = "sunplus,spmp8000-scua"; > + reg = <0x7000 0x1000>; > + }; > + }; > +}; ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-10-24 12:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1318178172-7965-1-git-send-email-zoss@devai.org>
[not found] ` <201110162259.19960.arnd@arndb.de>
[not found] ` <20111016205221.GL1459@game.jcrosoft.org>
2011-10-17 11:44 ` [PATCH 2/9] ARM: SPMP8000: Add machine base files Arnd Bergmann
[not found] ` <1319040118-29773-1-git-send-email-zoss@devai.org>
2011-10-19 16:01 ` [PATCH v2 4/5] ARM: SPMP8000: Add SPMP8000 SoC and Letcool board dts descriptions Zoltan Devai
2011-10-24 12:47 ` Rob Herring
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).