[PATCH v3] davinci: Add additional JTAG code for AM-1808 and OMAP-L138 Rev 2.0 SoCs

[PATCH v3] davinci: Add additional JTAG code for AM-1808 and OMAP-L138 Rev 2.0 SoCs

[Michael Williamson]

From: Sudhakar Rajashekhara <sudhakar.raj@ti.com>

The JTAG variant code for Rev-2.0 silicon of the OMAP-L138 has changed.
In addition, the variant code for the AM-1808 SoC appears to match
the Rev-2.0 code for the OMAP-L138.  Add an additional entry to support
these chips.

This patch is originally from a patch on the arago project, here:
http://arago-project.org/git/projects/?p=linux-omapl1.git;a=commit;h=6157618435e313a444cdf059702bd34036a6e2b7

Further information related to the need for this patch can be located at
http://e2e.ti.com/support/embedded/f/354/p/67290/248486.aspx
http://linux.davincidsp.com/pipermail/davinci-linux-open-source/2010-November/021224.html

This patch was tested using an AM-1808 SoC on a MityARM-1808 SoM card.  It
was also tested using a Rev 1.0 silicon OMAP-L138 on a MityDSP-L138F card.

Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
Signed-off-by: Michael Williamson <michael.williamson@criticallink.com>
Tested-by: Michael Williamson <michael.williamson@criticallink.com>
Reported-by: Nicolas Luna <luna.id@gmail.com>
---
This is against davinci-linux tree.
Changes since v2:
 - Added Sudhakar's sign-off.
 - Fix spelling error in description.

 arch/arm/mach-davinci/da850.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
index 78b5ae2..2d0bba5 100644
--- a/arch/arm/mach-davinci/da850.c
+++ b/arch/arm/mach-davinci/da850.c
@@ -764,6 +764,13 @@ static struct davinci_id da850_ids[] = {
 		.cpu_id		= DAVINCI_CPU_ID_DA850,
 		.name		= "da850/omap-l138",
 	},
+	{
+		.variant	= 0x1,
+		.part_no	= 0xb7d1,
+		.manufacturer	= 0x017,	/* 0x02f >> 1 */
+		.cpu_id		= DAVINCI_CPU_ID_DA850,
+		.name		= "da850/omap-l138/am18xx",
+	},
 };
 
 static struct davinci_timer_instance da850_timer_instance[4] = {
-- 
1.7.0.4

[PATCH v3] davinci: Add additional JTAG code for AM-1808 and OMAP-L138 Rev 2.0 SoCs

[Nori, Sekhar]

On Mon, Dec 27, 2010 at 01:26:56, Michael Williamson wrote:
> From: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
>
> The JTAG variant code for Rev-2.0 silicon of the OMAP-L138 has changed.
> In addition, the variant code for the AM-1808 SoC appears to match
> the Rev-2.0 code for the OMAP-L138.  Add an additional entry to support
> these chips.
>
> This patch is originally from a patch on the arago project, here:
> http://arago-project.org/git/projects/?p=linux-omapl1.git;a=commit;h=6157618435e313a444cdf059702bd34036a6e2b7
>
> Further information related to the need for this patch can be located at
> http://e2e.ti.com/support/embedded/f/354/p/67290/248486.aspx
> http://linux.davincidsp.com/pipermail/davinci-linux-open-source/2010-November/021224.html
>
> This patch was tested using an AM-1808 SoC on a MityARM-1808 SoM card.  It
> was also tested using a Rev 1.0 silicon OMAP-L138 on a MityDSP-L138F card.
>
> Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
> Signed-off-by: Michael Williamson <michael.williamson@criticallink.com>
> Tested-by: Michael Williamson <michael.williamson@criticallink.com>
> Reported-by: Nicolas Luna <luna.id@gmail.com>
> ---
> This is against davinci-linux tree.
> Changes since v2:
>  - Added Sudhakar's sign-off.
>  - Fix spelling error in description.
>
>  arch/arm/mach-davinci/da850.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
> index 78b5ae2..2d0bba5 100644
> --- a/arch/arm/mach-davinci/da850.c
> +++ b/arch/arm/mach-davinci/da850.c
> @@ -764,6 +764,13 @@ static struct davinci_id da850_ids[] = {
>               .cpu_id         = DAVINCI_CPU_ID_DA850,
>               .name           = "da850/omap-l138",

Should have "/am18x" here. That should have been part
of my patches adding AM1x support. It should now be a
separate patch.

>       },
> +     {
> +             .variant        = 0x1,
> +             .part_no        = 0xb7d1,
> +             .manufacturer   = 0x017,        /* 0x02f >> 1 */
> +             .cpu_id         = DAVINCI_CPU_ID_DA850,
> +             .name           = "da850/omap-l138/am18xx",

Just "am18x".

Thanks,
Sekhar

[PATCH v3] davinci: Add additional JTAG code for AM-1808 and OMAP-L138 Rev 2.0 SoCs

[Michael Williamson]

On 12/27/2010 01:03 AM, Nori, Sekhar wrote:
> On Mon, Dec 27, 2010 at 01:26:56, Michael Williamson wrote:
>> From: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
>>
>> The JTAG variant code for Rev-2.0 silicon of the OMAP-L138 has changed.
>> In addition, the variant code for the AM-1808 SoC appears to match
>> the Rev-2.0 code for the OMAP-L138.  Add an additional entry to support
>> these chips.
>>
>> This patch is originally from a patch on the arago project, here:
>> http://arago-project.org/git/projects/?p=linux-omapl1.git;a=commit;h=6157618435e313a444cdf059702bd34036a6e2b7
>>
>> Further information related to the need for this patch can be located at
>> http://e2e.ti.com/support/embedded/f/354/p/67290/248486.aspx
>> http://linux.davincidsp.com/pipermail/davinci-linux-open-source/2010-November/021224.html
>>
>> This patch was tested using an AM-1808 SoC on a MityARM-1808 SoM card.  It
>> was also tested using a Rev 1.0 silicon OMAP-L138 on a MityDSP-L138F card.
>>
>> Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
>> Signed-off-by: Michael Williamson <michael.williamson@criticallink.com>
>> Tested-by: Michael Williamson <michael.williamson@criticallink.com>
>> Reported-by: Nicolas Luna <luna.id@gmail.com>
>> ---
>> This is against davinci-linux tree.
>> Changes since v2:
>>  - Added Sudhakar's sign-off.
>>  - Fix spelling error in description.
>>
>>  arch/arm/mach-davinci/da850.c |    7 +++++++
>>  1 files changed, 7 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c
>> index 78b5ae2..2d0bba5 100644
>> --- a/arch/arm/mach-davinci/da850.c
>> +++ b/arch/arm/mach-davinci/da850.c
>> @@ -764,6 +764,13 @@ static struct davinci_id da850_ids[] = {
>>               .cpu_id         = DAVINCI_CPU_ID_DA850,
>>               .name           = "da850/omap-l138",
> 
> Should have "/am18x" here. That should have been part
> of my patches adding AM1x support. It should now be a
> separate patch.
> 

I will add this.

>>       },
>> +     {
>> +             .variant        = 0x1,
>> +             .part_no        = 0xb7d1,
>> +             .manufacturer   = 0x017,        /* 0x02f >> 1 */
>> +             .cpu_id         = DAVINCI_CPU_ID_DA850,
>> +             .name           = "da850/omap-l138/am18xx",
> 
> Just "am18x".
> 

I will correct and resubmit.  Thanks.

-Mike