diff mbox

[v2,2/5] ARM: STi: DT: STiH407: Enable Mailbox testing facility

Message ID 1458744223-12164-3-git-send-email-lee.jones@linaro.org
State New
Headers show

Commit Message

Lee Jones March 23, 2016, 2:43 p.m. UTC
This patch supplies a Client node to enable the Mailbox testing
facility.  It will be used to send and receive messages from any
given co-processor in order to test the STi Mailbox Controller
driver.

Signed-off-by: Lee Jones <lee.jones@linaro.org>

---
 arch/arm/boot/dts/stih407-family.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)

-- 
1.9.1

Comments

Sudeep Holla April 12, 2016, 9:18 a.m. UTC | #1
Hi Lee,

On 23/03/16 14:43, Lee Jones wrote:
> This patch supplies a Client node to enable the Mailbox testing

> facility.  It will be used to send and receive messages from any

> given co-processor in order to test the STi Mailbox Controller

> driver.

>

> Signed-off-by: Lee Jones <lee.jones@linaro.org>

> ---

>   arch/arm/boot/dts/stih407-family.dtsi | 6 ++++++

>   1 file changed, 6 insertions(+)

>

> diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi

> index e6e34b4..9376c38 100644

> --- a/arch/arm/boot/dts/stih407-family.dtsi

> +++ b/arch/arm/boot/dts/stih407-family.dtsi

> @@ -695,6 +695,12 @@

>   			status		= "okay";

>   		};

>

> +		mailbox_test {

> +			compatible = "mailbox_test";


IIRC, we changed this to "mailbox-text" ?

-- 
Regards,
Sudeep
Lee Jones April 12, 2016, 2:19 p.m. UTC | #2
On Tue, 12 Apr 2016, Sudeep Holla wrote:

> Hi Lee,

> 

> On 23/03/16 14:43, Lee Jones wrote:

> >This patch supplies a Client node to enable the Mailbox testing

> >facility.  It will be used to send and receive messages from any

> >given co-processor in order to test the STi Mailbox Controller

> >driver.

> >

> >Signed-off-by: Lee Jones <lee.jones@linaro.org>

> >---

> >  arch/arm/boot/dts/stih407-family.dtsi | 6 ++++++

> >  1 file changed, 6 insertions(+)

> >

> >diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi

> >index e6e34b4..9376c38 100644

> >--- a/arch/arm/boot/dts/stih407-family.dtsi

> >+++ b/arch/arm/boot/dts/stih407-family.dtsi

> >@@ -695,6 +695,12 @@

> >  			status		= "okay";

> >  		};

> >

> >+		mailbox_test {

> >+			compatible = "mailbox_test";

> 

> IIRC, we changed this to "mailbox-text" ?


Great spot, thanks.  I will fix before sending to Maxime.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
diff mbox

Patch

diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
index e6e34b4..9376c38 100644
--- a/arch/arm/boot/dts/stih407-family.dtsi
+++ b/arch/arm/boot/dts/stih407-family.dtsi
@@ -695,6 +695,12 @@ 
 			status		= "okay";
 		};
 
+		mailbox_test {
+			compatible = "mailbox_test";
+			mboxes = <&mailbox2 1 31>, <&mailbox0 1 31>;
+			mbox-names = "tx", "rx";
+		};
+
 		mailbox0: mailbox@0  {
 			compatible	= "st,stih407-mailbox";
 			reg		= <0x8f00000 0x1000>;