Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-10-29 Thread Tony Lindgren
* Lokesh Vutla lokeshvu...@ti.com [131016 04:16]:
 + Rob and Mark
 On Tuesday 15 October 2013 12:19 PM, Benoit Cousson wrote:
  Hi Lokesh,
  
  On 12/10/2013 15:26, Lokesh Vutla wrote:
  Hi Benoit,
  On Thursday 29 August 2013 06:22 PM, Lokesh Vutla wrote:
  Add the AM33xx RNG module's device tree data.
  Also add Documentation file describing the data
  for the RNG module.
 Can you please review this patch..

OK enough of waiting here, I'm picking this patch into
omap-for-v3.13/dt branch as it removes some boot time warnings
we're seeing.

Regards,

Tony
 
 Thanks and regards,
 Lokesh
  Seems you missed this patch. Please consider this for this merge window.
  
  Indeed I missed it. Sorry :-(
  
  Patch 2/2 is already taken by Paul.
  Without this patch AM335x boot will crash as I mentioned in my cover 
  letter.
 
  Thanks and regards,
  Lokesh
 
  Signed-off-by: Lokesh Vutla lokeshvu...@ti.com
  ---
  Changes since V1:
  - Drop status=disabled entry in dt node, Since
 RNG is present on all AM33xx platforms.
 
.../devicetree/bindings/hwrng/omap_rng.txt |   22 
  
arch/arm/boot/dts/am33xx.dtsi  |7 +++
2 files changed, 29 insertions(+)
create mode 100644 Documentation/devicetree/bindings/hwrng/omap_rng.txt
  
  I do not see the acked-by for the binding part. You do need that for any 
  new bindings nowadays.
  
 
  diff --git a/Documentation/devicetree/bindings/hwrng/omap_rng.txt 
  b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
  new file mode 100644
  index 000..6a62acd
  --- /dev/null
  +++ b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
  @@ -0,0 +1,22 @@
  +OMAP SoC HWRNG Module
  +
  +Required properties:
  +
  +- compatible : Should contain entries for this and backward compatible
  +  RNG versions:
  +  - ti,omap2-rng for OMAP2.
  +  - ti,omap4-rng for OMAP4, OMAP5 and AM33XX.
  +  Note that these two versions are incompatible.
  +- ti,hwmods: Name of the hwmod associated with the RNG module
  +- reg : Offset and length of the register set for the module
  +- interrupts : the interrupt number for the RNG module.
  +Only used for ti,omap4-rng.
  +
  +Example:
  +/* AM335x */
  +rng: rng@4831 {
  +compatible = ti,omap4-rng;
  +ti,hwmods = rng;
  +reg = 0x4831 0x2000;
  +interrupts = 111;
  +};
  diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
  index 38b446b..26ebe30 100644
  --- a/arch/arm/boot/dts/am33xx.dtsi
  +++ b/arch/arm/boot/dts/am33xx.dtsi
  @@ -530,5 +530,12 @@
#size-cells = 1;
status = disabled;
};
  +
  +rng: rng@4831 {
  +compatible = ti,omap4-rng;
  +ti,hwmods = rng;
  +reg = 0x4831 0x2000;
  +interrupts = 111;
  +};
};
};
 
 
  
  It looks good to me, but it does not apply on my branch.
  Please rebase it and repost with the proper acked-by.
  
  Thanks,
  Benoit
  
  
 
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-10-29 Thread Kumar Gala

On Aug 29, 2013, at 7:52 AM, Lokesh Vutla wrote:

 Add the AM33xx RNG module's device tree data.
 Also add Documentation file describing the data
 for the RNG module.
 
 Signed-off-by: Lokesh Vutla lokeshvu...@ti.com
 ---
 Changes since V1:
 - Drop status=disabled entry in dt node, Since
  RNG is present on all AM33xx platforms.
 
 .../devicetree/bindings/hwrng/omap_rng.txt |   22 
 arch/arm/boot/dts/am33xx.dtsi  |7 +++
 2 files changed, 29 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/hwrng/omap_rng.txt
 
 diff --git a/Documentation/devicetree/bindings/hwrng/omap_rng.txt 
 b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 new file mode 100644
 index 000..6a62acd
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 @@ -0,0 +1,22 @@
 +OMAP SoC HWRNG Module
 +
 +Required properties:
 +
 +- compatible : Should contain entries for this and backward compatible
 +  RNG versions:
 +  - ti,omap2-rng for OMAP2.
 +  - ti,omap4-rng for OMAP4, OMAP5 and AM33XX.
 +  Note that these two versions are incompatible.
 +- ti,hwmods: Name of the hwmod associated with the RNG module
 +- reg : Offset and length of the register set for the module
 +- interrupts : the interrupt number for the RNG module.
 + Only used for ti,omap4-rng.
 +
 +Example:
 +/* AM335x */
 +rng: rng@4831 {
 + compatible = ti,omap4-rng;
 + ti,hwmods = rng;
 + reg = 0x4831 0x2000;
 + interrupts = 111;
 +};

For the DT-Binding portion:

Acked-by: Kumar Gala ga...@codeaurora.org

- k

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by 
The Linux Foundation

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-10-16 Thread Lokesh Vutla
Hi Benoit,
On Tuesday 15 October 2013 12:19 PM, Benoit Cousson wrote:
 Hi Lokesh,
 
 On 12/10/2013 15:26, Lokesh Vutla wrote:
 Hi Benoit,
 On Thursday 29 August 2013 06:22 PM, Lokesh Vutla wrote:
 Add the AM33xx RNG module's device tree data.
 Also add Documentation file describing the data
 for the RNG module.
 Seems you missed this patch. Please consider this for this merge window.
 
 Indeed I missed it. Sorry :-(
 
 Patch 2/2 is already taken by Paul.
 Without this patch AM335x boot will crash as I mentioned in my cover letter.

 Thanks and regards,
 Lokesh

 Signed-off-by: Lokesh Vutla lokeshvu...@ti.com
 ---
 Changes since V1:
 - Drop status=disabled entry in dt node, Since
RNG is present on all AM33xx platforms.

   .../devicetree/bindings/hwrng/omap_rng.txt |   22 
 
   arch/arm/boot/dts/am33xx.dtsi  |7 +++
   2 files changed, 29 insertions(+)
   create mode 100644 Documentation/devicetree/bindings/hwrng/omap_rng.txt
 
 I do not see the acked-by for the binding part. You do need that for any new 
 bindings nowadays.
 

 diff --git a/Documentation/devicetree/bindings/hwrng/omap_rng.txt 
 b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 new file mode 100644
 index 000..6a62acd
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 @@ -0,0 +1,22 @@
 +OMAP SoC HWRNG Module
 +
 +Required properties:
 +
 +- compatible : Should contain entries for this and backward compatible
 +  RNG versions:
 +  - ti,omap2-rng for OMAP2.
 +  - ti,omap4-rng for OMAP4, OMAP5 and AM33XX.
 +  Note that these two versions are incompatible.
 +- ti,hwmods: Name of the hwmod associated with the RNG module
 +- reg : Offset and length of the register set for the module
 +- interrupts : the interrupt number for the RNG module.
 +Only used for ti,omap4-rng.
 +
 +Example:
 +/* AM335x */
 +rng: rng@4831 {
 +compatible = ti,omap4-rng;
 +ti,hwmods = rng;
 +reg = 0x4831 0x2000;
 +interrupts = 111;
 +};
 diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
 index 38b446b..26ebe30 100644
 --- a/arch/arm/boot/dts/am33xx.dtsi
 +++ b/arch/arm/boot/dts/am33xx.dtsi
 @@ -530,5 +530,12 @@
   #size-cells = 1;
   status = disabled;
   };
 +
 +rng: rng@4831 {
 +compatible = ti,omap4-rng;
 +ti,hwmods = rng;
 +reg = 0x4831 0x2000;
 +interrupts = 111;
 +};
   };
   };


 
 It looks good to me, but it does not apply on my branch.
Can I take this as your Acked-by: ?
Ok, Ill rebase on top of your tree and post it.

Thanks and Regards,
Lokesh
 Please rebase it and repost with the proper acked-by.
 
 Thanks,
 Benoit
 
 

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-10-16 Thread Lokesh Vutla
+ Rob and Mark
On Tuesday 15 October 2013 12:19 PM, Benoit Cousson wrote:
 Hi Lokesh,
 
 On 12/10/2013 15:26, Lokesh Vutla wrote:
 Hi Benoit,
 On Thursday 29 August 2013 06:22 PM, Lokesh Vutla wrote:
 Add the AM33xx RNG module's device tree data.
 Also add Documentation file describing the data
 for the RNG module.
Can you please review this patch..

