FW: mmc: zynq_sdhci: transfer fails with 8-bit data bus

2020-05-11 Thread Benedikt Grassl
Hi, with commit 942b5fc0 the 8-bit data bus seems to work fine in U-Boot. However, I have one situation where I get transfer errors lateron in Linux. At the moment I suspect that this is rather a problem of the Linux driver, but I would like to make sure here first. My ZynqMP has an eMMC flash c

Re: [PATCH v4] mmc: zynq: parse dt when probing

2020-04-15 Thread Benedikt Grassl
Hi Michal, I just tried to check if the read performance acutally increased when using an 8-bit data bus. As a quick test, I use the fatload command to read a chunk of 1 GByte from an eMMC flash (generated with dd from Linux). However, I don't see any difference at all: mmc info Device: mmc@ff160

[PATCH v4] mmc: zynq: parse dt when probing

2020-04-13 Thread Benedikt Grassl
ncy" entries. Remove the handling of those nodes from this driver. Signed-off-by: Benedikt Grassl --- drivers/mmc/sdhci.c | 3 +-- drivers/mmc/zynq_sdhci.c | 15 ++- include/sdhci.h | 1 - 3 files changed, 7 insertions(+), 12 deletions(-) diff --git a/driver

[PATCH v3] mmc: zynq: parse dt when probing

2020-04-11 Thread Benedikt Grassl
ncy" entries. Remove the handling of those nodes from this driver. Signed-off-by: Benedikt Grassl --- drivers/mmc/sdhci.c | 3 +-- drivers/mmc/zynq_sdhci.c | 16 +++- include/sdhci.h | 1 - 3 files changed, 8 insertions(+), 12 deletions(-) diff --git a/driver

mmc: zynq: parse dt when probing

2020-04-07 Thread Benedikt Grassl
Hi Michal, I did not revert b8e25ef16a58 ("mmc: sdhci: Read capabilities register1 and update host caps") since without those changes, the UHS caps would not disabled if you forgot to add the "no-1-8-v" property. I think relying on the voltage configuration instead is worth keeping. Best Regard

[PATCH V2] mmc: zynq: parse dt when probing

2020-04-07 Thread Benedikt Grassl
ncy" entries. Remove the handling of those nodes from this driver. Signed-off-by: Benedikt Grassl --- drivers/mmc/sdhci.c | 3 +-- drivers/mmc/zynq_sdhci.c | 15 ++- include/sdhci.h | 1 - 3 files changed, 7 insertions(+), 12 deletions(-) diff --git a/driver

[PATCH] mmc: zynq: parse dt when probing

2020-04-06 Thread Benedikt Grassl
Currently, the entry "bus-width = <8>" in the ZynqMP's sdhci nodes is not evaluated. This results in the bus width staying at its default value (4 bit in HS200 mode). Fix this by parsing the device tree while probing. Signed-off-by: Benedikt Grassl --- drivers/mmc/zynq_sdhc

mmc: zynq: "bus-width = <8>" entry ignored?

2020-04-06 Thread Benedikt Grassl
Hi, I added a "bus-width = <8>" entry to the ZynqMP's sdhci0 node on our custom board (that acutally uses an 8-bit data bus). I also set it to HS200 mode. However, this setting is ignored: when I run "mmc info" the bus width seems to be 4 bit: Device: mmc@ff16 (...) Bus Speed: 2 Mode