W dniu 17.02.2017 o 20:47, Kevin Hilman pisze: > Michał Zegan <webczat@xxxxxxxxxxxxx> writes: > >> The mmc host was added in meson_mmc_probe, but never removed in meson_mmc_remove. >> Fix that by removing the host before deallocating other resources. >> >> Signed-off-by: Michał Zegan <webczat@xxxxxxxxxxxxx> > Reviewed-by: Kevin Hilman <khilman@xxxxxxxxxxxx> I do not know how the mmc driver looks like after Heiner's cleanup. Does this patch get obsoleted by the prior cleanup, or conflict with it in some way? > >> --- >> drivers/mmc/host/meson-gx-mmc.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/mmc/host/meson-gx-mmc.c b/drivers/mmc/host/meson-gx-mmc.c >> index d444b6bfa02b..bb83446118a3 100644 >> --- a/drivers/mmc/host/meson-gx-mmc.c >> +++ b/drivers/mmc/host/meson-gx-mmc.c >> @@ -818,6 +818,8 @@ static int meson_mmc_remove(struct platform_device *pdev) >> if (WARN_ON(!host)) >> return 0; >> >> + mmc_remove_host(host->mmc); >> + >> if (host->bounce_buf) >> dma_free_coherent(host->dev, host->bounce_buf_size, >> host->bounce_buf, host->bounce_dma_addr); -- 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
