Re: [RESEND PATCH V2 1/2] ARM: dts: AM33xx: Add RNG node
* 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
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
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
+ 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
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
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
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
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