Fix bug in io64 target (introduced by commit aba27ac)
authorVasili Galka <vvv444@gmail.com>
Mon, 16 Jun 2014 14:40:59 +0000 (17:40 +0300)
committerTom Rini <trini@ti.com>
Thu, 19 Jun 2014 15:18:37 +0000 (11:18 -0400)
From comparison of current logic and the logic that was prior to commit
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.

Cc: Stefan Roese <sr@denx.de>
Acked-by: Dirk Eibach <dirk.eibach@gdsys.cc>
Signed-off-by: Vasili Galka <vvv444@gmail.com>
board/gdsys/405ex/io64.c

index 2f8e306261815acd00e51137c31d2034acf34b4b..3a075c471f87d63df491120267367bed84702243 100644 (file)
@@ -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);