ARM: dts: OMAP2+: Add mailbox nodes
Subsystems (4)
PATCH 3/3 ARM: dts: OMAP2+: Add mailbox nodes
The mailbox DT node data has been added for OMAP2420, OMAP2430, OMAP3430/OMAP3630, OMAP44xx devices. Data for OMAP5 is skipped for now since the corresponding hwmod entry is not present.
The mailbox static device initialization logic is also adjusted for a DT boot.
--- arch/arm/boot/dts/omap2420.dtsi | 13 +++++++++++++ arch/arm/boot/dts/omap2430.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap3.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap4.dtsi | 12 ++++++++++++ arch/arm/mach-omap2/devices.c | 2 +- 5 files changed, 50 insertions(+), 1 deletion(-)
arch/arm/boot/dts/omap2420.dtsi +13 -0
@@ -114,6 +114,19 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>, /* DSP Interrupt */ + <34>; /* IVA Interrupt */ + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp", "iva"; + ti,mbox-data = <0 1 0 0>, <2 3 1 3>; + }; + timer1: timer at 48028000 { compatible = "ti,omap2420-timer"; reg = <0x48028000 0x400>;
arch/arm/boot/dts/omap2430.dtsi +12 -0
@@ -175,6 +175,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 49018000 { compatible = "ti,omap2420-timer"; reg = <0x49018000 0x400>;
arch/arm/boot/dts/omap3.dtsi +12 -0
@@ -380,6 +380,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <2>; + ti,mbox-num-fifos = <2>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 48318000 { compatible = "ti,omap3430-timer"; reg = <0x48318000 0x400>;
arch/arm/boot/dts/omap4.dtsi +12 -0
@@ -516,6 +516,18 @@ }; }; + mailbox: mailbox at 4a0f4000 { + compatible = "ti,omap4-mailbox"; + reg = <0x4a0f4000 0x200>; + interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <3>; + ti,mbox-num-fifos = <8>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "mbox-ipu", "mbox-dsp"; + ti,mbox-data = <0 1 0 0>, <3 2 0 0>; + }; + timer1: timer at 4a318000 { compatible = "ti,omap3430-timer"; reg = <0x4a318000 0x80>;
arch/arm/mach-omap2/devices.c +1 -1
@@ -655,11 +655,11 @@ static int __init omap2_init_devices(void)omap_init_audio ();omap_init_camera ();omap_init_hdmi_audio (); -omap_init_mbox ();/* If dtb is there, the devices will be created dynamically */ if (!of_have_populated_dt ()) {omap_init_control_usb ();omap_init_dmic (); +omap_init_mbox ();omap_init_mcpdm ();omap_init_mcspi ();omap_init_sham ();
Signature · 1 line
1.8.3.1
PATCH 3/3 ARM: dts: OMAP2+: Add mailbox nodes
Quoted 9 lines · Suman Anna
The mailbox DT node data has been added for OMAP2420, OMAP2430, OMAP3430/OMAP3630, OMAP44xx devices. Data for OMAP5 is skipped for now since the corresponding hwmod entry is not present.
The mailbox static device initialization logic is also adjusted for a DT boot.
Signed-off-by Suman Anna <[email protected]>
Quoted 125 lines
--- arch/arm/boot/dts/omap2420.dtsi | 13 +++++++++++++ arch/arm/boot/dts/omap2430.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap3.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap4.dtsi | 12 ++++++++++++ arch/arm/mach-omap2/devices.c | 2 +- 5 files changed, 50 insertions(+), 1 deletion(-)arch/arm/boot/dts/omap2420.dtsi +13
-0@@ -114,6 +114,19 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>, /* DSP Interrupt */ + <34>; /* IVA Interrupt */ + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp", "iva"; + ti,mbox-data = <0 1 0 0>, <2 3 1 3>; + }; + timer1: timer at 48028000 { compatible = "ti,omap2420-timer"; reg = <0x48028000 0x400>;arch/arm/boot/dts/omap2430.dtsi +12
-0@@ -175,6 +175,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 49018000 { compatible = "ti,omap2420-timer"; reg = <0x49018000 0x400>;arch/arm/boot/dts/omap3.dtsi +12
-0@@ -380,6 +380,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <2>; + ti,mbox-num-fifos = <2>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 48318000 { compatible = "ti,omap3430-timer"; reg = <0x48318000 0x400>;arch/arm/boot/dts/omap4.dtsi +12
-0@@ -516,6 +516,18 @@ }; }; + mailbox: mailbox at 4a0f4000 { + compatible = "ti,omap4-mailbox"; + reg = <0x4a0f4000 0x200>; + interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <3>; + ti,mbox-num-fifos = <8>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "mbox-ipu", "mbox-dsp"; + ti,mbox-data = <0 1 0 0>, <3 2 0 0>; + }; + timer1: timer at 4a318000 { compatible = "ti,omap3430-timer"; reg = <0x4a318000 0x80>;arch/arm/mach-omap2/devices.c +1
-2@@ -655,11 +655,11 @@ static int __init omap2_init_devices(void)omap_init_audio ();omap_init_camera ();omap_init_hdmi_audio (); -omap_init_mbox ();/* If dtb is there, the devices will be created dynamically */ if (!of_have_populated_dt ()) {omap_init_control_usb ();omap_init_dmic (); +omap_init_mbox ();omap_init_mcpdm ();omap_init_mcspi ();omap_init_sham (); --1.8.3.1
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
PATCH 3/3 ARM: dts: OMAP2+: Add mailbox nodes
In reply to Suman AnnaQuoted 24 lines · Suman Anna
The mailbox DT node data has been added for OMAP2420, OMAP2430, OMAP3430/OMAP3630, OMAP44xx devices. Data for OMAP5 is skipped for now since the corresponding hwmod entry is not present.
The mailbox static device initialization logic is also adjusted for a DT boot.
Signed-off-by Suman Anna <[email protected]>--- arch/arm/boot/dts/omap2420.dtsi | 13 +++++++++++++ arch/arm/boot/dts/omap2430.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap3.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap4.dtsi | 12 ++++++++++++ arch/arm/mach-omap2/devices.c | 2 +- 5 files changed, 50 insertions(+), 1 deletion(-)arch/arm/boot/dts/omap2420.dtsi +0
-0@@ -114,6 +114,19 @@ dma-names = "tx", "rx"; };
arch/arm/boot/dts/omap2420.dtsi line 125
Quoted 10 lines
arch/arm/boot/dts/omap2420.dtsi +9
-0+ mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>, /* DSP Interrupt */ + <34>; /* IVA Interrupt */ + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>;
If this is always 4, why do you want to expose that?
BTW, I guess that most of these attribute are generic enough to avoid the "ti," prefix.
Benoit
Quoted 94 lines
arch/arm/boot/dts/omap2420.dtsi +4
-0+ ti,mbox-names = "dsp", "iva"; + ti,mbox-data = <0 1 0 0>, <2 3 1 3>; + }; + timer1: timer at 48028000 { compatible = "ti,omap2420-timer"; reg = <0x48028000 0x400>;arch/arm/boot/dts/omap2430.dtsi +12
-0@@ -175,6 +175,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 49018000 { compatible = "ti,omap2420-timer"; reg = <0x49018000 0x400>;arch/arm/boot/dts/omap3.dtsi +12
-0@@ -380,6 +380,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <2>; + ti,mbox-num-fifos = <2>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 48318000 { compatible = "ti,omap3430-timer"; reg = <0x48318000 0x400>;arch/arm/boot/dts/omap4.dtsi +12
-0@@ -516,6 +516,18 @@ }; }; + mailbox: mailbox at 4a0f4000 { + compatible = "ti,omap4-mailbox"; + reg = <0x4a0f4000 0x200>; + interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <3>; + ti,mbox-num-fifos = <8>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "mbox-ipu", "mbox-dsp"; + ti,mbox-data = <0 1 0 0>, <3 2 0 0>; + }; + timer1: timer at 4a318000 { compatible = "ti,omap3430-timer"; reg = <0x4a318000 0x80>;arch/arm/mach-omap2/devices.c +1
-1@@ -655,11 +655,11 @@ static int __init omap2_init_devices(void)omap_init_audio ();omap_init_camera ();omap_init_hdmi_audio (); -omap_init_mbox ();/* If dtb is there, the devices will be created dynamically */ if (!of_have_populated_dt ()) {omap_init_control_usb ();omap_init_dmic (); +omap_init_mbox ();omap_init_mcpdm ();omap_init_mcspi ();omap_init_sham ();
PATCH 3/3 ARM: dts: OMAP2+: Add mailbox nodes
Hi Benoit,
Quoted 27 lines · Benoit
Quoted 26 lines · Suman Anna
The mailbox DT node data has been added for OMAP2420, OMAP2430, OMAP3430/OMAP3630, OMAP44xx devices. Data for OMAP5 is skipped for now since the corresponding hwmod entry is not present.
The mailbox static device initialization logic is also adjusted for a DT boot.
Signed-off-by Suman Anna <[email protected]>--- arch/arm/boot/dts/omap2420.dtsi | 13 +++++++++++++ arch/arm/boot/dts/omap2430.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap3.dtsi | 12 ++++++++++++ arch/arm/boot/dts/omap4.dtsi | 12 ++++++++++++ arch/arm/mach-omap2/devices.c | 2 +- 5 files changed, 50 insertions(+), 1 deletion(-)arch/arm/boot/dts/omap2420.dtsi +0
-0@@ -114,6 +114,19 @@ dma-names = "tx", "rx"; };
Quoted 11 lines
arch/arm/boot/dts/omap2420.dtsi line 125
Quoted 9 lines
arch/arm/boot/dts/omap2420.dtsi +9
-0+ mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>, /* DSP Interrupt */ + <34>; /* IVA Interrupt */ + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>;If this is always 4, why do you want to expose that?
I have followed the example of interrupt-cells here, as it indicates the length that each mbox-data should have. If you do think that it doesn't belong here, I can remove this and do the check directly in the driver, and update the DT binding accordingly.
Quoted 2 lines
BTW, I guess that most of these attribute are generic enough to avoid the "ti," prefix.
Since these are specific attributes needed for describing the data for TI mailboxes, I used the "ti,' prefix so that it won't clash with other SoCs.
regards
Suman
Quoted 100 lines
Benoit
Quoted 96 lines
arch/arm/boot/dts/omap2420.dtsi +4
-0+ ti,mbox-names = "dsp", "iva"; + ti,mbox-data = <0 1 0 0>, <2 3 1 3>; + }; + timer1: timer at 48028000 { compatible = "ti,omap2420-timer"; reg = <0x48028000 0x400>;arch/arm/boot/dts/omap2430.dtsi +12
-0@@ -175,6 +175,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <4>; + ti,mbox-num-fifos = <6>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 49018000 { compatible = "ti,omap2420-timer"; reg = <0x49018000 0x400>;arch/arm/boot/dts/omap3.dtsi +12
-0@@ -380,6 +380,18 @@ dma-names = "tx", "rx"; }; + mailbox: mailbox at 48094000 { + compatible = "ti,omap2-mailbox"; + reg = <0x48094000 0x200>; + interrupts = <26>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <2>; + ti,mbox-num-fifos = <2>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "dsp"; + ti,mbox-data = <0 1 0 0>; + }; + timer1: timer at 48318000 { compatible = "ti,omap3430-timer"; reg = <0x48318000 0x400>;arch/arm/boot/dts/omap4.dtsi +12
-0@@ -516,6 +516,18 @@ }; }; + mailbox: mailbox at 4a0f4000 { + compatible = "ti,omap4-mailbox"; + reg = <0x4a0f4000 0x200>; + interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>; + ti,hwmods = "mailbox"; + ti,mbox-num-users = <3>; + ti,mbox-num-fifos = <8>; + #ti,mbox-data-cells = <4>; + ti,mbox-names = "mbox-ipu", "mbox-dsp"; + ti,mbox-data = <0 1 0 0>, <3 2 0 0>; + }; + timer1: timer at 4a318000 { compatible = "ti,omap3430-timer"; reg = <0x4a318000 0x80>;arch/arm/mach-omap2/devices.c +1
-1@@ -655,11 +655,11 @@ static int __init omap2_init_devices(void)omap_init_audio ();omap_init_camera ();omap_init_hdmi_audio (); -omap_init_mbox ();/* If dtb is there, the devices will be created dynamically */ if (!of_have_populated_dt ()) {omap_init_control_usb ();omap_init_dmic (); +omap_init_mbox ();omap_init_mcpdm ();omap_init_mcspi ();omap_init_sham ();