Blackfin: fix typo in isram_write()
authorRobin Getz <robin.getz@analog.com>
Thu, 27 Aug 2009 14:39:31 +0000 (14:39 +0000)
committerMike Frysinger <vapier@gentoo.org>
Thu, 17 Sep 2009 02:10:38 +0000 (22:10 -0400)
The DTEST write bit is 2, not 1.  Improve comments in the related macro
while we're here.

Signed-off-by: Robin Getz <robin.getz@analog.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
arch/blackfin/mm/isram-driver.c

index 02eaaa641d49f44df927145966beffc031656c04..beb1a608824c1f7c47225a493158477503b9ce76 100644 (file)
@@ -53,10 +53,12 @@ static DEFINE_SPINLOCK(dtest_lock);
 #define IADDR2DTEST(x) \
        ({ unsigned long __addr = (unsigned long)(x); \
                (__addr & 0x47F8)        | /* address bits 14 & 10:3 */ \
+               (__addr & 0x8000) << 23  | /* Bank A/B               */ \
                (__addr & 0x0800) << 15  | /* address bit  11        */ \
-               (__addr  & 0x3000) << 4  | /* address bits 13:12     */ \
-               (__addr  & 0x8000) << 8  | /* address bit  15        */ \
-               (0x1000004);               /* isram access           */ \
+               (__addr & 0x3000) <<  4  | /* address bits 13:12     */ \
+               (__addr & 0x8000) <<  8  | /* address bit  15        */ \
+               (0x1000000)              | /* instruction access = 1 */ \
+               (0x4);                     /* data array = 1         */ \
        })
 
 /* Takes a pointer, and returns the offset (in bits) which things should be shifted */
@@ -73,7 +75,7 @@ static void isram_write(const void *addr, uint64_t data)
        if (addr >= (void *)(L1_CODE_START + L1_CODE_LENGTH))
                return;
 
-       cmd = IADDR2DTEST(addr) | 1;             /* write */
+       cmd = IADDR2DTEST(addr) | 2;             /* write */
 
        /*
         * Writes to DTEST_DATA[0:1] need to be atomic with write to DTEST_COMMAND