Hey Lucas,

> Seems you imported this patch from somewhere and it left some traces in
> the commit message?

yes, sorry, my local mirror. I will fix that.

> Also I don't understand what this change is supposed to be doing. You
> are building just another Barebox binary, with no real differences in
> the configuration. I would much prefer a proper 2-stage loading in the
> PBL, but that requires FAT support for the SDcard boot, which I didn't
> get around to take a look at yet.

Of course, one would need a seperate configuration file. I'm using 
"zynq_fsbl_defconfig" for the small one and "zynq_defconfig" for the bigger one 
for my own tests.
I'm currently booting from SDcard and i have not realised any problems with the 
FAT support so far. But with the arasan-sdhci driver when writing, see my patch 
below which fixes it for me but I haven't dived into the drivers very deeply 
yet.

best regards
Michael


 drivers/mci/arasan-sdhci.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/mci/arasan-sdhci.c b/drivers/mci/arasan-sdhci.c
index 520bf30ff..669675369 100644
--- a/drivers/mci/arasan-sdhci.c
+++ b/drivers/mci/arasan-sdhci.c
@@ -265,7 +265,7 @@ static int arasan_sdhci_send_cmd(struct mci_host *mci, 
struct mci_cmd *cmd,
        if (cmd->cmdidx != MMC_CMD_STOP_TRANSMISSION)
                mask |= SDHCI_CMD_INHIBIT_DATA;
 
-       ret = wait_on_timeout(10 * MSECOND,
+       ret = wait_on_timeout(100 * MSECOND,
                        !(sdhci_read32(&host->sdhci, SDHCI_PRESENT_STATE) & 
mask));
 
        if (ret) {
@@ -277,8 +277,6 @@ static int arasan_sdhci_send_cmd(struct mci_host *mci, 
struct mci_cmd *cmd,
        sdhci_write32(&host->sdhci, SDHCI_INT_STATUS, ~0);
 
        mask = SDHCI_INT_CMD_COMPLETE;
-       if (data)
-               mask |= SDHCI_INT_DATA_AVAIL;
 
        sdhci_set_cmd_xfer_mode(&host->sdhci, cmd, data, false, &command, 
&xfer);
 
-- 
2.25.1

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to