diff mbox

mmc: sdhci-of-at91: remove bogus MMC_SDHCI_IO_ACCESSORS select

Message ID 1480565146-4857-1-git-send-email-yamada.masahiro@socionext.com
State New
Headers show

Commit Message

Masahiro Yamada Dec. 1, 2016, 4:05 a.m. UTC
I see no override of read/write callbacks in sdhci-of-at91.c.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

---

BTW, this config may not be so useful in recent multi-platforms.
Perhaps, is it better to remove this config entirely instead of
the AT91 fix only.


 drivers/mmc/host/Kconfig | 1 -
 1 file changed, 1 deletion(-)

-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Ludovic Desroches Dec. 2, 2016, 6:41 a.m. UTC | #1
On Thu, Dec 01, 2016 at 01:05:46PM +0900, Masahiro Yamada wrote:
> I see no override of read/write callbacks in sdhci-of-at91.c.

> 

> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

Acked-by: Ludovic Desroches <ludovic.desroches@atmel.com>


Thanks

> ---

> 

> BTW, this config may not be so useful in recent multi-platforms.

> Perhaps, is it better to remove this config entirely instead of

> the AT91 fix only.

> 

> 

>  drivers/mmc/host/Kconfig | 1 -

>  1 file changed, 1 deletion(-)

> 

> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig

> index 39f6f96..8ac1640 100644

> --- a/drivers/mmc/host/Kconfig

> +++ b/drivers/mmc/host/Kconfig

> @@ -135,7 +135,6 @@ config MMC_SDHCI_OF_AT91

>  	tristate "SDHCI OF support for the Atmel SDMMC controller"

>  	depends on MMC_SDHCI_PLTFM

>  	depends on OF

> -	select MMC_SDHCI_IO_ACCESSORS

>  	help

>  	  This selects the Atmel SDMMC driver

>  

> -- 

> 2.7.4

> 

> --

> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in

> the body of a message to majordomo@vger.kernel.org

> 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 majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ulf Hansson Dec. 5, 2016, 1:18 p.m. UTC | #2
On 1 December 2016 at 05:05, Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
> I see no override of read/write callbacks in sdhci-of-at91.c.

>

> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>


Thanks, applied for next!

Kind regards
Uffe

> ---

>

> BTW, this config may not be so useful in recent multi-platforms.

> Perhaps, is it better to remove this config entirely instead of

> the AT91 fix only.

>

>

>  drivers/mmc/host/Kconfig | 1 -

>  1 file changed, 1 deletion(-)

>

> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig

> index 39f6f96..8ac1640 100644

> --- a/drivers/mmc/host/Kconfig

> +++ b/drivers/mmc/host/Kconfig

> @@ -135,7 +135,6 @@ config MMC_SDHCI_OF_AT91

>         tristate "SDHCI OF support for the Atmel SDMMC controller"

>         depends on MMC_SDHCI_PLTFM

>         depends on OF

> -       select MMC_SDHCI_IO_ACCESSORS

>         help

>           This selects the Atmel SDMMC driver

>

> --

> 2.7.4

>

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
index 39f6f96..8ac1640 100644
--- a/drivers/mmc/host/Kconfig
+++ b/drivers/mmc/host/Kconfig
@@ -135,7 +135,6 @@  config MMC_SDHCI_OF_AT91
 	tristate "SDHCI OF support for the Atmel SDMMC controller"
 	depends on MMC_SDHCI_PLTFM
 	depends on OF
-	select MMC_SDHCI_IO_ACCESSORS
 	help
 	  This selects the Atmel SDMMC driver