public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH V2 1/2] mtd: nand: xway: disable module support
@ 2016-12-05 21:14 Hauke Mehrtens
  2016-12-05 21:14 ` [PATCH V2 2/2] mtd: nand: xway: fix build because of module functions Hauke Mehrtens
  2017-01-03  8:19 ` [PATCH V2 1/2] mtd: nand: xway: disable module support Boris Brezillon
  0 siblings, 2 replies; 3+ messages in thread
From: Hauke Mehrtens @ 2016-12-05 21:14 UTC (permalink / raw)
  To: boris.brezillon
  Cc: richard, dwmw2, computersforpeace, marek.vasut, cyrille.pitchen,
	linux-mtd, john, Hauke Mehrtens, stable

The xway_nand driver accesses the ltq_ebu_membase symbol which is not
exported. This also should not get exported and we should handle the
EBU interface in a better way later. This quick fix just deactivated
support for building as module.

Cc: <stable@vger.kernel.org>
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
---
 drivers/mtd/nand/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
index 7b7a887..b254090 100644
--- a/drivers/mtd/nand/Kconfig
+++ b/drivers/mtd/nand/Kconfig
@@ -537,7 +537,7 @@ config MTD_NAND_FSMC
 	  Flexible Static Memory Controller (FSMC)
 
 config MTD_NAND_XWAY
-	tristate "Support for NAND on Lantiq XWAY SoC"
+	bool "Support for NAND on Lantiq XWAY SoC"
 	depends on LANTIQ && SOC_TYPE_XWAY
 	help
 	  Enables support for NAND Flash chips on Lantiq XWAY SoCs. NAND is attached
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH V2 2/2] mtd: nand: xway: fix build because of module functions
  2016-12-05 21:14 [PATCH V2 1/2] mtd: nand: xway: disable module support Hauke Mehrtens
@ 2016-12-05 21:14 ` Hauke Mehrtens
  2017-01-03  8:19 ` [PATCH V2 1/2] mtd: nand: xway: disable module support Boris Brezillon
  1 sibling, 0 replies; 3+ messages in thread
From: Hauke Mehrtens @ 2016-12-05 21:14 UTC (permalink / raw)
  To: boris.brezillon
  Cc: richard, dwmw2, computersforpeace, marek.vasut, cyrille.pitchen,
	linux-mtd, john, Hauke Mehrtens, stable

Remove the usage of modules functions to make this driver compile
again. Otherwise an include of linux/modules.h would be needed.

Fixes: 024366750c2e ("mtd: nand: xway: convert to normal platform driver")
Cc: <stable@vger.kernel.org>
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
---
 drivers/mtd/nand/xway_nand.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/mtd/nand/xway_nand.c b/drivers/mtd/nand/xway_nand.c
index 1f2948c..895101a 100644
--- a/drivers/mtd/nand/xway_nand.c
+++ b/drivers/mtd/nand/xway_nand.c
@@ -232,7 +232,6 @@ static const struct of_device_id xway_nand_match[] = {
 	{ .compatible = "lantiq,nand-xway" },
 	{},
 };
-MODULE_DEVICE_TABLE(of, xway_nand_match);
 
 static struct platform_driver xway_nand_driver = {
 	.probe	= xway_nand_probe,
@@ -243,6 +242,4 @@ static struct platform_driver xway_nand_driver = {
 	},
 };
 
-module_platform_driver(xway_nand_driver);
-
-MODULE_LICENSE("GPL");
+builtin_platform_driver(xway_nand_driver);
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH V2 1/2] mtd: nand: xway: disable module support
  2016-12-05 21:14 [PATCH V2 1/2] mtd: nand: xway: disable module support Hauke Mehrtens
  2016-12-05 21:14 ` [PATCH V2 2/2] mtd: nand: xway: fix build because of module functions Hauke Mehrtens
@ 2017-01-03  8:19 ` Boris Brezillon
  1 sibling, 0 replies; 3+ messages in thread
From: Boris Brezillon @ 2017-01-03  8:19 UTC (permalink / raw)
  To: Hauke Mehrtens
  Cc: richard, dwmw2, computersforpeace, marek.vasut, cyrille.pitchen,
	linux-mtd, john, stable

On Mon,  5 Dec 2016 22:14:36 +0100
Hauke Mehrtens <hauke@hauke-m.de> wrote:

> The xway_nand driver accesses the ltq_ebu_membase symbol which is not
> exported. This also should not get exported and we should handle the
> EBU interface in a better way later. This quick fix just deactivated
> support for building as module.
> 
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>

Applied both to nand/fixes. They should appear in one of the next -rc
(probably not -rc3 though).

Thanks,

Boris

> ---
>  drivers/mtd/nand/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
> index 7b7a887..b254090 100644
> --- a/drivers/mtd/nand/Kconfig
> +++ b/drivers/mtd/nand/Kconfig
> @@ -537,7 +537,7 @@ config MTD_NAND_FSMC
>  	  Flexible Static Memory Controller (FSMC)
>  
>  config MTD_NAND_XWAY
> -	tristate "Support for NAND on Lantiq XWAY SoC"
> +	bool "Support for NAND on Lantiq XWAY SoC"
>  	depends on LANTIQ && SOC_TYPE_XWAY
>  	help
>  	  Enables support for NAND Flash chips on Lantiq XWAY SoCs. NAND is attached

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-01-03  8:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-05 21:14 [PATCH V2 1/2] mtd: nand: xway: disable module support Hauke Mehrtens
2016-12-05 21:14 ` [PATCH V2 2/2] mtd: nand: xway: fix build because of module functions Hauke Mehrtens
2017-01-03  8:19 ` [PATCH V2 1/2] mtd: nand: xway: disable module support Boris Brezillon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox