* [PATCH resend] mmc: mtk-sd: don't ignore max-frequency from device tree
@ 2021-03-15 15:31 Daniel Golle
2021-03-18 16:56 ` Tom Rini
0 siblings, 1 reply; 2+ messages in thread
From: Daniel Golle @ 2021-03-15 15:31 UTC (permalink / raw)
To: u-boot
commit e58e68d9 ("mmc: mtk-sd: assign plat->cfg.f_max with a correct value")
wrongly assumed that plat->cfg.f_max is always unset at the time
mscd_drv_probe() is run. This is not true in case max-frequency being
defined in device tree, as it is then already set by mmc_of_parse()
in msdc_of_to_plat().
Only set plat->cfg.f_max to the default maximum value in case it is
not already set to a sane value.
Fixes: e58e68d93e ("mmc: mtk-sd: assign plat->cfg.f_max with a correct value")
Cc: Stefan Roese <sr@denx.de>
Cc: Weijie Gao <weijie.gao@mediatek.com>
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
---
drivers/mmc/mtk-sd.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/mmc/mtk-sd.c b/drivers/mmc/mtk-sd.c
index 3b9c12266a..48a764be82 100644
--- a/drivers/mmc/mtk-sd.c
+++ b/drivers/mmc/mtk-sd.c
@@ -1639,7 +1639,8 @@ static int msdc_drv_probe(struct udevice *dev)
else
cfg->f_min = host->src_clk_freq / (4 * 4095);
- cfg->f_max = host->src_clk_freq;
+ if (cfg->f_max < cfg->f_min || cfg->f_max > host->src_clk_freq)
+ cfg->f_max = host->src_clk_freq;
cfg->b_max = 1024;
cfg->voltages = MMC_VDD_32_33 | MMC_VDD_33_34;
--
2.30.2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210315/817faf6f/attachment.sig>
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH resend] mmc: mtk-sd: don't ignore max-frequency from device tree
2021-03-15 15:31 [PATCH resend] mmc: mtk-sd: don't ignore max-frequency from device tree Daniel Golle
@ 2021-03-18 16:56 ` Tom Rini
0 siblings, 0 replies; 2+ messages in thread
From: Tom Rini @ 2021-03-18 16:56 UTC (permalink / raw)
To: u-boot
On Mon, Mar 15, 2021 at 03:31:11PM +0000, Daniel Golle wrote:
> commit e58e68d9 ("mmc: mtk-sd: assign plat->cfg.f_max with a correct value")
> wrongly assumed that plat->cfg.f_max is always unset at the time
> mscd_drv_probe() is run. This is not true in case max-frequency being
> defined in device tree, as it is then already set by mmc_of_parse()
> in msdc_of_to_plat().
> Only set plat->cfg.f_max to the default maximum value in case it is
> not already set to a sane value.
>
> Fixes: e58e68d93e ("mmc: mtk-sd: assign plat->cfg.f_max with a correct value")
> Cc: Stefan Roese <sr@denx.de>
> Cc: Weijie Gao <weijie.gao@mediatek.com>
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210318/4ab91a0d/attachment.sig>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-03-18 16:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-15 15:31 [PATCH resend] mmc: mtk-sd: don't ignore max-frequency from device tree Daniel Golle
2021-03-18 16:56 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.