This is an automated email from Gerrit.

Uwe Hermann (u...@hermann-uwe.de) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/136

-- gerrit

commit 6ada46d8639bb44800e526111291172506122c0d
Author: Uwe Hermann <u...@hermann-uwe.de>
Date:   Sat Oct 29 22:58:12 2011 +0200

    config files: Drop incorrect comments.
    
    There are many "force an error till we get a good number" comments in
    target/board files. This refers to the use-case where a config script
    sets _CPUTAPID to 0xffffffff (which presumely gets overridden later):
    
     if { [info exists CPUTAPID ] } {
        set _CPUTAPID $CPUTAPID
     } else {
        # Force an error until we get a good number.
        set _CPUTAPID 0xffffffff
     }
    
    However, the same comment was also copy-pasted in many files which do
    _not_ set _CPUTAPID to 0xffffffff, where the comment doesn't make any
    sense at all. Drop those comments. Also, add one missing comment, and
    fix small whitespace and grammar issues.
    
    Change-Id: Ic4ba3b5ccba87ed40cea0d6a7d66609fbdfa3c71
    Signed-off-by: Uwe Hermann <u...@hermann-uwe.de>

diff --git a/tcl/board/at91eb40a.cfg b/tcl/board/at91eb40a.cfg
index dc5aacb..b43dee3 100644
--- a/tcl/board/at91eb40a.cfg
+++ b/tcl/board/at91eb40a.cfg
@@ -19,7 +19,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x1f0f0f0f
 }
 
diff --git a/tcl/board/mini2440.cfg b/tcl/board/mini2440.cfg
index c5adac3..a034538 100644
--- a/tcl/board/mini2440.cfg
+++ b/tcl/board/mini2440.cfg
@@ -100,7 +100,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x0032409d
 }
 
diff --git a/tcl/target/at91sam7se512.cfg b/tcl/target/at91sam7se512.cfg
index c48afef..7273a06 100644
--- a/tcl/target/at91sam7se512.cfg
+++ b/tcl/target/at91sam7se512.cfg
@@ -18,7 +18,7 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
+   # Force an error until we get a good number.
    set _CPUTAPID 0xffffffff
 }
 
diff --git a/tcl/target/at91sam9.cfg b/tcl/target/at91sam9.cfg
index ba0197d..08154ce 100644
--- a/tcl/target/at91sam9.cfg
+++ b/tcl/target/at91sam9.cfg
@@ -17,13 +17,11 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
        set _CPUTAPID $CPUTAPID
 } else {
-       # force an error till we get a good number
        set _CPUTAPID 0x0792603f
 }
 
 reset_config trst_and_srst separate trst_push_pull srst_open_drain
 
-#
 jtag newtap $_CHIPNAME cpu -irlen 4 -ircapture 0x1 -irmask 0xf -expected-id 
$_CPUTAPID
 
 adapter_nsrst_delay 300
diff --git a/tcl/target/avr32.cfg b/tcl/target/avr32.cfg
index 7fe98b2..1a2ea33 100644
--- a/tcl/target/avr32.cfg
+++ b/tcl/target/avr32.cfg
@@ -1,7 +1,6 @@
 set  _CHIPNAME avr32
 set  _ENDIAN big
 
-# force an error till we get a good number
 set _CPUTAPID  0x21e8203f
 
 jtag_nsrst_delay 100
diff --git a/tcl/target/dsp56321.cfg b/tcl/target/dsp56321.cfg
index 281c4dd..74279d9 100644
--- a/tcl/target/dsp56321.cfg
+++ b/tcl/target/dsp56321.cfg
@@ -17,7 +17,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x1181501d
 }
 
diff --git a/tcl/target/dsp568013.cfg b/tcl/target/dsp568013.cfg
index 80adc76..2b25ed7 100644
--- a/tcl/target/dsp568013.cfg
+++ b/tcl/target/dsp568013.cfg
@@ -16,7 +16,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x01f2401d
 }
 
diff --git a/tcl/target/dsp568037.cfg b/tcl/target/dsp568037.cfg
index 6089e9d..21eb0b7 100644
--- a/tcl/target/dsp568037.cfg
+++ b/tcl/target/dsp568037.cfg
@@ -16,7 +16,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x01f2801d
 }
 
