No subject

PATCH [email protected] 1 participant 1 message May 30, 2014

Share & tools

[email protected]

author

2014-05-30 07:51 UTC #0 lore raw

aba27ac, we see that first parameter of FPGA_GET_REG() shall be the FPGA index and not channel number. The re-factoring in commit aba27ac accidentally changed that.

Signed-off-by Vasili Galka <[email protected]>

Cc Dirk Eibach <[email protected]>, Stefan Roese <[email protected]>

---
 board/gdsys/405ex/io64.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

board/gdsys/405ex/io64.c +2 -2

index 2f8e306..3a075c4 100644
@@ -287,7 +287,7 @@ int last_stage_init(void)
 	for (fpga = 0; fpga < 2; ++fpga) {
 		for (k = 0; k < 32; ++k) {
 			u16 status;
-			FPGA_GET_REG(k, ch[k].status_int, &status);
+			FPGA_GET_REG(fpga, ch[k].status_int, &status);
 			if (!(status & (1 << 4))) {
 				failed = 1;
 				printf("fpga %d channel %d: no serdes lock\n",

@@ -304,7 +304,7 @@ int last_stage_init(void)
 	for (fpga = 0; fpga < 2; ++fpga) {
 		for (k = 0; k < 32; ++k) {
 			u16 status;
-			FPGA_GET_REG(k, hicb_ch[k].status_int, &status);
+			FPGA_GET_REG(fpga, hicb_ch[k].status_int, &status);
 			if (status)
 				printf("fpga %d hicb %d: hicb status %04x\n",
 					fpga, k, status);

Signature · 1 line

1.7.9