On 1 October 2014 11:25, Andrew Gabbasov <[email protected]> wrote:
> From: Dirk Behme <[email protected]>
>
> The commit ad93220de7da ("mmc: sdhci-esdhc-imx: change pinctrl state
> according to uhs mode") exits the probe in case there are no valid
> pinctrl states found.
>
> As there are configurations doing the pin mux properly in the boot
> loader, don't exit. Just warn, but go on in case if there are no
> pinctrl states in the device tree.
>
> Signed-off-by: Dirk Behme <[email protected]>
> Signed-off-by: Andrew Gabbasov <[email protected]>
Thanks! Applied for next with a minor fix, see below!
Kind regards
Uffe
> ---
> drivers/mmc/host/sdhci-esdhc-imx.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci-esdhc-imx.c
> b/drivers/mmc/host/sdhci-esdhc-imx.c
> index ccec0e3..b714b9d 100644
> --- a/drivers/mmc/host/sdhci-esdhc-imx.c
> +++ b/drivers/mmc/host/sdhci-esdhc-imx.c
> @@ -1012,9 +1012,7 @@ static int sdhci_esdhc_imx_probe(struct platform_device
> *pdev)
> imx_data->pins_default = pinctrl_lookup_state(imx_data->pinctrl,
> PINCTRL_STATE_DEFAULT);
> if (IS_ERR(imx_data->pins_default)) {
> - err = PTR_ERR(imx_data->pins_default);
> - dev_err(mmc_dev(host->mmc), "could not get default state\n");
> - goto disable_clk;
> + dev_warn(mmc_dev(host->mmc), "could not get default state\n");
This "if" don't need the brackets any more. I removed them.
> }
>
> host->quirks |= SDHCI_QUIRK_BROKEN_TIMEOUT_VAL;
> @@ -1103,7 +1101,8 @@ static int sdhci_esdhc_imx_probe(struct platform_device
> *pdev)
> }
>
> /* sdr50 and sdr104 needs work on 1.8v signal voltage */
> - if ((boarddata->support_vsel) && esdhc_is_usdhc(imx_data)) {
> + if ((boarddata->support_vsel) && esdhc_is_usdhc(imx_data) &&
> + !IS_ERR(imx_data->pins_default)) {
> imx_data->pins_100mhz =
> pinctrl_lookup_state(imx_data->pinctrl,
> ESDHC_PINCTRL_STATE_100MHZ);
> imx_data->pins_200mhz =
> pinctrl_lookup_state(imx_data->pinctrl,
> --
> 2.1.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
