Skip to content

Commit

Permalink
Blackfin: fix typo in isram_write()
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Robin Getz authored and Mike Frysinger committed Sep 17, 2009
1 parent ba877d4 commit 774b802
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions arch/blackfin/mm/isram-driver.c
Original file line number Diff line number Diff line change
Expand Up @@ -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 */
Expand All @@ -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
Expand Down

0 comments on commit 774b802

Please sign in to comment.