diff mbox

[3/4] ARM/dts: omap5: Update UART with address space and interrupts

Message ID 1350901328-3525-4-git-send-email-s-guiriec@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sebastien Guiriec Oct. 22, 2012, 10:22 a.m. UTC
Add base address and interrupt line inside Device Tree data for

Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
---
 arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

Comments

Poddar, Sourav Oct. 22, 2012, 11:16 a.m. UTC | #1
Hi Sebastien,
On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
> Add base address and interrupt line inside Device Tree data for
Incomplete sentence!
> Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
> ---
>   arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
>   1 file changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
> index 6c22e1b..413df94 100644
> --- a/arch/arm/boot/dts/omap5.dtsi
> +++ b/arch/arm/boot/dts/omap5.dtsi
> @@ -237,36 +237,48 @@
>   
>   		uart1: serial@4806a000 {
>   			compatible = "ti,omap4-uart";
> +			reg = <0x4806a000 0x100>;
> +			interrupts = <0 72 0x4>;
>   			ti,hwmods = "uart1";
>   			clock-frequency = <48000000>;
>   		};
>   
>   		uart2: serial@4806c000 {
>   			compatible = "ti,omap4-uart";
> +			reg = <0x4806c000 0x100>;
> +			interrupts = <0 73 0x4>;
>   			ti,hwmods = "uart2";
>   			clock-frequency = <48000000>;
>   		};
>   
>   		uart3: serial@48020000 {
>   			compatible = "ti,omap4-uart";
> +			reg = <0x48020000 0x100>;
> +			interrupts = <0 74 0x4>;
>   			ti,hwmods = "uart3";
>   			clock-frequency = <48000000>;
>   		};
>   
>   		uart4: serial@4806e000 {
>   			compatible = "ti,omap4-uart";
> +			reg = <0x4806e000 0x100>;
> +			interrupts = <0 70 0x4>;
>   			ti,hwmods = "uart4";
>   			clock-frequency = <48000000>;
>   		};
>   
>   		uart5: serial@48066000 {
> -			compatible = "ti,omap5-uart";
> +			compatible = "ti,omap4-uart";
> +			reg = <0x48066000 0x100>;
> +			interrupts = <0 105 0x4>;
In Omap5 TRM, the interrupt number mentioned for uart5 is 138. How is 
105 coming?
>   			ti,hwmods = "uart5";
>   			clock-frequency = <48000000>;
>   		};
>   
>   		uart6: serial@48068000 {
> -			compatible = "ti,omap6-uart";
> +			compatible = "ti,omap4-uart";
> +			reg = <0x48068000 0x100>;
> +			interrupts = <0 106 0x4>;
Same here, TRM shows this number to be 139 ?
>   			ti,hwmods = "uart6";
>   			clock-frequency = <48000000>;
>   		};

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Benoit Cousson Oct. 22, 2012, 11:57 a.m. UTC | #2
Hi Sourav,

On 10/22/2012 01:16 PM, Sourav wrote:
> Hi Sebastien,
> On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
>> Add base address and interrupt line inside Device Tree data for
> Incomplete sentence!
>> Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
>> ---
>>   arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
>>   1 file changed, 14 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
>> index 6c22e1b..413df94 100644
>> --- a/arch/arm/boot/dts/omap5.dtsi
>> +++ b/arch/arm/boot/dts/omap5.dtsi
>> @@ -237,36 +237,48 @@
>>             uart1: serial@4806a000 {
>>               compatible = "ti,omap4-uart";
>> +            reg = <0x4806a000 0x100>;
>> +            interrupts = <0 72 0x4>;
>>               ti,hwmods = "uart1";
>>               clock-frequency = <48000000>;
>>           };
>>             uart2: serial@4806c000 {
>>               compatible = "ti,omap4-uart";
>> +            reg = <0x4806c000 0x100>;
>> +            interrupts = <0 73 0x4>;
>>               ti,hwmods = "uart2";
>>               clock-frequency = <48000000>;
>>           };
>>             uart3: serial@48020000 {
>>               compatible = "ti,omap4-uart";
>> +            reg = <0x48020000 0x100>;
>> +            interrupts = <0 74 0x4>;
>>               ti,hwmods = "uart3";
>>               clock-frequency = <48000000>;
>>           };
>>             uart4: serial@4806e000 {
>>               compatible = "ti,omap4-uart";
>> +            reg = <0x4806e000 0x100>;
>> +            interrupts = <0 70 0x4>;
>>               ti,hwmods = "uart4";
>>               clock-frequency = <48000000>;
>>           };
>>             uart5: serial@48066000 {
>> -            compatible = "ti,omap5-uart";
>> +            compatible = "ti,omap4-uart";
>> +            reg = <0x48066000 0x100>;
>> +            interrupts = <0 105 0x4>;
> In Omap5 TRM, the interrupt number mentioned for uart5 is 138. How is
> 105 coming?

It is from hwmod and thus from the HW spec. It looks like the TRM is
wrong... or the HW spec :-)

