Dear, Russell.
I didn't know which version you have checked.
But I have known that mmc_of_parse() have already included at mmc-next repository.
commit 6ca5c60605dde3420ce7fee6f93ec4a7c7924848
Author: Jaehoon Chung <jh80.chung@xxxxxxxxx>
Date: Mon Mar 3 11:36:41 2014 +0900
mmc: dw_mmc: use the mmc_of_parse() instead of local parser
mmc_of_parse() have been already parsed the general capability.
Didn't need to use the local parser.
Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx>
Acked-by: Seungwon Jeon <tgih.jun@xxxxxxxxxxx>
Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
Signed-off-by: Chris Ball <chris@xxxxxxxxxx>
Could you check it, plz?
Best Regards,
Jaehoon Chung
On 04/24/2014 04:09 AM, Russell King wrote:
> From: Olof Johansson <olof@xxxxxxxxx>
>
> The shared of parse function fills in common options for capabilities,
> etc, but it needs to be called from each driver that wants to make use
> of it. dw_mmc was missing the call.
>
> Signed-off-by: Olof Johansson <olof@xxxxxxxxx>
> Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx>
> ---
> drivers/mmc/host/dw_mmc.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> index cced599d5aeb..a2c12ebb376e 100644
> --- a/drivers/mmc/host/dw_mmc.c
> +++ b/drivers/mmc/host/dw_mmc.c
> @@ -2140,6 +2140,8 @@ static int dw_mci_init_slot(struct dw_mci *host, unsigned int id)
> if (!mmc)
> return -ENOMEM;
>
> + mmc_of_parse(mmc);
> +
> slot = mmc_priv(mmc);
> slot->id = id;
> slot->mmc = mmc;
>
--
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
