[PATCH v2 4/8] mmc: dw_mmc: Use mmc_regulator_set_vqmmc in start_signal_voltage_switch
dianders at chromium.org
Fri Oct 9 13:12:10 PDT 2015
On Fri, Oct 9, 2015 at 12:06 PM, Kevin Hilman <khilman at kernel.org> wrote:
> On Wed, Sep 30, 2015 at 4:07 PM, Heiko Stuebner <heiko at sntech.de> wrote:
>> From: Douglas Anderson <dianders at chromium.org>
>> We've introduced a new helper in the MMC core:
>> mmc_regulator_set_vqmmc(). Let's use this in dw_mmc. Using this new
>> helper has some advantages:
>> 1. We get the mmc_regulator_set_vqmmc() behavior of trying to match
>> VQMMC and VMMC when the signal voltage is 3.3V. This ensures max
>> 2. We get rid of a few more warnings when probing unsupported
>> 3. We get rid of some non-dw_mmc specific code in dw_mmc.
>> Signed-off-by: Douglas Anderson <dianders at chromium.org>
>> Signed-off-by: Heiko Stuebner <heiko at sntech.de>
> kernelci.org found a new failure in linux-next on
> exynos5800-peach-pi and it was bisected down to this patch, which
> has hit linux-next in the form of commit fba2e23128d0 (mmc: dw_mmc:
> Use mmc_regulator_set_vqmmc in start_signal_voltage_switch). Based on
> a quick glance at the boot log, it looks to be a bad pointer
> dereference in regulator_get_voltage().
> I confirmed that reverting this patch on top of next-20151009 gets my
> peach-pi booting again.
> Note that the boot failure is only on exynos_defconfig, not multi_v7_defconfig.
Right. See <http://www.spinics.net/lists/arm-kernel/msg450802.html>
for discussion. Ulf responded to the cover letter rather than the
More information about the linux-arm-kernel