diff --git a/tcl/target/epc9301.cfg b/tcl/target/epc9301.cfg
index d2dc7ec..6b67b9d 100644
--- a/tcl/target/epc9301.cfg
+++ b/tcl/target/epc9301.cfg
@@ -15,7 +15,7 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
+   # Force an error until we get a good number.
    set _CPUTAPID 0xffffffff
 }
 
diff --git a/tcl/target/is5114.cfg b/tcl/target/is5114.cfg
index 64c0c8e..56b9b65 100644
--- a/tcl/target/is5114.cfg
+++ b/tcl/target/is5114.cfg
@@ -18,7 +18,7 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
+   # Force an error until we get a good number.
    set _CPUTAPID 0xffffffff
 }
 
diff --git a/tcl/target/pic32mx.cfg b/tcl/target/pic32mx.cfg
index efffd04..0b3522e 100644
--- a/tcl/target/pic32mx.cfg
+++ b/tcl/target/pic32mx.cfg
@@ -14,7 +14,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-   # force an error till we get a good number
    set _CPUTAPID 0x30938053
 }
 
diff --git a/tcl/target/samsung_s3c2410.cfg b/tcl/target/samsung_s3c2410.cfg
index 9682c4c..73d3b50 100644
--- a/tcl/target/samsung_s3c2410.cfg
+++ b/tcl/target/samsung_s3c2410.cfg
@@ -16,6 +16,7 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID] } {
    set  _CPUTAPID $CPUTAPID
 } else {
+   # Force an error until we get a good number.
    set  _CPUTAPID 0xffffffff
 }
 
diff --git a/tcl/target/samsung_s3c2440.cfg b/tcl/target/samsung_s3c2440.cfg
index d3b665f..9cd27d1 100644
--- a/tcl/target/samsung_s3c2440.cfg
+++ b/tcl/target/samsung_s3c2440.cfg
@@ -19,7 +19,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x0032409d
 }
 
diff --git a/tcl/target/samsung_s3c2450.cfg b/tcl/target/samsung_s3c2450.cfg
index 0075426..164fe16 100644
--- a/tcl/target/samsung_s3c2450.cfg
+++ b/tcl/target/samsung_s3c2450.cfg
@@ -29,7 +29,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
   set _CPUTAPID $CPUTAPID
 } else {
- # force an error till we get a good number
   set _CPUTAPID 0x07926f0f
 }
 
diff --git a/tcl/target/samsung_s3c6410.cfg b/tcl/target/samsung_s3c6410.cfg
index f9738c2..eb95242 100644
--- a/tcl/target/samsung_s3c6410.cfg
+++ b/tcl/target/samsung_s3c6410.cfg
@@ -23,14 +23,12 @@ if { [info exists ENDIAN] } {
 if { [info exists ETBTAPID ] } {
    set _ETBTAPID $ETBTAPID
 } else {
-  # force an error till we get a good number
    set _ETBTAPID 0x2b900f0f
 }
 
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x07b76f0f
 }
 
diff --git a/tcl/target/smp8634.cfg b/tcl/target/smp8634.cfg
index 4f3959d..dc9bf0b 100644
--- a/tcl/target/smp8634.cfg
+++ b/tcl/target/smp8634.cfg
@@ -15,7 +15,6 @@ if { [info exists ENDIAN] } {
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-   # force an error till we get a good number
    set _CPUTAPID 0x08630001
 }
 
diff --git a/tcl/target/tmpa900.cfg b/tcl/target/tmpa900.cfg
index cfb445e..dd5d177 100644
--- a/tcl/target/tmpa900.cfg
+++ b/tcl/target/tmpa900.cfg
@@ -14,7 +14,6 @@ set _ENDIAN little
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x07926031
 }
 
diff --git a/tcl/target/tmpa910.cfg b/tcl/target/tmpa910.cfg
index 4d4d3fc..271ca57 100644
--- a/tcl/target/tmpa910.cfg
+++ b/tcl/target/tmpa910.cfg
@@ -14,7 +14,6 @@ set _ENDIAN little
 if { [info exists CPUTAPID ] } {
    set _CPUTAPID $CPUTAPID
 } else {
-  # force an error till we get a good number
    set _CPUTAPID 0x07926031
 }
 

-- 
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to