diff mbox series

[U-Boot,v2,4/4] mmc: omap_hsmmc: use a default 52MHz max clock rate if none is specified

Message ID 1519378819-19867-5-git-send-email-jjhiblot@ti.com
State Accepted
Delegated to: Jaehoon Chung
Headers show
Series mmc: omap_hsmmc: Reduce the footprint of the driver and fix am335x clock | expand

Commit Message

Jean-Jacques Hiblot Feb. 23, 2018, 9:40 a.m. UTC 
mmc_of_parse() doesn't set a default value if none is available in DT.
In that case, use a default 52MHz clock rate.

Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>

---

Changes in v2: None

 drivers/mmc/omap_hsmmc.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Tom Rini Feb. 23, 2018, 12:21 p.m. UTC | #1 
On Fri, Feb 23, 2018 at 10:40:19AM +0100, Jean-Jacques Hiblot wrote:

> mmc_of_parse() doesn't set a default value if none is available in DT.
> In that case, use a default 52MHz clock rate.
> 
> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
> 

Reviewed-by: Tom Rini <trini@konsulko.com>
Jaehoon Chung Feb. 26, 2018, 1:54 a.m. UTC | #2 
On 02/23/2018 09:21 PM, Tom Rini wrote:
> On Fri, Feb 23, 2018 at 10:40:19AM +0100, Jean-Jacques Hiblot wrote:
> 
>> mmc_of_parse() doesn't set a default value if none is available in DT.
>> In that case, use a default 52MHz clock rate.
>>
>> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
>> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
>>
> 
> Reviewed-by: Tom Rini <trini@konsulko.com>


Applied to u-boot-mmc. Thanks!

Best Regards,
Jaehoon Chung

>
diff mbox series

Patch

diff --git a/drivers/mmc/omap_hsmmc.c b/drivers/mmc/omap_hsmmc.c
index 3d2836d..caaa914 100644
--- a/drivers/mmc/omap_hsmmc.c
+++ b/drivers/mmc/omap_hsmmc.c
@@ -1832,6 +1832,8 @@  static int omap_hsmmc_ofdata_to_platdata(struct udevice *dev)
 	if (ret < 0)
 		return ret;
 
+	if (!cfg->f_max)
+		cfg->f_max = 52000000;
 	cfg->host_caps |= MMC_MODE_HS_52MHz | MMC_MODE_HS;
 	cfg->f_min = 400000;
 	cfg->voltages = MMC_VDD_32_33 | MMC_VDD_33_34 | MMC_VDD_165_195;