Hello. On 03/21/2012 06:41 PM, ludovic.desroches@xxxxxxxxx wrote:
From: Ludovic Desroches<ludovic.desroches@xxxxxxxxx>
The HSMCI operates at a rate of up to Master Clock divided by two.
Signed-off-by: Ludovic Desroches<ludovic.desroches@xxxxxxxxx> --- drivers/mmc/host/atmel-mci.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mmc/host/atmel-mci.c b/drivers/mmc/host/atmel-mci.c index e4449a5..e161452 100644 --- a/drivers/mmc/host/atmel-mci.c +++ b/drivers/mmc/host/atmel-mci.c @@ -480,7 +480,7 @@ err: static inline unsigned int atmci_ns_to_clocks(struct atmel_mci *host, unsigned int ns) { - return (ns * (host->bus_hz / 1000000) + 999) / 1000; + return (ns * ((host->bus_hz/2) / 1000000) + 999) / 1000;
Why don't you keep the notation by addign spaces around '/'? BTW, could use DIV_ROUND_UP() here. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html