mmc: sunxi: Fix NULL pointer reference on clk_delays
authorChen-Yu Tsai <wens@csie.org>
Tue, 8 Aug 2017 07:02:44 +0000 (15:02 +0800)
committerUlf Hansson <ulf.hansson@linaro.org>
Wed, 30 Aug 2017 12:01:50 +0000 (14:01 +0200)
Some SoCs do not support clk delays for MMC in the clock control unit.
These include the old controllers in A10/A10s/A13/R8, and the new eMMC
controller in A64. The config structure for these controllers do not
specify clk_delays, but the check for this was replaced in change
"mmc: sunxi: Support controllers that can use both old and new timings".

This patch adds back the check for clk_delays, and also adds comments
for both checks in sunxi_mmc_clk_set_phase().

Signed-off-by: Chen-Yu Tsai <wens@csie.org>
Tested-by: Icenowy Zheng <icenowy@aosc.io>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/sunxi-mmc.c

index 3777517982dd2589790db85eafe1bc6f6e528e19..9dc6d726ec493ec3c9a352bfde3bbc2c5ccbfea6 100644 (file)
@@ -722,9 +722,14 @@ static int sunxi_mmc_clk_set_phase(struct sunxi_mmc_host *host,
 {
        int index;
 
+       /* clk controller delays not used under new timings mode */
        if (host->use_new_timings)
                return 0;
 
+       /* some old controllers don't support delays */
+       if (!host->cfg->clk_delays)
+               return 0;
+
        /* determine delays */
        if (rate <= 400000) {
                index = SDXC_CLK_400K;