>>               ti,hwmods = "uart5";
>>               clock-frequency = <48000000>;
>>           };
>>             uart6: serial@48068000 {
>> -            compatible = "ti,omap6-uart";
>> +            compatible = "ti,omap4-uart";
>> +            reg = <0x48068000 0x100>;
>> +            interrupts = <0 106 0x4>;
> Same here, TRM shows this number to be 139 ?

Same source.

Regards,
Benoit

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Benoit Cousson Oct. 22, 2012, 12:07 p.m. UTC | #3
On 10/22/2012 01:57 PM, Benoit Cousson wrote:
> Hi Sourav,
> 
> On 10/22/2012 01:16 PM, Sourav wrote:
>> Hi Sebastien,
>> On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
>>> Add base address and interrupt line inside Device Tree data for
>> Incomplete sentence!
>>> Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
>>> ---
>>>   arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
>>>   1 file changed, 14 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
>>> index 6c22e1b..413df94 100644
>>> --- a/arch/arm/boot/dts/omap5.dtsi
>>> +++ b/arch/arm/boot/dts/omap5.dtsi
>>> @@ -237,36 +237,48 @@
>>>             uart1: serial@4806a000 {
>>>               compatible = "ti,omap4-uart";
>>> +            reg = <0x4806a000 0x100>;
>>> +            interrupts = <0 72 0x4>;
>>>               ti,hwmods = "uart1";
>>>               clock-frequency = <48000000>;
>>>           };
>>>             uart2: serial@4806c000 {
>>>               compatible = "ti,omap4-uart";
>>> +            reg = <0x4806c000 0x100>;
>>> +            interrupts = <0 73 0x4>;
>>>               ti,hwmods = "uart2";
>>>               clock-frequency = <48000000>;
>>>           };
>>>             uart3: serial@48020000 {
>>>               compatible = "ti,omap4-uart";
>>> +            reg = <0x48020000 0x100>;
>>> +            interrupts = <0 74 0x4>;
>>>               ti,hwmods = "uart3";
>>>               clock-frequency = <48000000>;
>>>           };
>>>             uart4: serial@4806e000 {
>>>               compatible = "ti,omap4-uart";
>>> +            reg = <0x4806e000 0x100>;
>>> +            interrupts = <0 70 0x4>;
>>>               ti,hwmods = "uart4";
>>>               clock-frequency = <48000000>;
>>>           };
>>>             uart5: serial@48066000 {
>>> -            compatible = "ti,omap5-uart";
>>> +            compatible = "ti,omap4-uart";
>>> +            reg = <0x48066000 0x100>;
>>> +            interrupts = <0 105 0x4>;
>> In Omap5 TRM, the interrupt number mentioned for uart5 is 138. How is
>> 105 coming?
> 
> It is from hwmod and thus from the HW spec. It looks like the TRM is
> wrong... or the HW spec :-)
> 
>>>               ti,hwmods = "uart5";
>>>               clock-frequency = <48000000>;
>>>           };
>>>             uart6: serial@48068000 {
>>> -            compatible = "ti,omap6-uart";
>>> +            compatible = "ti,omap4-uart";
>>> +            reg = <0x48068000 0x100>;
>>> +            interrupts = <0 106 0x4>;
>> Same here, TRM shows this number to be 139 ?

In fact, even the TRM (ES1.0 NDA vM) is aligned with these data.
Where did you see 138 and 139?

Regards,
Benoit



--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Poddar, Sourav Oct. 22, 2012, 12:27 p.m. UTC | #4
Hi Benoit,
On Monday 22 October 2012 05:37 PM, Benoit Cousson wrote:
> On 10/22/2012 01:57 PM, Benoit Cousson wrote:
>> Hi Sourav,
>>
>> On 10/22/2012 01:16 PM, Sourav wrote:
>>> Hi Sebastien,
>>> On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
>>>> Add base address and interrupt line inside Device Tree data for
>>> Incomplete sentence!
>>>> Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
>>>> ---
>>>>    arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
>>>>    1 file changed, 14 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
>>>> index 6c22e1b..413df94 100644
>>>> --- a/arch/arm/boot/dts/omap5.dtsi
>>>> +++ b/arch/arm/boot/dts/omap5.dtsi
>>>> @@ -237,36 +237,48 @@
>>>>              uart1: serial@4806a000 {
>>>>                compatible = "ti,omap4-uart";
>>>> +            reg = <0x4806a000 0x100>;
>>>> +            interrupts = <0 72 0x4>;
>>>>                ti,hwmods = "uart1";
>>>>                clock-frequency = <48000000>;
>>>>            };
>>>>              uart2: serial@4806c000 {
>>>>                compatible = "ti,omap4-uart";
>>>> +            reg = <0x4806c000 0x100>;
>>>> +            interrupts = <0 73 0x4>;
>>>>                ti,hwmods = "uart2";
>>>>                clock-frequency = <48000000>;
>>>>            };
>>>>              uart3: serial@48020000 {
>>>>                compatible = "ti,omap4-uart";
>>>> +            reg = <0x48020000 0x100>;
>>>> +            interrupts = <0 74 0x4>;
>>>>                ti,hwmods = "uart3";
>>>>                clock-frequency = <48000000>;
>>>>            };
>>>>              uart4: serial@4806e000 {
>>>>                compatible = "ti,omap4-uart";
>>>> +            reg = <0x4806e000 0x100>;
>>>> +            interrupts = <0 70 0x4>;
>>>>                ti,hwmods = "uart4";
>>>>                clock-frequency = <48000000>;
>>>>            };
>>>>              uart5: serial@48066000 {
>>>> -            compatible = "ti,omap5-uart";
>>>> +            compatible = "ti,omap4-uart";
>>>> +            reg = <0x48066000 0x100>;
>>>> +            interrupts = <0 105 0x4>;
>>> In Omap5 TRM, the interrupt number mentioned for uart5 is 138. How is
>>> 105 coming?
>> It is from hwmod and thus from the HW spec. It looks like the TRM is
>> wrong... or the HW spec :-)
>>
>>>>                ti,hwmods = "uart5";
>>>>                clock-frequency = <48000000>;
>>>>            };
>>>>              uart6: serial@48068000 {
>>>> -            compatible = "ti,omap6-uart";
>>>> +            compatible = "ti,omap4-uart";
>>>> +            reg = <0x48068000 0x100>;
>>>> +            interrupts = <0 106 0x4>;
>>> Same here, TRM shows this number to be 139 ?
> In fact, even the TRM (ES1.0 NDA vM) is aligned with these data.
> Where did you see 138 and 139?
I looked at Page 6300 of the above TRM, Figure 24-60. Is this place not 
correct to look
up for these data?

> Regards,
> Benoit
>
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Felipe Balbi Oct. 22, 2012, 12:31 p.m. UTC | #5
On Mon, Oct 22, 2012 at 05:57:00PM +0530, Sourav wrote:
> Hi Benoit,
> On Monday 22 October 2012 05:37 PM, Benoit Cousson wrote:
> >On 10/22/2012 01:57 PM, Benoit Cousson wrote:
> >>Hi Sourav,
> >>
> >>On 10/22/2012 01:16 PM, Sourav wrote:
> >>>Hi Sebastien,
> >>>On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
> >>>>Add base address and interrupt line inside Device Tree data for
> >>>Incomplete sentence!
> >>>>Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
> >>>>---
> >>>>   arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
> >>>>   1 file changed, 14 insertions(+), 2 deletions(-)
> >>>>
> >>>>diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
> >>>>index 6c22e1b..413df94 100644
> >>>>--- a/arch/arm/boot/dts/omap5.dtsi
> >>>>+++ b/arch/arm/boot/dts/omap5.dtsi
> >>>>@@ -237,36 +237,48 @@
> >>>>             uart1: serial@4806a000 {
> >>>>               compatible = "ti,omap4-uart";
> >>>>+            reg = <0x4806a000 0x100>;
> >>>>+            interrupts = <0 72 0x4>;
> >>>>               ti,hwmods = "uart1";
> >>>>               clock-frequency = <48000000>;
> >>>>           };
> >>>>             uart2: serial@4806c000 {
> >>>>               compatible = "ti,omap4-uart";
> >>>>+            reg = <0x4806c000 0x100>;
> >>>>+            interrupts = <0 73 0x4>;
> >>>>               ti,hwmods = "uart2";
> >>>>               clock-frequency = <48000000>;
> >>>>           };
> >>>>             uart3: serial@48020000 {
> >>>>               compatible = "ti,omap4-uart";
> >>>>+            reg = <0x48020000 0x100>;
> >>>>+            interrupts = <0 74 0x4>;
> >>>>               ti,hwmods = "uart3";
> >>>>               clock-frequency = <48000000>;
> >>>>           };
> >>>>             uart4: serial@4806e000 {
> >>>>               compatible = "ti,omap4-uart";
> >>>>+            reg = <0x4806e000 0x100>;
> >>>>+            interrupts = <0 70 0x4>;
> >>>>               ti,hwmods = "uart4";
> >>>>               clock-frequency = <48000000>;
> >>>>           };
> >>>>             uart5: serial@48066000 {
> >>>>-            compatible = "ti,omap5-uart";
> >>>>+            compatible = "ti,omap4-uart";
> >>>>+            reg = <0x48066000 0x100>;
> >>>>+            interrupts = <0 105 0x4>;
> >>>In Omap5 TRM, the interrupt number mentioned for uart5 is 138. How is
> >>>105 coming?
> >>It is from hwmod and thus from the HW spec. It looks like the TRM is
> >>wrong... or the HW spec :-)
> >>
> >>>>               ti,hwmods = "uart5";
> >>>>               clock-frequency = <48000000>;
> >>>>           };
> >>>>             uart6: serial@48068000 {
> >>>>-            compatible = "ti,omap6-uart";
> >>>>+            compatible = "ti,omap4-uart";
> >>>>+            reg = <0x48068000 0x100>;
> >>>>+            interrupts = <0 106 0x4>;
> >>>Same here, TRM shows this number to be 139 ?
> >In fact, even the TRM (ES1.0 NDA vM) is aligned with these data.
> >Where did you see 138 and 139?
> I looked at Page 6300 of the above TRM, Figure 24-60. Is this place

correct page is 6366
Benoit Cousson Oct. 22, 2012, 12:50 p.m. UTC | #6
On 10/22/2012 02:27 PM, Sourav wrote:
> Hi Benoit,
> On Monday 22 October 2012 05:37 PM, Benoit Cousson wrote:
>> On 10/22/2012 01:57 PM, Benoit Cousson wrote:
>>> Hi Sourav,
>>>
>>> On 10/22/2012 01:16 PM, Sourav wrote:
>>>> Hi Sebastien,
>>>> On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
>>>>> Add base address and interrupt line inside Device Tree data for
>>>> Incomplete sentence!
>>>>> Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
>>>>> ---
>>>>>    arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
>>>>>    1 file changed, 14 insertions(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/arch/arm/boot/dts/omap5.dtsi
>>>>> b/arch/arm/boot/dts/omap5.dtsi
>>>>> index 6c22e1b..413df94 100644
>>>>> --- a/arch/arm/boot/dts/omap5.dtsi
>>>>> +++ b/arch/arm/boot/dts/omap5.dtsi
>>>>> @@ -237,36 +237,48 @@
>>>>>              uart1: serial@4806a000 {
>>>>>                compatible = "ti,omap4-uart";
>>>>> +            reg = <0x4806a000 0x100>;
>>>>> +            interrupts = <0 72 0x4>;
>>>>>                ti,hwmods = "uart1";
>>>>>                clock-frequency = <48000000>;
>>>>>            };
>>>>>              uart2: serial@4806c000 {
>>>>>                compatible = "ti,omap4-uart";
>>>>> +            reg = <0x4806c000 0x100>;
>>>>> +            interrupts = <0 73 0x4>;
>>>>>                ti,hwmods = "uart2";
>>>>>                clock-frequency = <48000000>;
>>>>>            };
>>>>>              uart3: serial@48020000 {
>>>>>                compatible = "ti,omap4-uart";
>>>>> +            reg = <0x48020000 0x100>;
>>>>> +            interrupts = <0 74 0x4>;
>>>>>                ti,hwmods = "uart3";
>>>>>                clock-frequency = <48000000>;
>>>>>            };
>>>>>              uart4: serial@4806e000 {
>>>>>                compatible = "ti,omap4-uart";
>>>>> +            reg = <0x4806e000 0x100>;
>>>>> +            interrupts = <0 70 0x4>;
>>>>>                ti,hwmods = "uart4";
>>>>>                clock-frequency = <48000000>;
>>>>>            };
>>>>>              uart5: serial@48066000 {
>>>>> -            compatible = "ti,omap5-uart";
>>>>> +            compatible = "ti,omap4-uart";
>>>>> +            reg = <0x48066000 0x100>;
>>>>> +            interrupts = <0 105 0x4>;
>>>> In Omap5 TRM, the interrupt number mentioned for uart5 is 138. How is
>>>> 105 coming?
>>> It is from hwmod and thus from the HW spec. It looks like the TRM is
>>> wrong... or the HW spec :-)
>>>
>>>>>                ti,hwmods = "uart5";
>>>>>                clock-frequency = <48000000>;
>>>>>            };
>>>>>              uart6: serial@48068000 {
>>>>> -            compatible = "ti,omap6-uart";
>>>>> +            compatible = "ti,omap4-uart";
>>>>> +            reg = <0x48068000 0x100>;
>>>>> +            interrupts = <0 106 0x4>;
>>>> Same here, TRM shows this number to be 139 ?
>> In fact, even the TRM (ES1.0 NDA vM) is aligned with these data.
>> Where did you see 138 and 139?
> I looked at Page 6300 of the above TRM, Figure 24-60. Is this place not
> correct to look
> up for these data?

Nope. Well it should be accurate but since it is a diagram, it does not
necessarily reflect the latest integration data like IRQ line.

You'd better use the table that list all the IRQ per CPU:

18.3.2 Interrupt Requests to INTC_MPU

Regards,
Benoit

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Poddar, Sourav Oct. 22, 2012, 12:56 p.m. UTC | #7
On Monday 22 October 2012 06:20 PM, Benoit Cousson wrote:
> On 10/22/2012 02:27 PM, Sourav wrote:
>> Hi Benoit,
>> On Monday 22 October 2012 05:37 PM, Benoit Cousson wrote:
>>> On 10/22/2012 01:57 PM, Benoit Cousson wrote:
>>>> Hi Sourav,
>>>>
>>>> On 10/22/2012 01:16 PM, Sourav wrote:
>>>>> Hi Sebastien,
>>>>> On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
>>>>>> Add base address and interrupt line inside Device Tree data for
>>>>> Incomplete sentence!
>>>>>> Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
>>>>>> ---
>>>>>>     arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
>>>>>>     1 file changed, 14 insertions(+), 2 deletions(-)
>>>>>>
>>>>>> diff --git a/arch/arm/boot/dts/omap5.dtsi
>>>>>> b/arch/arm/boot/dts/omap5.dtsi
>>>>>> index 6c22e1b..413df94 100644
>>>>>> --- a/arch/arm/boot/dts/omap5.dtsi
>>>>>> +++ b/arch/arm/boot/dts/omap5.dtsi
>>>>>> @@ -237,36 +237,48 @@
>>>>>>               uart1: serial@4806a000 {
>>>>>>                 compatible = "ti,omap4-uart";
>>>>>> +            reg = <0x4806a000 0x100>;
>>>>>> +            interrupts = <0 72 0x4>;
>>>>>>                 ti,hwmods = "uart1";
>>>>>>                 clock-frequency = <48000000>;
>>>>>>             };
>>>>>>               uart2: serial@4806c000 {
>>>>>>                 compatible = "ti,omap4-uart";
>>>>>> +            reg = <0x4806c000 0x100>;
>>>>>> +            interrupts = <0 73 0x4>;
>>>>>>                 ti,hwmods = "uart2";
>>>>>>                 clock-frequency = <48000000>;
>>>>>>             };
>>>>>>               uart3: serial@48020000 {
>>>>>>                 compatible = "ti,omap4-uart";
>>>>>> +            reg = <0x48020000 0x100>;
>>>>>> +            interrupts = <0 74 0x4>;
>>>>>>                 ti,hwmods = "uart3";
>>>>>>                 clock-frequency = <48000000>;
>>>>>>             };
>>>>>>               uart4: serial@4806e000 {
>>>>>>                 compatible = "ti,omap4-uart";
>>>>>> +            reg = <0x4806e000 0x100>;
>>>>>> +            interrupts = <0 70 0x4>;
>>>>>>                 ti,hwmods = "uart4";
>>>>>>                 clock-frequency = <48000000>;
>>>>>>             };
>>>>>>               uart5: serial@48066000 {
>>>>>> -            compatible = "ti,omap5-uart";
>>>>>> +            compatible = "ti,omap4-uart";
>>>>>> +            reg = <0x48066000 0x100>;
>>>>>> +            interrupts = <0 105 0x4>;
>>>>> In Omap5 TRM, the interrupt number mentioned for uart5 is 138. How is
>>>>> 105 coming?
>>>> It is from hwmod and thus from the HW spec. It looks like the TRM is
>>>> wrong... or the HW spec :-)
>>>>
>>>>>>                 ti,hwmods = "uart5";
>>>>>>                 clock-frequency = <48000000>;
>>>>>>             };
>>>>>>               uart6: serial@48068000 {
>>>>>> -            compatible = "ti,omap6-uart";
>>>>>> +            compatible = "ti,omap4-uart";
>>>>>> +            reg = <0x48068000 0x100>;
>>>>>> +            interrupts = <0 106 0x4>;
>>>>> Same here, TRM shows this number to be 139 ?
>>> In fact, even the TRM (ES1.0 NDA vM) is aligned with these data.
>>> Where did you see 138 and 139?
>> I looked at Page 6300 of the above TRM, Figure 24-60. Is this place not
>> correct to look
>> up for these data?
> Nope. Well it should be accurate but since it is a diagram, it does not
> necessarily reflect the latest integration data like IRQ line.
>
> You'd better use the table that list all the IRQ per CPU:
>
> 18.3.2 Interrupt Requests to INTC_MPU
Ahh. True. The table does show the numbers to be 105 and 106.
Thanks for the pointer.

~Sourav
> Regards,
> Benoit
>

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Poddar, Sourav Oct. 22, 2012, 1:55 p.m. UTC | #8
Hi Sebestien,
On Monday 22 October 2012 06:26 PM, Sourav wrote:
> On Monday 22 October 2012 06:20 PM, Benoit Cousson wrote:
>> On 10/22/2012 02:27 PM, Sourav wrote:
>>> Hi Benoit,
>>> On Monday 22 October 2012 05:37 PM, Benoit Cousson wrote:
>>>> On 10/22/2012 01:57 PM, Benoit Cousson wrote:
>>>>> Hi Sourav,
>>>>>
>>>>> On 10/22/2012 01:16 PM, Sourav wrote:
>>>>>> Hi Sebastien,
>>>>>> On Monday 22 October 2012 03:52 PM, Sebastien Guiriec wrote:
>>>>>>> Add base address and interrupt line inside Device Tree data for
>>>>>> Incomplete sentence!
>>>>>>> Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com>
>>>>>>> ---
>>>>>>>     arch/arm/boot/dts/omap5.dtsi |   16 ++++++++++++++--
>>>>>>>     1 file changed, 14 insertions(+), 2 deletions(-)
>>>>>>>
>>>>>>> diff --git a/arch/arm/boot/dts/omap5.dtsi
>>>>>>> b/arch/arm/boot/dts/omap5.dtsi
>>>>>>> index 6c22e1b..413df94 100644
>>>>>>> --- a/arch/arm/boot/dts/omap5.dtsi
>>>>>>> +++ b/arch/arm/boot/dts/omap5.dtsi
>>>>>>> @@ -237,36 +237,48 @@
>>>>>>>               uart1: serial@4806a000 {
>>>>>>>                 compatible = "ti,omap4-uart";
>>>>>>> +            reg = <0x4806a000 0x100>;
>>>>>>> +            interrupts = <0 72 0x4>;
>>>>>>>                 ti,hwmods = "uart1";
>>>>>>>                 clock-frequency = <48000000>;
>>>>>>>             };
>>>>>>>               uart2: serial@4806c000 {
>>>>>>>                 compatible = "ti,omap4-uart";
>>>>>>> +            reg = <0x4806c000 0x100>;
>>>>>>> +            interrupts = <0 73 0x4>;
>>>>>>>                 ti,hwmods = "uart2";
>>>>>>>                 clock-frequency = <48000000>;
>>>>>>>             };
>>>>>>>               uart3: serial@48020000 {
>>>>>>>                 compatible = "ti,omap4-uart";
>>>>>>> +            reg = <0x48020000 0x100>;
>>>>>>> +            interrupts = <0 74 0x4>;
>>>>>>>                 ti,hwmods = "uart3";
>>>>>>>                 clock-frequency = <48000000>;
>>>>>>>             };
>>>>>>>               uart4: serial@4806e000 {
>>>>>>>                 compatible = "ti,omap4-uart";
>>>>>>> +            reg = <0x4806e000 0x100>;
>>>>>>> +            interrupts = <0 70 0x4>;
>>>>>>>                 ti,hwmods = "uart4";
>>>>>>>                 clock-frequency = <48000000>;
>>>>>>>             };
>>>>>>>               uart5: serial@48066000 {
>>>>>>> -            compatible = "ti,omap5-uart";
>>>>>>> +            compatible = "ti,omap4-uart";
>>>>>>> +            reg = <0x48066000 0x100>;
>>>>>>> +            interrupts = <0 105 0x4>;
>>>>>> In Omap5 TRM, the interrupt number mentioned for uart5 is 138. 
>>>>>> How is
>>>>>> 105 coming?
>>>>> It is from hwmod and thus from the HW spec. It looks like the TRM is
>>>>> wrong... or the HW spec :-)
>>>>>
>>>>>>>                 ti,hwmods = "uart5";
>>>>>>>                 clock-frequency = <48000000>;
>>>>>>>             };
>>>>>>>               uart6: serial@48068000 {
>>>>>>> -            compatible = "ti,omap6-uart";
>>>>>>> +            compatible = "ti,omap4-uart";
>>>>>>> +            reg = <0x48068000 0x100>;
>>>>>>> +            interrupts = <0 106 0x4>;
>>>>>> Same here, TRM shows this number to be 139 ?
>>>> In fact, even the TRM (ES1.0 NDA vM) is aligned with these data.
>>>> Where did you see 138 and 139?
>>> I looked at Page 6300 of the above TRM, Figure 24-60. Is this place not
>>> correct to look
>>> up for these data?
>> Nope. Well it should be accurate but since it is a diagram, it does not
>> necessarily reflect the latest integration data like IRQ line.
>>
>> You'd better use the table that list all the IRQ per CPU:
>>
>> 18.3.2 Interrupt Requests to INTC_MPU
> Ahh. True. The table does show the numbers to be 105 and 106.
> Thanks for the pointer.
>
> ~Sourav
>> Regards,
>> Benoit
>>
>
After fixing the minor comment on the commit log, you can add

Acked-by: Sourav Poddar <sourav.poddar@ti.com>

>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

Patch

diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
index 6c22e1b..413df94 100644
--- a/arch/arm/boot/dts/omap5.dtsi
+++ b/arch/arm/boot/dts/omap5.dtsi
@@ -237,36 +237,48 @@ 
 
 		uart1: serial@4806a000 {
 			compatible = "ti,omap4-uart";
+			reg = <0x4806a000 0x100>;
+			interrupts = <0 72 0x4>;
 			ti,hwmods = "uart1";
 			clock-frequency = <48000000>;
 		};
 
 		uart2: serial@4806c000 {
 			compatible = "ti,omap4-uart";
+			reg = <0x4806c000 0x100>;
+			interrupts = <0 73 0x4>;
 			ti,hwmods = "uart2";
 			clock-frequency = <48000000>;
 		};
 
 		uart3: serial@48020000 {
 			compatible = "ti,omap4-uart";
+			reg = <0x48020000 0x100>;
+			interrupts = <0 74 0x4>;
 			ti,hwmods = "uart3";
 			clock-frequency = <48000000>;
 		};
 
 		uart4: serial@4806e000 {
 			compatible = "ti,omap4-uart";
+			reg = <0x4806e000 0x100>;
+			interrupts = <0 70 0x4>;
 			ti,hwmods = "uart4";
 			clock-frequency = <48000000>;
 		};
 
 		uart5: serial@48066000 {
-			compatible = "ti,omap5-uart";
+			compatible = "ti,omap4-uart";
+			reg = <0x48066000 0x100>;
+			interrupts = <0 105 0x4>;
 			ti,hwmods = "uart5";
 			clock-frequency = <48000000>;
 		};
 
 		uart6: serial@48068000 {
-			compatible = "ti,omap6-uart";
+			compatible = "ti,omap4-uart";
+			reg = <0x48068000 0x100>;
+			interrupts = <0 106 0x4>;
 			ti,hwmods = "uart6";
 			clock-frequency = <48000000>;
 		};