Thanks and regards,
Lokesh
 Seems you missed this patch. Please consider this for this merge window.
 
 Indeed I missed it. Sorry :-(
 
 Patch 2/2 is already taken by Paul.
 Without this patch AM335x boot will crash as I mentioned in my cover letter.

 Thanks and regards,
 Lokesh

 Signed-off-by: Lokesh Vutla lokeshvu...@ti.com
 ---
 Changes since V1:
 - Drop status=disabled entry in dt node, Since
RNG is present on all AM33xx platforms.

   .../devicetree/bindings/hwrng/omap_rng.txt |   22 
 
   arch/arm/boot/dts/am33xx.dtsi  |7 +++
   2 files changed, 29 insertions(+)
   create mode 100644 Documentation/devicetree/bindings/hwrng/omap_rng.txt
 
 I do not see the acked-by for the binding part. You do need that for any new 
 bindings nowadays.
 

 diff --git a/Documentation/devicetree/bindings/hwrng/omap_rng.txt 
 b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 new file mode 100644
 index 000..6a62acd
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 @@ -0,0 +1,22 @@
 +OMAP SoC HWRNG Module
 +
 +Required properties:
 +
 +- compatible : Should contain entries for this and backward compatible
 +  RNG versions:
 +  - ti,omap2-rng for OMAP2.
 +  - ti,omap4-rng for OMAP4, OMAP5 and AM33XX.
 +  Note that these two versions are incompatible.
 +- ti,hwmods: Name of the hwmod associated with the RNG module
 +- reg : Offset and length of the register set for the module
 +- interrupts : the interrupt number for the RNG module.
 +Only used for ti,omap4-rng.
 +
 +Example:
 +/* AM335x */
 +rng: rng@4831 {
 +compatible = ti,omap4-rng;
 +ti,hwmods = rng;
 +reg = 0x4831 0x2000;
 +interrupts = 111;
 +};
 diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
 index 38b446b..26ebe30 100644
 --- a/arch/arm/boot/dts/am33xx.dtsi
 +++ b/arch/arm/boot/dts/am33xx.dtsi
 @@ -530,5 +530,12 @@
   #size-cells = 1;
   status = disabled;
   };
 +
 +rng: rng@4831 {
 +compatible = ti,omap4-rng;
 +ti,hwmods = rng;
 +reg = 0x4831 0x2000;
 +interrupts = 111;
 +};
   };
   };


 
 It looks good to me, but it does not apply on my branch.
 Please rebase it and repost with the proper acked-by.
 
 Thanks,
 Benoit
 
 

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-10-15 Thread Benoit Cousson

Hi Lokesh,

On 12/10/2013 15:26, Lokesh Vutla wrote:

Hi Benoit,
On Thursday 29 August 2013 06:22 PM, Lokesh Vutla wrote:

Add the AM33xx RNG module's device tree data.
Also add Documentation file describing the data
for the RNG module.

Seems you missed this patch. Please consider this for this merge window.


Indeed I missed it. Sorry :-(


Patch 2/2 is already taken by Paul.
Without this patch AM335x boot will crash as I mentioned in my cover letter.

Thanks and regards,
Lokesh


Signed-off-by: Lokesh Vutla lokeshvu...@ti.com
---
Changes since V1:
- Drop status=disabled entry in dt node, Since
   RNG is present on all AM33xx platforms.

  .../devicetree/bindings/hwrng/omap_rng.txt |   22 
  arch/arm/boot/dts/am33xx.dtsi  |7 +++
  2 files changed, 29 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/hwrng/omap_rng.txt


I do not see the acked-by for the binding part. You do need that for any 
new bindings nowadays.




diff --git a/Documentation/devicetree/bindings/hwrng/omap_rng.txt 
b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
new file mode 100644
index 000..6a62acd
--- /dev/null
+++ b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
@@ -0,0 +1,22 @@
+OMAP SoC HWRNG Module
+
+Required properties:
+
+- compatible : Should contain entries for this and backward compatible
+  RNG versions:
+  - ti,omap2-rng for OMAP2.
+  - ti,omap4-rng for OMAP4, OMAP5 and AM33XX.
+  Note that these two versions are incompatible.
+- ti,hwmods: Name of the hwmod associated with the RNG module
+- reg : Offset and length of the register set for the module
+- interrupts : the interrupt number for the RNG module.
+   Only used for ti,omap4-rng.
+
+Example:
+/* AM335x */
+rng: rng@4831 {
+   compatible = ti,omap4-rng;
+   ti,hwmods = rng;
+   reg = 0x4831 0x2000;
+   interrupts = 111;
+};
diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 38b446b..26ebe30 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -530,5 +530,12 @@
#size-cells = 1;
status = disabled;
};
+
+   rng: rng@4831 {
+   compatible = ti,omap4-rng;
+   ti,hwmods = rng;
+   reg = 0x4831 0x2000;
+   interrupts = 111;
+   };
};
  };





It looks good to me, but it does not apply on my branch.
Please rebase it and repost with the proper acked-by.

Thanks,
Benoit


--
BenoƮt Cousson
BayLibre
Embedded Linux Technology Lab
www.baylibre.com
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-10-12 Thread Lokesh Vutla
Hi Benoit,
On Thursday 29 August 2013 06:22 PM, Lokesh Vutla wrote:
 Add the AM33xx RNG module's device tree data.
 Also add Documentation file describing the data
 for the RNG module.
Seems you missed this patch. Please consider this for this merge window.
Patch 2/2 is already taken by Paul.
Without this patch AM335x boot will crash as I mentioned in my cover letter.

Thanks and regards,
Lokesh
 
 Signed-off-by: Lokesh Vutla lokeshvu...@ti.com
 ---
 Changes since V1:
 - Drop status=disabled entry in dt node, Since
   RNG is present on all AM33xx platforms.
 
  .../devicetree/bindings/hwrng/omap_rng.txt |   22 
 
  arch/arm/boot/dts/am33xx.dtsi  |7 +++
  2 files changed, 29 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/hwrng/omap_rng.txt
 
 diff --git a/Documentation/devicetree/bindings/hwrng/omap_rng.txt 
 b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 new file mode 100644
 index 000..6a62acd
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
 @@ -0,0 +1,22 @@
 +OMAP SoC HWRNG Module
 +
 +Required properties:
 +
 +- compatible : Should contain entries for this and backward compatible
 +  RNG versions:
 +  - ti,omap2-rng for OMAP2.
 +  - ti,omap4-rng for OMAP4, OMAP5 and AM33XX.
 +  Note that these two versions are incompatible.
 +- ti,hwmods: Name of the hwmod associated with the RNG module
 +- reg : Offset and length of the register set for the module
 +- interrupts : the interrupt number for the RNG module.
 + Only used for ti,omap4-rng.
 +
 +Example:
 +/* AM335x */
 +rng: rng@4831 {
 + compatible = ti,omap4-rng;
 + ti,hwmods = rng;
 + reg = 0x4831 0x2000;
 + interrupts = 111;
 +};
 diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
 index 38b446b..26ebe30 100644
 --- a/arch/arm/boot/dts/am33xx.dtsi
 +++ b/arch/arm/boot/dts/am33xx.dtsi
 @@ -530,5 +530,12 @@
   #size-cells = 1;
   status = disabled;
   };
 +
 + rng: rng@4831 {
 + compatible = ti,omap4-rng;
 + ti,hwmods = rng;
 + reg = 0x4831 0x2000;
 + interrupts = 111;
 + };
   };
  };
 

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-10-12 Thread Paul Walmsley
On Sat, 12 Oct 2013, Lokesh Vutla wrote:

 Seems you missed this patch. Please consider this for this merge window.
 Patch 2/2 is already taken by Paul.
 Without this patch AM335x boot will crash as I mentioned in my cover letter.

That won't happen since this patch from Suman got merged:

http://www.mail-archive.com/linux-omap@vger.kernel.org/msg96447.html


- Paul
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node

2013-08-29 Thread Lokesh Vutla
Add the AM33xx RNG module's device tree data.
Also add Documentation file describing the data
for the RNG module.

Signed-off-by: Lokesh Vutla lokeshvu...@ti.com
---
Changes since V1:
- Drop status=disabled entry in dt node, Since
  RNG is present on all AM33xx platforms.

 .../devicetree/bindings/hwrng/omap_rng.txt |   22 
 arch/arm/boot/dts/am33xx.dtsi  |7 +++
 2 files changed, 29 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/hwrng/omap_rng.txt

diff --git a/Documentation/devicetree/bindings/hwrng/omap_rng.txt 
b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
new file mode 100644
index 000..6a62acd
--- /dev/null
+++ b/Documentation/devicetree/bindings/hwrng/omap_rng.txt
@@ -0,0 +1,22 @@
+OMAP SoC HWRNG Module
+
+Required properties:
+
+- compatible : Should contain entries for this and backward compatible
+  RNG versions:
+  - ti,omap2-rng for OMAP2.
+  - ti,omap4-rng for OMAP4, OMAP5 and AM33XX.
+  Note that these two versions are incompatible.
+- ti,hwmods: Name of the hwmod associated with the RNG module
+- reg : Offset and length of the register set for the module
+- interrupts : the interrupt number for the RNG module.
+   Only used for ti,omap4-rng.
+
+Example:
+/* AM335x */
+rng: rng@4831 {
+   compatible = ti,omap4-rng;
+   ti,hwmods = rng;
+   reg = 0x4831 0x2000;
+   interrupts = 111;
+};
diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 38b446b..26ebe30 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -530,5 +530,12 @@
#size-cells = 1;
status = disabled;
};
+
+   rng: rng@4831 {
+   compatible = ti,omap4-rng;
+   ti,hwmods = rng;
+   reg = 0x4831 0x2000;
+   interrupts = 111;
+   };
};
 };
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html