diff options
author | Linus Walleij <linus.walleij@linaro.org> | 2011-07-08 04:57:15 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2011-07-19 11:04:35 -0400 |
commit | 7f294e4983b67be802277d139522837b3ce5769f (patch) | |
tree | 9132eb2d87e70db9c8cee1d8132d6f69d1b6cdec /drivers/mmc | |
parent | eb96c925152fc289311e5d7e956b919e9b60ab53 (diff) |
ARM: 6981/2: mmci: adjust calculation of f_min
The ARM version maximum clock divider is 512 whereas for the ST
variants it's 257. Let's use DIV_ROUND_UP() for both cases so we
can see clearly what's going on here.
[Use DIV_ROUND_UP to clarify elder code]
Signed-off-by: Ulf Hansson <ulf.hansson@stericsson.com>
Reviewed-by: Sebastian Rasmussen <sebastian.rasmussen@stericsson.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'drivers/mmc')
-rw-r--r-- | drivers/mmc/host/mmci.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c index 7721de942c69..c471cdb1a0ba 100644 --- a/drivers/mmc/host/mmci.c +++ b/drivers/mmc/host/mmci.c | |||
@@ -1061,7 +1061,15 @@ static int __devinit mmci_probe(struct amba_device *dev, | |||
1061 | } | 1061 | } |
1062 | 1062 | ||
1063 | mmc->ops = &mmci_ops; | 1063 | mmc->ops = &mmci_ops; |
1064 | mmc->f_min = (host->mclk + 511) / 512; | 1064 | /* |
1065 | * The ARM and ST versions of the block have slightly different | ||
1066 | * clock divider equations which means that the minimum divider | ||
1067 | * differs too. | ||
1068 | */ | ||
1069 | if (variant->st_clkdiv) | ||
1070 | mmc->f_min = DIV_ROUND_UP(host->mclk, 257); | ||
1071 | else | ||
1072 | mmc->f_min = DIV_ROUND_UP(host->mclk, 512); | ||
1065 | /* | 1073 | /* |
1066 | * If the platform data supplies a maximum operating | 1074 | * If the platform data supplies a maximum operating |
1067 | * frequency, this takes precedence. Else, we fall back | 1075 | * frequency, this takes precedence. Else, we fall back |