From: hvaibhav@ti•com (Vaibhav Hiremath)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v2 1/8] ARM/dts: OMAP2: Add McBSP entries for OMAP2420 and OMAP2430 SoC
Date: Tue, 4 Sep 2012 09:12:43 +0530 [thread overview]
Message-ID: <504578B3.3090804@ti.com> (raw)
In-Reply-To: <5044C2DD.5000705@ti.com>
On 9/3/2012 8:16 PM, Benoit Cousson wrote:
> Hi Peter,
>
> The overall series looks good to me, but I do have a couple of comments.
>
> On 08/29/2012 03:31 PM, Peter Ujfalusi wrote:
>> The McBSP IP within OMAP2420 and 2430 is different we need to create separate
>> dtsi files for them.
>>
>> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti•com>
>> ---
>> arch/arm/boot/dts/omap2420.dtsi | 39 ++++++++++++++++++
>> arch/arm/boot/dts/omap2430.dtsi | 83 +++++++++++++++++++++++++++++++++++++++
>> 2 files changed, 122 insertions(+), 0 deletions(-)
>> create mode 100644 arch/arm/boot/dts/omap2420.dtsi
>> create mode 100644 arch/arm/boot/dts/omap2430.dtsi
>>
>> diff --git a/arch/arm/boot/dts/omap2420.dtsi b/arch/arm/boot/dts/omap2420.dtsi
>> new file mode 100644
>> index 0000000..f375c68
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/omap2420.dtsi
>> @@ -0,0 +1,39 @@
>> +/*
>> + * Device Tree Source for OMAP2420 SoC
>> + *
>> + * Copyright (C) 2011 Texas Instruments Incorporated - http://www.ti.com/
>
> Nit: 2012
>
>> + *
>> + * This file is licensed under the terms of the GNU General Public License
>> + * version 2. This program is licensed "as is" without any warranty of any
>> + * kind, whether express or implied.
>> + */
>> +
>> +/include/ "omap2.dtsi"
>> +
>> +/ {
>> + compatible = "ti,omap2420", "ti,omap2";
>> +
>> + ocp {
>> + mcbsp1: mcbsp at 48074000 {
>> + compatible = "ti,omap2420-mcbsp";
>> + reg = <0x48074000 0xff>;
>> + reg-names = "mpu";
>> + interrupts = <0 59 0x4>, /* TX interrupt */
>> + <0 60 0x4>; /* RX interrupt */
>
> That one is not correct because it does comply with the interrupt
> controller specifier that require only one cell:
>
> intc: interrupt-controller at 48200000 {
> compatible = "ti,omap2-intc";
> interrupt-controller;
> #interrupt-cells = <1>;
> ...
>
> The one you are using is for GIC IRQ controller.
> It works probably because we are using hwmod so far :-)
>
I think now we should kill the resource overwrite path, and should
respect and use resources passed from DT.
Benoit,
Did you get a chance to validate patch submitted towards this??
https://patchwork.kernel.org/patch/1384351/
Thanks,
Vaibhav
> Didn't you get some warning?
>
> In the case of OMAP2 & 3, it is much simpler:
>
>> + interrupts = <59>, /* TX interrupt */
>> + <60>; /* RX interrupt */
>
> That comment is applicable for OMAP2420, OMAP2430, and OMAP3 in general.
>
>> + interrupt-names = "tx", "rx";
>> + interrupt-parent = <&intc>;
>> + ti,hwmods = "mcbsp1";
>> + };
>> +
>> + mcbsp2: mcbsp at 48076000 {
>> + compatible = "ti,omap2420-mcbsp";
>> + reg = <0x48076000 0xff>;
>> + reg-names = "mpu";
>> + interrupts = <0 62 0x4>, /* TX interrupt */
>> + <0 63 0x4>; /* RX interrupt */
>> + interrupt-names = "tx", "rx";
>> + interrupt-parent = <&intc>;
>> + ti,hwmods = "mcbsp2";
>> + };
>> + };
>> +};
>> diff --git a/arch/arm/boot/dts/omap2430.dtsi b/arch/arm/boot/dts/omap2430.dtsi
>> new file mode 100644
>> index 0000000..531e346
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/omap2430.dtsi
>> @@ -0,0 +1,83 @@
>> +/*
>> + * Device Tree Source for OMAP243x SoC
>> + *
>> + * Copyright (C) 2011 Texas Instruments Incorporated - http://www.ti.com/
>
> 2012.
>
> Regards,
> Benoit
>
> _______________________________________________
> devicetree-discuss mailing list
> devicetree-discuss at lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/devicetree-discuss
>
next prev parent reply other threads:[~2012-09-04 3:42 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-29 13:30 [PATCH v2 0/8] ARM/dts: OMAP audio related updates for OMAP2/3/4/5 Peter Ujfalusi
2012-08-29 13:31 ` [PATCH v2 1/8] ARM/dts: OMAP2: Add McBSP entries for OMAP2420 and OMAP2430 SoC Peter Ujfalusi
2012-09-03 14:46 ` Benoit Cousson
2012-09-04 3:42 ` Vaibhav Hiremath [this message]
2012-09-04 10:17 ` Benoit Cousson
2012-09-05 6:11 ` Hiremath, Vaibhav
2012-09-05 8:23 ` Peter Ujfalusi
2012-09-05 9:15 ` Hiremath, Vaibhav
2012-09-05 13:31 ` Benoit Cousson
2012-09-05 14:41 ` Hiremath, Vaibhav
2012-09-06 14:42 ` Benoit Cousson
2012-09-11 11:42 ` Hiremath, Vaibhav
2012-09-07 5:45 ` AnilKumar, Chimata
2012-09-07 8:42 ` Benoit Cousson
2012-09-05 11:14 ` Peter Ujfalusi
2012-08-29 13:31 ` [PATCH v2 2/8] ARM/dts: omap2420-h4: Include omap2420.dtsi file instead the common omap2 Peter Ujfalusi
2012-09-06 20:18 ` Tony Lindgren
2012-08-29 13:31 ` [PATCH v2 3/8] ARM/dts: OMAP3: Add McBSP entries Peter Ujfalusi
2012-08-29 13:31 ` [PATCH v2 4/8] ARM/dts: OMAP4: " Peter Ujfalusi
2012-08-29 13:31 ` [PATCH v2 5/8] ARM/dts: OMAP5: " Peter Ujfalusi
2012-08-29 13:31 ` [PATCH v2 6/8] ARM/dts: omap3-beagle: Enable audio support Peter Ujfalusi
2012-09-06 20:20 ` Tony Lindgren
2012-08-29 13:31 ` [PATCH v2 7/8] ARM/dts: omap4: Add reg-names for McPDM and DMIC Peter Ujfalusi
2012-08-29 13:31 ` [PATCH v2 8/8] ARM/dts: omap5: Add McPDM and DMIC section to the dtsi file Peter Ujfalusi
2012-09-03 14:51 ` [PATCH v2 0/8] ARM/dts: OMAP audio related updates for OMAP2/3/4/5 Benoit Cousson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=504578B3.3090804@ti.com \
--to=hvaibhav@ti$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox