All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH -next] mtd: rawnand: ingenic: Make jz4725b_ooblayout_ops static
@ 2019-04-10 14:00 ` Yue Haibing
  0 siblings, 0 replies; 4+ messages in thread
From: Yue Haibing @ 2019-04-10 14:00 UTC (permalink / raw)
  To: bbrezillon, miquel.raynal, richard, dwmw2, computersforpeace,
	marek.vasut, paul
  Cc: linux-kernel, linux-mtd, YueHaibing

From: YueHaibing <yuehaibing@huawei.com>

Fix sparse warning:

drivers/mtd/nand/raw/ingenic/ingenic_nand.c:140:32: warning:
 symbol 'jz4725b_ooblayout_ops' was not declared. Should it be static?

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 drivers/mtd/nand/raw/ingenic/ingenic_nand.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/raw/ingenic/ingenic_nand.c b/drivers/mtd/nand/raw/ingenic/ingenic_nand.c
index ad0c905..d7b7c0f 100644
--- a/drivers/mtd/nand/raw/ingenic/ingenic_nand.c
+++ b/drivers/mtd/nand/raw/ingenic/ingenic_nand.c
@@ -137,7 +137,7 @@ static int jz4725b_ooblayout_free(struct mtd_info *mtd, int section,
 	return 0;
 }
 
-const struct mtd_ooblayout_ops jz4725b_ooblayout_ops = {
+static const struct mtd_ooblayout_ops jz4725b_ooblayout_ops = {
 	.ecc = jz4725b_ooblayout_ecc,
 	.free = jz4725b_ooblayout_free,
 };
-- 
2.7.4



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

* [PATCH -next] mtd: rawnand: ingenic: Make jz4725b_ooblayout_ops static
@ 2019-04-10 14:00 ` Yue Haibing
  0 siblings, 0 replies; 4+ messages in thread
From: Yue Haibing @ 2019-04-10 14:00 UTC (permalink / raw)
  To: bbrezillon, miquel.raynal, richard, dwmw2, computersforpeace,
	marek.vasut, paul
  Cc: YueHaibing, linux-mtd, linux-kernel

From: YueHaibing <yuehaibing@huawei.com>

Fix sparse warning:

drivers/mtd/nand/raw/ingenic/ingenic_nand.c:140:32: warning:
 symbol 'jz4725b_ooblayout_ops' was not declared. Should it be static?

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 drivers/mtd/nand/raw/ingenic/ingenic_nand.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/raw/ingenic/ingenic_nand.c b/drivers/mtd/nand/raw/ingenic/ingenic_nand.c
index ad0c905..d7b7c0f 100644
--- a/drivers/mtd/nand/raw/ingenic/ingenic_nand.c
+++ b/drivers/mtd/nand/raw/ingenic/ingenic_nand.c
@@ -137,7 +137,7 @@ static int jz4725b_ooblayout_free(struct mtd_info *mtd, int section,
 	return 0;
 }
 
-const struct mtd_ooblayout_ops jz4725b_ooblayout_ops = {
+static const struct mtd_ooblayout_ops jz4725b_ooblayout_ops = {
 	.ecc = jz4725b_ooblayout_ecc,
 	.free = jz4725b_ooblayout_free,
 };
-- 
2.7.4



______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

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

* Re: [PATCH -next] mtd: rawnand: ingenic: Make jz4725b_ooblayout_ops static
  2019-04-10 14:00 ` Yue Haibing
@ 2019-04-18 16:22   ` Miquel Raynal
  -1 siblings, 0 replies; 4+ messages in thread
From: Miquel Raynal @ 2019-04-18 16:22 UTC (permalink / raw)
  To: Yue Haibing
  Cc: bbrezillon, richard, dwmw2, computersforpeace, marek.vasut, paul,
	linux-kernel, linux-mtd

Hi Yue,

Yue Haibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 22:00:35
+0800:

> From: YueHaibing <yuehaibing@huawei.com>
> 
> Fix sparse warning:
> 
> drivers/mtd/nand/raw/ingenic/ingenic_nand.c:140:32: warning:
>  symbol 'jz4725b_ooblayout_ops' was not declared. Should it be static?
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
> ---


Applied to 
https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git 
branch nand/next.

Thanks,
Miquèl

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

* Re: [PATCH -next] mtd: rawnand: ingenic: Make jz4725b_ooblayout_ops static
@ 2019-04-18 16:22   ` Miquel Raynal
  0 siblings, 0 replies; 4+ messages in thread
From: Miquel Raynal @ 2019-04-18 16:22 UTC (permalink / raw)
  To: Yue Haibing
  Cc: bbrezillon, richard, linux-kernel, paul, marek.vasut, linux-mtd,
	computersforpeace, dwmw2

Hi Yue,

Yue Haibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 22:00:35
+0800:

> From: YueHaibing <yuehaibing@huawei.com>
> 
> Fix sparse warning:
> 
> drivers/mtd/nand/raw/ingenic/ingenic_nand.c:140:32: warning:
>  symbol 'jz4725b_ooblayout_ops' was not declared. Should it be static?
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>
> ---


Applied to 
https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git 
branch nand/next.

Thanks,
Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

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

end of thread, other threads:[~2019-04-18 16:22 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-10 14:00 [PATCH -next] mtd: rawnand: ingenic: Make jz4725b_ooblayout_ops static Yue Haibing
2019-04-10 14:00 ` Yue Haibing
2019-04-18 16:22 ` Miquel Raynal
2019-04-18 16:22   ` Miquel Raynal

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.