OSDN Git Service

mmc: moxart: fix probe logic
authorArnd Bergmann <arnd@arndb.de>
Tue, 3 Feb 2015 15:55:54 +0000 (16:55 +0100)
committerUlf Hansson <ulf.hansson@linaro.org>
Wed, 4 Feb 2015 08:42:29 +0000 (09:42 +0100)
commit3981c516664d81a3e83c1923fa1ab3988c529402
treec7c43a53c07d85fe91eabe9dc508b191e256df8b
parent4febb7e20aa619e2b2845519dad247e4038dc383
mmc: moxart: fix probe logic

Jonas Jensen wanted to submit a patch for these, but apparently
forgot about it. I stumbled over this symptom first:

drivers/built-in.o: In function `moxart_probe':
:(.text+0x2af128): undefined reference to `of_dma_request_slave_channel'

This is because of_dma_request_slave_channel is an internal helper
and not exported to loadable module. I'm changing the driver to
use dma_request_slave_channel_reason() instead.

Further problems from inspection:

* The remove function must not call kfree on the host pointer,
  because it is allocated together with the mmc_host.

* The clock is never released

* The dma_cap_mask_t is completely unused and can be removed

* deferred probing does not work if the dma driver is loaded
  after the mmc driver.

This patch should fix all of the above.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Jonas Jensen <jonas.jensen@gmail.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/moxart-mmc.c