* [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module
@ 2019-04-10 13:07 ` Yue Haibing
0 siblings, 0 replies; 24+ messages in thread
From: Yue Haibing @ 2019-04-10 13:07 UTC (permalink / raw)
To: miquel.raynal, richard, computersforpeace, marek.vasut,
paul.burton, bbrezillon
Cc: YueHaibing, linux-mtd, linux-kernel
From: YueHaibing <yuehaibing@huawei.com>
Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH
is set to module:
drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup':
(.text+0xef6): undefined reference to `nand_bch_free'
drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail':
nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc'
nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data'
nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init'
CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M,
because MTD_RAW_NAND need it while linked.
Reported-by: Hulk Robot <hulkci@huawei.com>
Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support"
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
drivers/mtd/nand/raw/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig
index 615d738..0500c42 100644
--- a/drivers/mtd/nand/raw/Kconfig
+++ b/drivers/mtd/nand/raw/Kconfig
@@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND
if MTD_RAW_NAND
config MTD_NAND_ECC_SW_BCH
- tristate "Support software BCH ECC"
+ bool "Support software BCH ECC"
select BCH
default n
help
--
2.7.4
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply related [flat|nested] 24+ messages in thread* [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 13:07 ` Yue Haibing 0 siblings, 0 replies; 24+ messages in thread From: Yue Haibing @ 2019-04-10 13:07 UTC (permalink / raw) To: miquel.raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon Cc: linux-kernel, linux-mtd, YueHaibing From: YueHaibing <yuehaibing@huawei.com> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module: drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': (.text+0xef6): undefined reference to `nand_bch_free' drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, because MTD_RAW_NAND need it while linked. Reported-by: Hulk Robot <hulkci@huawei.com> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/mtd/nand/raw/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig index 615d738..0500c42 100644 --- a/drivers/mtd/nand/raw/Kconfig +++ b/drivers/mtd/nand/raw/Kconfig @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND if MTD_RAW_NAND config MTD_NAND_ECC_SW_BCH - tristate "Support software BCH ECC" + bool "Support software BCH ECC" select BCH default n help -- 2.7.4 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 13:07 ` Yue Haibing @ 2019-04-10 13:39 ` Boris Brezillon -1 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 13:39 UTC (permalink / raw) To: Yue Haibing Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, miquel.raynal, computersforpeace On Wed, 10 Apr 2019 21:07:47 +0800 Yue Haibing <yuehaibing@huawei.com> wrote: > From: YueHaibing <yuehaibing@huawei.com> > > Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > is set to module: > > drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > (.text+0xef6): undefined reference to `nand_bch_free' > drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > because MTD_RAW_NAND need it while linked. > > Reported-by: Hulk Robot <hulkci@huawei.com> > Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" Nope, it's not this one that introduced the regression. > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > drivers/mtd/nand/raw/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > index 615d738..0500c42 100644 > --- a/drivers/mtd/nand/raw/Kconfig > +++ b/drivers/mtd/nand/raw/Kconfig > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > if MTD_RAW_NAND > > config MTD_NAND_ECC_SW_BCH > - tristate "Support software BCH ECC" > + bool "Support software BCH ECC" > select BCH > default n > help Should be fixed with the following diff squashed into: 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") --->8--- diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h index b8106651f807..06ce2b655c13 100644 --- a/include/linux/mtd/nand_bch.h +++ b/include/linux/mtd/nand_bch.h @@ -15,7 +15,7 @@ struct mtd_info; struct nand_chip; struct nand_bch_control; -#if defined(CONFIG_MTD_NAND_ECC_BCH) +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) static inline int mtd_nand_has_bch(void) { return 1; } @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); */ void nand_bch_free(struct nand_bch_control *nbc); -#else /* !CONFIG_MTD_NAND_ECC_BCH */ +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ static inline int mtd_nand_has_bch(void) { return 0; } @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) static inline void nand_bch_free(struct nand_bch_control *nbc) {} -#endif /* CONFIG_MTD_NAND_ECC_BCH */ +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ #endif /* __MTD_NAND_BCH_H__ */ ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 13:39 ` Boris Brezillon 0 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 13:39 UTC (permalink / raw) To: Yue Haibing Cc: miquel.raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On Wed, 10 Apr 2019 21:07:47 +0800 Yue Haibing <yuehaibing@huawei.com> wrote: > From: YueHaibing <yuehaibing@huawei.com> > > Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > is set to module: > > drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > (.text+0xef6): undefined reference to `nand_bch_free' > drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > because MTD_RAW_NAND need it while linked. > > Reported-by: Hulk Robot <hulkci@huawei.com> > Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" Nope, it's not this one that introduced the regression. > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > drivers/mtd/nand/raw/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > index 615d738..0500c42 100644 > --- a/drivers/mtd/nand/raw/Kconfig > +++ b/drivers/mtd/nand/raw/Kconfig > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > if MTD_RAW_NAND > > config MTD_NAND_ECC_SW_BCH > - tristate "Support software BCH ECC" > + bool "Support software BCH ECC" > select BCH > default n > help Should be fixed with the following diff squashed into: 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") --->8--- diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h index b8106651f807..06ce2b655c13 100644 --- a/include/linux/mtd/nand_bch.h +++ b/include/linux/mtd/nand_bch.h @@ -15,7 +15,7 @@ struct mtd_info; struct nand_chip; struct nand_bch_control; -#if defined(CONFIG_MTD_NAND_ECC_BCH) +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) static inline int mtd_nand_has_bch(void) { return 1; } @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); */ void nand_bch_free(struct nand_bch_control *nbc); -#else /* !CONFIG_MTD_NAND_ECC_BCH */ +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ static inline int mtd_nand_has_bch(void) { return 0; } @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) static inline void nand_bch_free(struct nand_bch_control *nbc) {} -#endif /* CONFIG_MTD_NAND_ECC_BCH */ +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ #endif /* __MTD_NAND_BCH_H__ */ ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 13:39 ` Boris Brezillon @ 2019-04-10 13:58 ` Boris Brezillon -1 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 13:58 UTC (permalink / raw) To: Yue Haibing Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, miquel.raynal, computersforpeace On Wed, 10 Apr 2019 15:39:28 +0200 Boris Brezillon <boris.brezillon@collabora.com> wrote: > On Wed, 10 Apr 2019 21:07:47 +0800 > Yue Haibing <yuehaibing@huawei.com> wrote: > > > From: YueHaibing <yuehaibing@huawei.com> > > > > Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > > is set to module: > > > > drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > > (.text+0xef6): undefined reference to `nand_bch_free' > > drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > > nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > > nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > > nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > > > CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > > because MTD_RAW_NAND need it while linked. > > > > Reported-by: Hulk Robot <hulkci@huawei.com> > > Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > > Nope, it's not this one that introduced the regression. > > > > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > > --- > > drivers/mtd/nand/raw/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > > index 615d738..0500c42 100644 > > --- a/drivers/mtd/nand/raw/Kconfig > > +++ b/drivers/mtd/nand/raw/Kconfig > > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > > if MTD_RAW_NAND > > > > config MTD_NAND_ECC_SW_BCH > > - tristate "Support software BCH ECC" > > + bool "Support software BCH ECC" > > select BCH > > default n > > help > > Should be fixed with the following diff squashed into: > > 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > > --->8--- > diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > index b8106651f807..06ce2b655c13 100644 > --- a/include/linux/mtd/nand_bch.h > +++ b/include/linux/mtd/nand_bch.h > @@ -15,7 +15,7 @@ struct mtd_info; > struct nand_chip; > struct nand_bch_control; > > -#if defined(CONFIG_MTD_NAND_ECC_BCH) > +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > > static inline int mtd_nand_has_bch(void) { return 1; } > > @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > */ > void nand_bch_free(struct nand_bch_control *nbc); > > -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > > static inline int mtd_nand_has_bch(void) { return 0; } > > @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > > static inline void nand_bch_free(struct nand_bch_control *nbc) {} > > -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > > #endif /* __MTD_NAND_BCH_H__ */ Sorry, I didn't look at the right branch, this part of the code was correct, but we still have a problem to express the RAW_NAND(y) -> SW_BCH(y) dependency. ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 13:58 ` Boris Brezillon 0 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 13:58 UTC (permalink / raw) To: Yue Haibing Cc: miquel.raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On Wed, 10 Apr 2019 15:39:28 +0200 Boris Brezillon <boris.brezillon@collabora.com> wrote: > On Wed, 10 Apr 2019 21:07:47 +0800 > Yue Haibing <yuehaibing@huawei.com> wrote: > > > From: YueHaibing <yuehaibing@huawei.com> > > > > Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > > is set to module: > > > > drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > > (.text+0xef6): undefined reference to `nand_bch_free' > > drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > > nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > > nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > > nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > > > CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > > because MTD_RAW_NAND need it while linked. > > > > Reported-by: Hulk Robot <hulkci@huawei.com> > > Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > > Nope, it's not this one that introduced the regression. > > > > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > > --- > > drivers/mtd/nand/raw/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > > index 615d738..0500c42 100644 > > --- a/drivers/mtd/nand/raw/Kconfig > > +++ b/drivers/mtd/nand/raw/Kconfig > > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > > if MTD_RAW_NAND > > > > config MTD_NAND_ECC_SW_BCH > > - tristate "Support software BCH ECC" > > + bool "Support software BCH ECC" > > select BCH > > default n > > help > > Should be fixed with the following diff squashed into: > > 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > > --->8--- > diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > index b8106651f807..06ce2b655c13 100644 > --- a/include/linux/mtd/nand_bch.h > +++ b/include/linux/mtd/nand_bch.h > @@ -15,7 +15,7 @@ struct mtd_info; > struct nand_chip; > struct nand_bch_control; > > -#if defined(CONFIG_MTD_NAND_ECC_BCH) > +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > > static inline int mtd_nand_has_bch(void) { return 1; } > > @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > */ > void nand_bch_free(struct nand_bch_control *nbc); > > -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > > static inline int mtd_nand_has_bch(void) { return 0; } > > @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > > static inline void nand_bch_free(struct nand_bch_control *nbc) {} > > -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > > #endif /* __MTD_NAND_BCH_H__ */ Sorry, I didn't look at the right branch, this part of the code was correct, but we still have a problem to express the RAW_NAND(y) -> SW_BCH(y) dependency. ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 13:58 ` Boris Brezillon @ 2019-04-10 14:10 ` Boris Brezillon -1 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 14:10 UTC (permalink / raw) To: Yue Haibing Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, miquel.raynal, computersforpeace On Wed, 10 Apr 2019 15:58:56 +0200 Boris Brezillon <boris.brezillon@collabora.com> wrote: > On Wed, 10 Apr 2019 15:39:28 +0200 > Boris Brezillon <boris.brezillon@collabora.com> wrote: > > > On Wed, 10 Apr 2019 21:07:47 +0800 > > Yue Haibing <yuehaibing@huawei.com> wrote: > > > > > From: YueHaibing <yuehaibing@huawei.com> > > > > > > Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > > > is set to module: > > > > > > drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > > > (.text+0xef6): undefined reference to `nand_bch_free' > > > drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > > > nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > > > nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > > > nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > > > > > CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > > > because MTD_RAW_NAND need it while linked. > > > > > > Reported-by: Hulk Robot <hulkci@huawei.com> > > > Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > > > > Nope, it's not this one that introduced the regression. > > > > > > > > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > > > --- > > > drivers/mtd/nand/raw/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > > > index 615d738..0500c42 100644 > > > --- a/drivers/mtd/nand/raw/Kconfig > > > +++ b/drivers/mtd/nand/raw/Kconfig > > > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > > > if MTD_RAW_NAND > > > > > > config MTD_NAND_ECC_SW_BCH > > > - tristate "Support software BCH ECC" > > > + bool "Support software BCH ECC" > > > select BCH > > > default n > > > help > > > > Should be fixed with the following diff squashed into: > > > > 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > > > > --->8--- > > diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > > index b8106651f807..06ce2b655c13 100644 > > --- a/include/linux/mtd/nand_bch.h > > +++ b/include/linux/mtd/nand_bch.h > > @@ -15,7 +15,7 @@ struct mtd_info; > > struct nand_chip; > > struct nand_bch_control; > > > > -#if defined(CONFIG_MTD_NAND_ECC_BCH) > > +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > > > > static inline int mtd_nand_has_bch(void) { return 1; } > > > > @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > > */ > > void nand_bch_free(struct nand_bch_control *nbc); > > > > -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > > +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > > > > static inline int mtd_nand_has_bch(void) { return 0; } > > > > @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > > > > static inline void nand_bch_free(struct nand_bch_control *nbc) {} > > > > -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > > +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > > > > #endif /* __MTD_NAND_BCH_H__ */ > > Sorry, I didn't look at the right branch, this part of the code was > correct, but we still have a problem to express the RAW_NAND(y) -> > SW_BCH(y) dependency. This one should do the trick, though imply means MTD_NAND_ECC_SW_BCH will now be selected by default. --->8--- diff --git a/drivers/mtd/devices/Kconfig b/drivers/mtd/devices/Kconfig index 7fcdaf6c279d..f9258d666846 100644 --- a/drivers/mtd/devices/Kconfig +++ b/drivers/mtd/devices/Kconfig @@ -207,7 +207,7 @@ comment "Disk-On-Chip Device Drivers" config MTD_DOCG3 tristate "M-Systems Disk-On-Chip G3" select BCH - select BCH_CONST_PARAMS if !CONFIG_MTD_NAND_ECC_SW_BCH + select BCH_CONST_PARAMS if !MTD_NAND_ECC_SW_BCH select BITREVERSE help This provides an MTD device driver for the M-Systems DiskOnChip diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig index 615d738be411..65c85fc53c41 100644 --- a/drivers/mtd/nand/raw/Kconfig +++ b/drivers/mtd/nand/raw/Kconfig @@ -9,18 +9,6 @@ config MTD_NAND_ECC_SW_HAMMING_SMC Software ECC according to the Smart Media Specification. The original Linux implementation had byte 0 and 1 swapped. -menuconfig MTD_RAW_NAND - tristate "Raw/Parallel NAND Device Support" - depends on MTD - select MTD_NAND_CORE - select MTD_NAND_ECC_SW_HAMMING - help - This enables support for accessing all type of raw/parallel - NAND flash devices. For further information see - <http://www.linux-mtd.infradead.org/doc/nand.html>. - -if MTD_RAW_NAND - config MTD_NAND_ECC_SW_BCH tristate "Support software BCH ECC" select BCH @@ -31,6 +19,19 @@ config MTD_NAND_ECC_SW_BCH ECC codes. They are used with NAND devices requiring more than 1 bit of error correction. +menuconfig MTD_RAW_NAND + tristate "Raw/Parallel NAND Device Support" + depends on MTD + select MTD_NAND_CORE + select MTD_NAND_ECC_SW_HAMMING + imply MTD_NAND_ECC_SW_BCH + help + This enables support for accessing all type of raw/parallel + NAND flash devices. For further information see + <http://www.linux-mtd.infradead.org/doc/nand.html>. + +if MTD_RAW_NAND + comment "Raw/parallel NAND flash controllers" config MTD_NAND_DENALI ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 14:10 ` Boris Brezillon 0 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 14:10 UTC (permalink / raw) To: Yue Haibing Cc: miquel.raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On Wed, 10 Apr 2019 15:58:56 +0200 Boris Brezillon <boris.brezillon@collabora.com> wrote: > On Wed, 10 Apr 2019 15:39:28 +0200 > Boris Brezillon <boris.brezillon@collabora.com> wrote: > > > On Wed, 10 Apr 2019 21:07:47 +0800 > > Yue Haibing <yuehaibing@huawei.com> wrote: > > > > > From: YueHaibing <yuehaibing@huawei.com> > > > > > > Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > > > is set to module: > > > > > > drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > > > (.text+0xef6): undefined reference to `nand_bch_free' > > > drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > > > nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > > > nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > > > nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > > > > > CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > > > because MTD_RAW_NAND need it while linked. > > > > > > Reported-by: Hulk Robot <hulkci@huawei.com> > > > Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > > > > Nope, it's not this one that introduced the regression. > > > > > > > > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > > > --- > > > drivers/mtd/nand/raw/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > > > index 615d738..0500c42 100644 > > > --- a/drivers/mtd/nand/raw/Kconfig > > > +++ b/drivers/mtd/nand/raw/Kconfig > > > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > > > if MTD_RAW_NAND > > > > > > config MTD_NAND_ECC_SW_BCH > > > - tristate "Support software BCH ECC" > > > + bool "Support software BCH ECC" > > > select BCH > > > default n > > > help > > > > Should be fixed with the following diff squashed into: > > > > 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > > > > --->8--- > > diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > > index b8106651f807..06ce2b655c13 100644 > > --- a/include/linux/mtd/nand_bch.h > > +++ b/include/linux/mtd/nand_bch.h > > @@ -15,7 +15,7 @@ struct mtd_info; > > struct nand_chip; > > struct nand_bch_control; > > > > -#if defined(CONFIG_MTD_NAND_ECC_BCH) > > +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > > > > static inline int mtd_nand_has_bch(void) { return 1; } > > > > @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > > */ > > void nand_bch_free(struct nand_bch_control *nbc); > > > > -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > > +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > > > > static inline int mtd_nand_has_bch(void) { return 0; } > > > > @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > > > > static inline void nand_bch_free(struct nand_bch_control *nbc) {} > > > > -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > > +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > > > > #endif /* __MTD_NAND_BCH_H__ */ > > Sorry, I didn't look at the right branch, this part of the code was > correct, but we still have a problem to express the RAW_NAND(y) -> > SW_BCH(y) dependency. This one should do the trick, though imply means MTD_NAND_ECC_SW_BCH will now be selected by default. --->8--- diff --git a/drivers/mtd/devices/Kconfig b/drivers/mtd/devices/Kconfig index 7fcdaf6c279d..f9258d666846 100644 --- a/drivers/mtd/devices/Kconfig +++ b/drivers/mtd/devices/Kconfig @@ -207,7 +207,7 @@ comment "Disk-On-Chip Device Drivers" config MTD_DOCG3 tristate "M-Systems Disk-On-Chip G3" select BCH - select BCH_CONST_PARAMS if !CONFIG_MTD_NAND_ECC_SW_BCH + select BCH_CONST_PARAMS if !MTD_NAND_ECC_SW_BCH select BITREVERSE help This provides an MTD device driver for the M-Systems DiskOnChip diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig index 615d738be411..65c85fc53c41 100644 --- a/drivers/mtd/nand/raw/Kconfig +++ b/drivers/mtd/nand/raw/Kconfig @@ -9,18 +9,6 @@ config MTD_NAND_ECC_SW_HAMMING_SMC Software ECC according to the Smart Media Specification. The original Linux implementation had byte 0 and 1 swapped. -menuconfig MTD_RAW_NAND - tristate "Raw/Parallel NAND Device Support" - depends on MTD - select MTD_NAND_CORE - select MTD_NAND_ECC_SW_HAMMING - help - This enables support for accessing all type of raw/parallel - NAND flash devices. For further information see - <http://www.linux-mtd.infradead.org/doc/nand.html>. - -if MTD_RAW_NAND - config MTD_NAND_ECC_SW_BCH tristate "Support software BCH ECC" select BCH @@ -31,6 +19,19 @@ config MTD_NAND_ECC_SW_BCH ECC codes. They are used with NAND devices requiring more than 1 bit of error correction. +menuconfig MTD_RAW_NAND + tristate "Raw/Parallel NAND Device Support" + depends on MTD + select MTD_NAND_CORE + select MTD_NAND_ECC_SW_HAMMING + imply MTD_NAND_ECC_SW_BCH + help + This enables support for accessing all type of raw/parallel + NAND flash devices. For further information see + <http://www.linux-mtd.infradead.org/doc/nand.html>. + +if MTD_RAW_NAND + comment "Raw/parallel NAND flash controllers" config MTD_NAND_DENALI ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 13:58 ` Boris Brezillon @ 2019-04-10 14:22 ` YueHaibing -1 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-10 14:22 UTC (permalink / raw) To: Boris Brezillon Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, miquel.raynal, computersforpeace On 2019/4/10 21:58, Boris Brezillon wrote: > On Wed, 10 Apr 2019 15:39:28 +0200 > Boris Brezillon <boris.brezillon@collabora.com> wrote: > >> On Wed, 10 Apr 2019 21:07:47 +0800 >> Yue Haibing <yuehaibing@huawei.com> wrote: >> >>> From: YueHaibing <yuehaibing@huawei.com> >>> >>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>> is set to module: >>> >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>> (.text+0xef6): undefined reference to `nand_bch_free' >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>> >>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>> because MTD_RAW_NAND need it while linked. >>> >>> Reported-by: Hulk Robot <hulkci@huawei.com> >>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >> >> Nope, it's not this one that introduced the regression. >> >> >> >>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>> --- >>> drivers/mtd/nand/raw/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>> index 615d738..0500c42 100644 >>> --- a/drivers/mtd/nand/raw/Kconfig >>> +++ b/drivers/mtd/nand/raw/Kconfig >>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>> if MTD_RAW_NAND >>> >>> config MTD_NAND_ECC_SW_BCH >>> - tristate "Support software BCH ECC" >>> + bool "Support software BCH ECC" >>> select BCH >>> default n >>> help >> >> Should be fixed with the following diff squashed into: >> >> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >> >> --->8--- >> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >> index b8106651f807..06ce2b655c13 100644 >> --- a/include/linux/mtd/nand_bch.h >> +++ b/include/linux/mtd/nand_bch.h >> @@ -15,7 +15,7 @@ struct mtd_info; >> struct nand_chip; >> struct nand_bch_control; >> >> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >> >> static inline int mtd_nand_has_bch(void) { return 1; } >> >> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >> */ >> void nand_bch_free(struct nand_bch_control *nbc); >> >> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >> >> static inline int mtd_nand_has_bch(void) { return 0; } >> >> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >> >> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >> >> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >> >> #endif /* __MTD_NAND_BCH_H__ */ > > Sorry, I didn't look at the right branch, this part of the code was > correct, but we still have a problem to express the RAW_NAND(y) -> > SW_BCH(y) dependency. It seems this dependency is not always need, case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > . > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 14:22 ` YueHaibing 0 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-10 14:22 UTC (permalink / raw) To: Boris Brezillon Cc: miquel.raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On 2019/4/10 21:58, Boris Brezillon wrote: > On Wed, 10 Apr 2019 15:39:28 +0200 > Boris Brezillon <boris.brezillon@collabora.com> wrote: > >> On Wed, 10 Apr 2019 21:07:47 +0800 >> Yue Haibing <yuehaibing@huawei.com> wrote: >> >>> From: YueHaibing <yuehaibing@huawei.com> >>> >>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>> is set to module: >>> >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>> (.text+0xef6): undefined reference to `nand_bch_free' >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>> >>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>> because MTD_RAW_NAND need it while linked. >>> >>> Reported-by: Hulk Robot <hulkci@huawei.com> >>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >> >> Nope, it's not this one that introduced the regression. >> >> >> >>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>> --- >>> drivers/mtd/nand/raw/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>> index 615d738..0500c42 100644 >>> --- a/drivers/mtd/nand/raw/Kconfig >>> +++ b/drivers/mtd/nand/raw/Kconfig >>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>> if MTD_RAW_NAND >>> >>> config MTD_NAND_ECC_SW_BCH >>> - tristate "Support software BCH ECC" >>> + bool "Support software BCH ECC" >>> select BCH >>> default n >>> help >> >> Should be fixed with the following diff squashed into: >> >> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >> >> --->8--- >> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >> index b8106651f807..06ce2b655c13 100644 >> --- a/include/linux/mtd/nand_bch.h >> +++ b/include/linux/mtd/nand_bch.h >> @@ -15,7 +15,7 @@ struct mtd_info; >> struct nand_chip; >> struct nand_bch_control; >> >> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >> >> static inline int mtd_nand_has_bch(void) { return 1; } >> >> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >> */ >> void nand_bch_free(struct nand_bch_control *nbc); >> >> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >> >> static inline int mtd_nand_has_bch(void) { return 0; } >> >> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >> >> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >> >> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >> >> #endif /* __MTD_NAND_BCH_H__ */ > > Sorry, I didn't look at the right branch, this part of the code was > correct, but we still have a problem to express the RAW_NAND(y) -> > SW_BCH(y) dependency. It seems this dependency is not always need, case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > . > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 14:22 ` YueHaibing @ 2019-04-10 14:29 ` Boris Brezillon -1 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 14:29 UTC (permalink / raw) To: YueHaibing Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, miquel.raynal, computersforpeace On Wed, 10 Apr 2019 22:22:16 +0800 YueHaibing <yuehaibing@huawei.com> wrote: > On 2019/4/10 21:58, Boris Brezillon wrote: > > On Wed, 10 Apr 2019 15:39:28 +0200 > > Boris Brezillon <boris.brezillon@collabora.com> wrote: > > > >> On Wed, 10 Apr 2019 21:07:47 +0800 > >> Yue Haibing <yuehaibing@huawei.com> wrote: > >> > >>> From: YueHaibing <yuehaibing@huawei.com> > >>> > >>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > >>> is set to module: > >>> > >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > >>> (.text+0xef6): undefined reference to `nand_bch_free' > >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > >>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > >>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > >>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > >>> > >>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > >>> because MTD_RAW_NAND need it while linked. > >>> > >>> Reported-by: Hulk Robot <hulkci@huawei.com> > >>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > >> > >> Nope, it's not this one that introduced the regression. > >> > >> > >> > >>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > >>> --- > >>> drivers/mtd/nand/raw/Kconfig | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > >>> index 615d738..0500c42 100644 > >>> --- a/drivers/mtd/nand/raw/Kconfig > >>> +++ b/drivers/mtd/nand/raw/Kconfig > >>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > >>> if MTD_RAW_NAND > >>> > >>> config MTD_NAND_ECC_SW_BCH > >>> - tristate "Support software BCH ECC" > >>> + bool "Support software BCH ECC" > >>> select BCH > >>> default n > >>> help > >> > >> Should be fixed with the following diff squashed into: > >> > >> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > >> > >> --->8--- > >> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > >> index b8106651f807..06ce2b655c13 100644 > >> --- a/include/linux/mtd/nand_bch.h > >> +++ b/include/linux/mtd/nand_bch.h > >> @@ -15,7 +15,7 @@ struct mtd_info; > >> struct nand_chip; > >> struct nand_bch_control; > >> > >> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > >> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > >> > >> static inline int mtd_nand_has_bch(void) { return 1; } > >> > >> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > >> */ > >> void nand_bch_free(struct nand_bch_control *nbc); > >> > >> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > >> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > >> > >> static inline int mtd_nand_has_bch(void) { return 0; } > >> > >> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > >> > >> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > >> > >> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > >> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > >> > >> #endif /* __MTD_NAND_BCH_H__ */ > > > > Sorry, I didn't look at the right branch, this part of the code was > > correct, but we still have a problem to express the RAW_NAND(y) -> > > SW_BCH(y) dependency. > > It seems this dependency is not always need, > > case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. Yes, I know, but forcing nand_bch to a be a boolean is not the right solution either, hence my suggestion to use 'imply'. ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 14:29 ` Boris Brezillon 0 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 14:29 UTC (permalink / raw) To: YueHaibing Cc: miquel.raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On Wed, 10 Apr 2019 22:22:16 +0800 YueHaibing <yuehaibing@huawei.com> wrote: > On 2019/4/10 21:58, Boris Brezillon wrote: > > On Wed, 10 Apr 2019 15:39:28 +0200 > > Boris Brezillon <boris.brezillon@collabora.com> wrote: > > > >> On Wed, 10 Apr 2019 21:07:47 +0800 > >> Yue Haibing <yuehaibing@huawei.com> wrote: > >> > >>> From: YueHaibing <yuehaibing@huawei.com> > >>> > >>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > >>> is set to module: > >>> > >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > >>> (.text+0xef6): undefined reference to `nand_bch_free' > >>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > >>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > >>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > >>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > >>> > >>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > >>> because MTD_RAW_NAND need it while linked. > >>> > >>> Reported-by: Hulk Robot <hulkci@huawei.com> > >>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > >> > >> Nope, it's not this one that introduced the regression. > >> > >> > >> > >>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > >>> --- > >>> drivers/mtd/nand/raw/Kconfig | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > >>> index 615d738..0500c42 100644 > >>> --- a/drivers/mtd/nand/raw/Kconfig > >>> +++ b/drivers/mtd/nand/raw/Kconfig > >>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > >>> if MTD_RAW_NAND > >>> > >>> config MTD_NAND_ECC_SW_BCH > >>> - tristate "Support software BCH ECC" > >>> + bool "Support software BCH ECC" > >>> select BCH > >>> default n > >>> help > >> > >> Should be fixed with the following diff squashed into: > >> > >> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > >> > >> --->8--- > >> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > >> index b8106651f807..06ce2b655c13 100644 > >> --- a/include/linux/mtd/nand_bch.h > >> +++ b/include/linux/mtd/nand_bch.h > >> @@ -15,7 +15,7 @@ struct mtd_info; > >> struct nand_chip; > >> struct nand_bch_control; > >> > >> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > >> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > >> > >> static inline int mtd_nand_has_bch(void) { return 1; } > >> > >> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > >> */ > >> void nand_bch_free(struct nand_bch_control *nbc); > >> > >> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > >> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > >> > >> static inline int mtd_nand_has_bch(void) { return 0; } > >> > >> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > >> > >> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > >> > >> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > >> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > >> > >> #endif /* __MTD_NAND_BCH_H__ */ > > > > Sorry, I didn't look at the right branch, this part of the code was > > correct, but we still have a problem to express the RAW_NAND(y) -> > > SW_BCH(y) dependency. > > It seems this dependency is not always need, > > case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. Yes, I know, but forcing nand_bch to a be a boolean is not the right solution either, hence my suggestion to use 'imply'. ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 14:29 ` Boris Brezillon @ 2019-04-10 15:03 ` YueHaibing -1 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-10 15:03 UTC (permalink / raw) To: Boris Brezillon Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, miquel.raynal, computersforpeace On 2019/4/10 22:29, Boris Brezillon wrote: > On Wed, 10 Apr 2019 22:22:16 +0800 > YueHaibing <yuehaibing@huawei.com> wrote: > >> On 2019/4/10 21:58, Boris Brezillon wrote: >>> On Wed, 10 Apr 2019 15:39:28 +0200 >>> Boris Brezillon <boris.brezillon@collabora.com> wrote: >>> >>>> On Wed, 10 Apr 2019 21:07:47 +0800 >>>> Yue Haibing <yuehaibing@huawei.com> wrote: >>>> >>>>> From: YueHaibing <yuehaibing@huawei.com> >>>>> >>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>>>> is set to module: >>>>> >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>>>> (.text+0xef6): undefined reference to `nand_bch_free' >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>>>> >>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>>>> because MTD_RAW_NAND need it while linked. >>>>> >>>>> Reported-by: Hulk Robot <hulkci@huawei.com> >>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >>>> >>>> Nope, it's not this one that introduced the regression. >>>> >>>> >>>> >>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>>>> --- >>>>> drivers/mtd/nand/raw/Kconfig | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>>>> index 615d738..0500c42 100644 >>>>> --- a/drivers/mtd/nand/raw/Kconfig >>>>> +++ b/drivers/mtd/nand/raw/Kconfig >>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>>>> if MTD_RAW_NAND >>>>> >>>>> config MTD_NAND_ECC_SW_BCH >>>>> - tristate "Support software BCH ECC" >>>>> + bool "Support software BCH ECC" >>>>> select BCH >>>>> default n >>>>> help >>>> >>>> Should be fixed with the following diff squashed into: >>>> >>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >>>> >>>> --->8--- >>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >>>> index b8106651f807..06ce2b655c13 100644 >>>> --- a/include/linux/mtd/nand_bch.h >>>> +++ b/include/linux/mtd/nand_bch.h >>>> @@ -15,7 +15,7 @@ struct mtd_info; >>>> struct nand_chip; >>>> struct nand_bch_control; >>>> >>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >>>> >>>> static inline int mtd_nand_has_bch(void) { return 1; } >>>> >>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >>>> */ >>>> void nand_bch_free(struct nand_bch_control *nbc); >>>> >>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >>>> >>>> static inline int mtd_nand_has_bch(void) { return 0; } >>>> >>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >>>> >>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >>>> >>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >>>> >>>> #endif /* __MTD_NAND_BCH_H__ */ >>> >>> Sorry, I didn't look at the right branch, this part of the code was >>> correct, but we still have a problem to express the RAW_NAND(y) -> >>> SW_BCH(y) dependency. >> >> It seems this dependency is not always need, >> >> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > Yes, I know, but forcing nand_bch to a be a boolean is not the right > solution either, hence my suggestion to use 'imply'. Got it, Thanks, will send v2 as your suggestion. > > . > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 15:03 ` YueHaibing 0 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-10 15:03 UTC (permalink / raw) To: Boris Brezillon Cc: miquel.raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On 2019/4/10 22:29, Boris Brezillon wrote: > On Wed, 10 Apr 2019 22:22:16 +0800 > YueHaibing <yuehaibing@huawei.com> wrote: > >> On 2019/4/10 21:58, Boris Brezillon wrote: >>> On Wed, 10 Apr 2019 15:39:28 +0200 >>> Boris Brezillon <boris.brezillon@collabora.com> wrote: >>> >>>> On Wed, 10 Apr 2019 21:07:47 +0800 >>>> Yue Haibing <yuehaibing@huawei.com> wrote: >>>> >>>>> From: YueHaibing <yuehaibing@huawei.com> >>>>> >>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>>>> is set to module: >>>>> >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>>>> (.text+0xef6): undefined reference to `nand_bch_free' >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>>>> >>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>>>> because MTD_RAW_NAND need it while linked. >>>>> >>>>> Reported-by: Hulk Robot <hulkci@huawei.com> >>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >>>> >>>> Nope, it's not this one that introduced the regression. >>>> >>>> >>>> >>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>>>> --- >>>>> drivers/mtd/nand/raw/Kconfig | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>>>> index 615d738..0500c42 100644 >>>>> --- a/drivers/mtd/nand/raw/Kconfig >>>>> +++ b/drivers/mtd/nand/raw/Kconfig >>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>>>> if MTD_RAW_NAND >>>>> >>>>> config MTD_NAND_ECC_SW_BCH >>>>> - tristate "Support software BCH ECC" >>>>> + bool "Support software BCH ECC" >>>>> select BCH >>>>> default n >>>>> help >>>> >>>> Should be fixed with the following diff squashed into: >>>> >>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >>>> >>>> --->8--- >>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >>>> index b8106651f807..06ce2b655c13 100644 >>>> --- a/include/linux/mtd/nand_bch.h >>>> +++ b/include/linux/mtd/nand_bch.h >>>> @@ -15,7 +15,7 @@ struct mtd_info; >>>> struct nand_chip; >>>> struct nand_bch_control; >>>> >>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >>>> >>>> static inline int mtd_nand_has_bch(void) { return 1; } >>>> >>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >>>> */ >>>> void nand_bch_free(struct nand_bch_control *nbc); >>>> >>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >>>> >>>> static inline int mtd_nand_has_bch(void) { return 0; } >>>> >>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >>>> >>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >>>> >>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >>>> >>>> #endif /* __MTD_NAND_BCH_H__ */ >>> >>> Sorry, I didn't look at the right branch, this part of the code was >>> correct, but we still have a problem to express the RAW_NAND(y) -> >>> SW_BCH(y) dependency. >> >> It seems this dependency is not always need, >> >> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > Yes, I know, but forcing nand_bch to a be a boolean is not the right > solution either, hence my suggestion to use 'imply'. Got it, Thanks, will send v2 as your suggestion. > > . > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 15:03 ` YueHaibing @ 2019-04-10 16:18 ` Miquel Raynal -1 siblings, 0 replies; 24+ messages in thread From: Miquel Raynal @ 2019-04-10 16:18 UTC (permalink / raw) To: YueHaibing Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, Boris Brezillon, linux-mtd, computersforpeace Hi YueHaibing, YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 +0800: > On 2019/4/10 22:29, Boris Brezillon wrote: > > On Wed, 10 Apr 2019 22:22:16 +0800 > > YueHaibing <yuehaibing@huawei.com> wrote: > > > >> On 2019/4/10 21:58, Boris Brezillon wrote: > >>> On Wed, 10 Apr 2019 15:39:28 +0200 > >>> Boris Brezillon <boris.brezillon@collabora.com> wrote: > >>> > >>>> On Wed, 10 Apr 2019 21:07:47 +0800 > >>>> Yue Haibing <yuehaibing@huawei.com> wrote: > >>>> > >>>>> From: YueHaibing <yuehaibing@huawei.com> > >>>>> > >>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > >>>>> is set to module: > >>>>> > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > >>>>> (.text+0xef6): undefined reference to `nand_bch_free' > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > >>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > >>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > >>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > >>>>> > >>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > >>>>> because MTD_RAW_NAND need it while linked. > >>>>> > >>>>> Reported-by: Hulk Robot <hulkci@huawei.com> > >>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > >>>> > >>>> Nope, it's not this one that introduced the regression. > >>>> > >>>> > >>>> > >>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > >>>>> --- > >>>>> drivers/mtd/nand/raw/Kconfig | 2 +- > >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>>> > >>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > >>>>> index 615d738..0500c42 100644 > >>>>> --- a/drivers/mtd/nand/raw/Kconfig > >>>>> +++ b/drivers/mtd/nand/raw/Kconfig > >>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > >>>>> if MTD_RAW_NAND > >>>>> > >>>>> config MTD_NAND_ECC_SW_BCH > >>>>> - tristate "Support software BCH ECC" > >>>>> + bool "Support software BCH ECC" > >>>>> select BCH > >>>>> default n > >>>>> help > >>>> > >>>> Should be fixed with the following diff squashed into: > >>>> > >>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > >>>> > >>>> --->8--- > >>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > >>>> index b8106651f807..06ce2b655c13 100644 > >>>> --- a/include/linux/mtd/nand_bch.h > >>>> +++ b/include/linux/mtd/nand_bch.h > >>>> @@ -15,7 +15,7 @@ struct mtd_info; > >>>> struct nand_chip; > >>>> struct nand_bch_control; > >>>> > >>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > >>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > >>>> > >>>> static inline int mtd_nand_has_bch(void) { return 1; } > >>>> > >>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > >>>> */ > >>>> void nand_bch_free(struct nand_bch_control *nbc); > >>>> > >>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > >>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>> > >>>> static inline int mtd_nand_has_bch(void) { return 0; } > >>>> > >>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > >>>> > >>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > >>>> > >>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > >>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>> > >>>> #endif /* __MTD_NAND_BCH_H__ */ > >>> > >>> Sorry, I didn't look at the right branch, this part of the code was > >>> correct, but we still have a problem to express the RAW_NAND(y) -> > >>> SW_BCH(y) dependency. > >> > >> It seems this dependency is not always need, > >> > >> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > > > Yes, I know, but forcing nand_bch to a be a boolean is not the right > > solution either, hence my suggestion to use 'imply'. Why exactly? SW BCH is going to be converted to the generic ECC engine abstraction and IIRC you told me that it would not be problematic to turn it into a boolean? Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 16:18 ` Miquel Raynal 0 siblings, 0 replies; 24+ messages in thread From: Miquel Raynal @ 2019-04-10 16:18 UTC (permalink / raw) To: YueHaibing Cc: Boris Brezillon, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd Hi YueHaibing, YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 +0800: > On 2019/4/10 22:29, Boris Brezillon wrote: > > On Wed, 10 Apr 2019 22:22:16 +0800 > > YueHaibing <yuehaibing@huawei.com> wrote: > > > >> On 2019/4/10 21:58, Boris Brezillon wrote: > >>> On Wed, 10 Apr 2019 15:39:28 +0200 > >>> Boris Brezillon <boris.brezillon@collabora.com> wrote: > >>> > >>>> On Wed, 10 Apr 2019 21:07:47 +0800 > >>>> Yue Haibing <yuehaibing@huawei.com> wrote: > >>>> > >>>>> From: YueHaibing <yuehaibing@huawei.com> > >>>>> > >>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > >>>>> is set to module: > >>>>> > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > >>>>> (.text+0xef6): undefined reference to `nand_bch_free' > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > >>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > >>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > >>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > >>>>> > >>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > >>>>> because MTD_RAW_NAND need it while linked. > >>>>> > >>>>> Reported-by: Hulk Robot <hulkci@huawei.com> > >>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > >>>> > >>>> Nope, it's not this one that introduced the regression. > >>>> > >>>> > >>>> > >>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > >>>>> --- > >>>>> drivers/mtd/nand/raw/Kconfig | 2 +- > >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>>> > >>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > >>>>> index 615d738..0500c42 100644 > >>>>> --- a/drivers/mtd/nand/raw/Kconfig > >>>>> +++ b/drivers/mtd/nand/raw/Kconfig > >>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > >>>>> if MTD_RAW_NAND > >>>>> > >>>>> config MTD_NAND_ECC_SW_BCH > >>>>> - tristate "Support software BCH ECC" > >>>>> + bool "Support software BCH ECC" > >>>>> select BCH > >>>>> default n > >>>>> help > >>>> > >>>> Should be fixed with the following diff squashed into: > >>>> > >>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > >>>> > >>>> --->8--- > >>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > >>>> index b8106651f807..06ce2b655c13 100644 > >>>> --- a/include/linux/mtd/nand_bch.h > >>>> +++ b/include/linux/mtd/nand_bch.h > >>>> @@ -15,7 +15,7 @@ struct mtd_info; > >>>> struct nand_chip; > >>>> struct nand_bch_control; > >>>> > >>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > >>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > >>>> > >>>> static inline int mtd_nand_has_bch(void) { return 1; } > >>>> > >>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > >>>> */ > >>>> void nand_bch_free(struct nand_bch_control *nbc); > >>>> > >>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > >>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>> > >>>> static inline int mtd_nand_has_bch(void) { return 0; } > >>>> > >>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > >>>> > >>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > >>>> > >>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > >>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>> > >>>> #endif /* __MTD_NAND_BCH_H__ */ > >>> > >>> Sorry, I didn't look at the right branch, this part of the code was > >>> correct, but we still have a problem to express the RAW_NAND(y) -> > >>> SW_BCH(y) dependency. > >> > >> It seems this dependency is not always need, > >> > >> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > > > Yes, I know, but forcing nand_bch to a be a boolean is not the right > > solution either, hence my suggestion to use 'imply'. Why exactly? SW BCH is going to be converted to the generic ECC engine abstraction and IIRC you told me that it would not be problematic to turn it into a boolean? Thanks, Miquèl ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 16:18 ` Miquel Raynal @ 2019-04-10 17:03 ` Boris Brezillon -1 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 17:03 UTC (permalink / raw) To: Miquel Raynal Cc: bbrezillon, richard, YueHaibing, linux-kernel, marek.vasut, paul.burton, linux-mtd, computersforpeace On Wed, 10 Apr 2019 18:18:43 +0200 Miquel Raynal <miquel.raynal@bootlin.com> wrote: > Hi YueHaibing, > > YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 > +0800: > > > On 2019/4/10 22:29, Boris Brezillon wrote: > > > On Wed, 10 Apr 2019 22:22:16 +0800 > > > YueHaibing <yuehaibing@huawei.com> wrote: > > > > > >> On 2019/4/10 21:58, Boris Brezillon wrote: > > >>> On Wed, 10 Apr 2019 15:39:28 +0200 > > >>> Boris Brezillon <boris.brezillon@collabora.com> wrote: > > >>> > > >>>> On Wed, 10 Apr 2019 21:07:47 +0800 > > >>>> Yue Haibing <yuehaibing@huawei.com> wrote: > > >>>> > > >>>>> From: YueHaibing <yuehaibing@huawei.com> > > >>>>> > > >>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > > >>>>> is set to module: > > >>>>> > > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > > >>>>> (.text+0xef6): undefined reference to `nand_bch_free' > > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > > >>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > > >>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > > >>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > >>>>> > > >>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > > >>>>> because MTD_RAW_NAND need it while linked. > > >>>>> > > >>>>> Reported-by: Hulk Robot <hulkci@huawei.com> > > >>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > > >>>> > > >>>> Nope, it's not this one that introduced the regression. > > >>>> > > >>>> > > >>>> > > >>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > > >>>>> --- > > >>>>> drivers/mtd/nand/raw/Kconfig | 2 +- > > >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > > >>>>> > > >>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > > >>>>> index 615d738..0500c42 100644 > > >>>>> --- a/drivers/mtd/nand/raw/Kconfig > > >>>>> +++ b/drivers/mtd/nand/raw/Kconfig > > >>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > > >>>>> if MTD_RAW_NAND > > >>>>> > > >>>>> config MTD_NAND_ECC_SW_BCH > > >>>>> - tristate "Support software BCH ECC" > > >>>>> + bool "Support software BCH ECC" > > >>>>> select BCH > > >>>>> default n > > >>>>> help > > >>>> > > >>>> Should be fixed with the following diff squashed into: > > >>>> > > >>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > > >>>> > > >>>> --->8--- > > >>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > > >>>> index b8106651f807..06ce2b655c13 100644 > > >>>> --- a/include/linux/mtd/nand_bch.h > > >>>> +++ b/include/linux/mtd/nand_bch.h > > >>>> @@ -15,7 +15,7 @@ struct mtd_info; > > >>>> struct nand_chip; > > >>>> struct nand_bch_control; > > >>>> > > >>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > > >>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > > >>>> > > >>>> static inline int mtd_nand_has_bch(void) { return 1; } > > >>>> > > >>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > > >>>> */ > > >>>> void nand_bch_free(struct nand_bch_control *nbc); > > >>>> > > >>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > > >>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > > >>>> > > >>>> static inline int mtd_nand_has_bch(void) { return 0; } > > >>>> > > >>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > > >>>> > > >>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > > >>>> > > >>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > > >>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > > >>>> > > >>>> #endif /* __MTD_NAND_BCH_H__ */ > > >>> > > >>> Sorry, I didn't look at the right branch, this part of the code was > > >>> correct, but we still have a problem to express the RAW_NAND(y) -> > > >>> SW_BCH(y) dependency. > > >> > > >> It seems this dependency is not always need, > > >> > > >> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > > > > > Yes, I know, but forcing nand_bch to a be a boolean is not the right > > > solution either, hence my suggestion to use 'imply'. > > Why exactly? SW BCH is going to be converted to the generic ECC engine > abstraction and IIRC you told me that it would not be problematic to > turn it into a boolean? Yes, I told you we should turn it into a boolean and embed the code in the nandcore.o object which can itself be linked as a module or embedded in the kernel image (see above for a version adapted for nand.o). That's not exactly what Yue proposed. --->8--- diff --git a/drivers/mtd/devices/Kconfig b/drivers/mtd/devices/Kconfig index 7fcdaf6c279d..f9258d666846 100644 --- a/drivers/mtd/devices/Kconfig +++ b/drivers/mtd/devices/Kconfig @@ -207,7 +207,7 @@ comment "Disk-On-Chip Device Drivers" config MTD_DOCG3 tristate "M-Systems Disk-On-Chip G3" select BCH - select BCH_CONST_PARAMS if !CONFIG_MTD_NAND_ECC_SW_BCH + select BCH_CONST_PARAMS if !MTD_NAND_ECC_SW_BCH select BITREVERSE help This provides an MTD device driver for the M-Systems DiskOnChip diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig index 615d738be411..0500c42f31cb 100644 --- a/drivers/mtd/nand/raw/Kconfig +++ b/drivers/mtd/nand/raw/Kconfig @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND if MTD_RAW_NAND config MTD_NAND_ECC_SW_BCH - tristate "Support software BCH ECC" + bool "Support software BCH ECC" select BCH default n help diff --git a/drivers/mtd/nand/raw/Makefile b/drivers/mtd/nand/raw/Makefile index 8bc6faaa3bc7..d644185fa18d 100644 --- a/drivers/mtd/nand/raw/Makefile +++ b/drivers/mtd/nand/raw/Makefile @@ -2,7 +2,6 @@ obj-$(CONFIG_MTD_RAW_NAND) += nand.o obj-$(CONFIG_MTD_NAND_ECC_SW_HAMMING) += nand_ecc.o -obj-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o obj-$(CONFIG_MTD_SM_COMMON) += sm_common.o obj-$(CONFIG_MTD_NAND_CAFE) += cafe_nand.o @@ -68,3 +67,5 @@ nand-objs += nand_macronix.o nand-objs += nand_micron.o nand-objs += nand_samsung.o nand-objs += nand_toshiba.o +nand-objs-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o +nand-objs += $(nand-objs-y) ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-10 17:03 ` Boris Brezillon 0 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-10 17:03 UTC (permalink / raw) To: Miquel Raynal Cc: YueHaibing, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On Wed, 10 Apr 2019 18:18:43 +0200 Miquel Raynal <miquel.raynal@bootlin.com> wrote: > Hi YueHaibing, > > YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 > +0800: > > > On 2019/4/10 22:29, Boris Brezillon wrote: > > > On Wed, 10 Apr 2019 22:22:16 +0800 > > > YueHaibing <yuehaibing@huawei.com> wrote: > > > > > >> On 2019/4/10 21:58, Boris Brezillon wrote: > > >>> On Wed, 10 Apr 2019 15:39:28 +0200 > > >>> Boris Brezillon <boris.brezillon@collabora.com> wrote: > > >>> > > >>>> On Wed, 10 Apr 2019 21:07:47 +0800 > > >>>> Yue Haibing <yuehaibing@huawei.com> wrote: > > >>>> > > >>>>> From: YueHaibing <yuehaibing@huawei.com> > > >>>>> > > >>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > > >>>>> is set to module: > > >>>>> > > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > > >>>>> (.text+0xef6): undefined reference to `nand_bch_free' > > >>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > > >>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > > >>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > > >>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > > >>>>> > > >>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > > >>>>> because MTD_RAW_NAND need it while linked. > > >>>>> > > >>>>> Reported-by: Hulk Robot <hulkci@huawei.com> > > >>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > > >>>> > > >>>> Nope, it's not this one that introduced the regression. > > >>>> > > >>>> > > >>>> > > >>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > > >>>>> --- > > >>>>> drivers/mtd/nand/raw/Kconfig | 2 +- > > >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > > >>>>> > > >>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > > >>>>> index 615d738..0500c42 100644 > > >>>>> --- a/drivers/mtd/nand/raw/Kconfig > > >>>>> +++ b/drivers/mtd/nand/raw/Kconfig > > >>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > > >>>>> if MTD_RAW_NAND > > >>>>> > > >>>>> config MTD_NAND_ECC_SW_BCH > > >>>>> - tristate "Support software BCH ECC" > > >>>>> + bool "Support software BCH ECC" > > >>>>> select BCH > > >>>>> default n > > >>>>> help > > >>>> > > >>>> Should be fixed with the following diff squashed into: > > >>>> > > >>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > > >>>> > > >>>> --->8--- > > >>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > > >>>> index b8106651f807..06ce2b655c13 100644 > > >>>> --- a/include/linux/mtd/nand_bch.h > > >>>> +++ b/include/linux/mtd/nand_bch.h > > >>>> @@ -15,7 +15,7 @@ struct mtd_info; > > >>>> struct nand_chip; > > >>>> struct nand_bch_control; > > >>>> > > >>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > > >>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > > >>>> > > >>>> static inline int mtd_nand_has_bch(void) { return 1; } > > >>>> > > >>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > > >>>> */ > > >>>> void nand_bch_free(struct nand_bch_control *nbc); > > >>>> > > >>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > > >>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > > >>>> > > >>>> static inline int mtd_nand_has_bch(void) { return 0; } > > >>>> > > >>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > > >>>> > > >>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > > >>>> > > >>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > > >>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > > >>>> > > >>>> #endif /* __MTD_NAND_BCH_H__ */ > > >>> > > >>> Sorry, I didn't look at the right branch, this part of the code was > > >>> correct, but we still have a problem to express the RAW_NAND(y) -> > > >>> SW_BCH(y) dependency. > > >> > > >> It seems this dependency is not always need, > > >> > > >> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > > > > > > Yes, I know, but forcing nand_bch to a be a boolean is not the right > > > solution either, hence my suggestion to use 'imply'. > > Why exactly? SW BCH is going to be converted to the generic ECC engine > abstraction and IIRC you told me that it would not be problematic to > turn it into a boolean? Yes, I told you we should turn it into a boolean and embed the code in the nandcore.o object which can itself be linked as a module or embedded in the kernel image (see above for a version adapted for nand.o). That's not exactly what Yue proposed. --->8--- diff --git a/drivers/mtd/devices/Kconfig b/drivers/mtd/devices/Kconfig index 7fcdaf6c279d..f9258d666846 100644 --- a/drivers/mtd/devices/Kconfig +++ b/drivers/mtd/devices/Kconfig @@ -207,7 +207,7 @@ comment "Disk-On-Chip Device Drivers" config MTD_DOCG3 tristate "M-Systems Disk-On-Chip G3" select BCH - select BCH_CONST_PARAMS if !CONFIG_MTD_NAND_ECC_SW_BCH + select BCH_CONST_PARAMS if !MTD_NAND_ECC_SW_BCH select BITREVERSE help This provides an MTD device driver for the M-Systems DiskOnChip diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig index 615d738be411..0500c42f31cb 100644 --- a/drivers/mtd/nand/raw/Kconfig +++ b/drivers/mtd/nand/raw/Kconfig @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND if MTD_RAW_NAND config MTD_NAND_ECC_SW_BCH - tristate "Support software BCH ECC" + bool "Support software BCH ECC" select BCH default n help diff --git a/drivers/mtd/nand/raw/Makefile b/drivers/mtd/nand/raw/Makefile index 8bc6faaa3bc7..d644185fa18d 100644 --- a/drivers/mtd/nand/raw/Makefile +++ b/drivers/mtd/nand/raw/Makefile @@ -2,7 +2,6 @@ obj-$(CONFIG_MTD_RAW_NAND) += nand.o obj-$(CONFIG_MTD_NAND_ECC_SW_HAMMING) += nand_ecc.o -obj-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o obj-$(CONFIG_MTD_SM_COMMON) += sm_common.o obj-$(CONFIG_MTD_NAND_CAFE) += cafe_nand.o @@ -68,3 +67,5 @@ nand-objs += nand_macronix.o nand-objs += nand_micron.o nand-objs += nand_samsung.o nand-objs += nand_toshiba.o +nand-objs-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o +nand-objs += $(nand-objs-y) ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-10 17:03 ` Boris Brezillon @ 2019-04-12 14:28 ` YueHaibing -1 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-12 14:28 UTC (permalink / raw) To: Boris Brezillon, Miquel Raynal Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, computersforpeace On 2019/4/11 1:03, Boris Brezillon wrote: > On Wed, 10 Apr 2019 18:18:43 +0200 > Miquel Raynal <miquel.raynal@bootlin.com> wrote: > >> Hi YueHaibing, >> >> YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 >> +0800: >> >>> On 2019/4/10 22:29, Boris Brezillon wrote: >>>> On Wed, 10 Apr 2019 22:22:16 +0800 >>>> YueHaibing <yuehaibing@huawei.com> wrote: >>>> >>>>> On 2019/4/10 21:58, Boris Brezillon wrote: >>>>>> On Wed, 10 Apr 2019 15:39:28 +0200 >>>>>> Boris Brezillon <boris.brezillon@collabora.com> wrote: >>>>>> >>>>>>> On Wed, 10 Apr 2019 21:07:47 +0800 >>>>>>> Yue Haibing <yuehaibing@huawei.com> wrote: >>>>>>> >>>>>>>> From: YueHaibing <yuehaibing@huawei.com> >>>>>>>> >>>>>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>>>>>>> is set to module: >>>>>>>> >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>>>>>>> (.text+0xef6): undefined reference to `nand_bch_free' >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>>>>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>>>>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>>>>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>>>>>>> >>>>>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>>>>>>> because MTD_RAW_NAND need it while linked. >>>>>>>> >>>>>>>> Reported-by: Hulk Robot <hulkci@huawei.com> >>>>>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >>>>>>> >>>>>>> Nope, it's not this one that introduced the regression. >>>>>>> >>>>>>> >>>>>>> >>>>>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>>>>>>> --- >>>>>>>> drivers/mtd/nand/raw/Kconfig | 2 +- >>>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>>> >>>>>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>>>>>>> index 615d738..0500c42 100644 >>>>>>>> --- a/drivers/mtd/nand/raw/Kconfig >>>>>>>> +++ b/drivers/mtd/nand/raw/Kconfig >>>>>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>>>>>>> if MTD_RAW_NAND >>>>>>>> >>>>>>>> config MTD_NAND_ECC_SW_BCH >>>>>>>> - tristate "Support software BCH ECC" >>>>>>>> + bool "Support software BCH ECC" >>>>>>>> select BCH >>>>>>>> default n >>>>>>>> help >>>>>>> >>>>>>> Should be fixed with the following diff squashed into: >>>>>>> >>>>>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >>>>>>> >>>>>>> --->8--- >>>>>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >>>>>>> index b8106651f807..06ce2b655c13 100644 >>>>>>> --- a/include/linux/mtd/nand_bch.h >>>>>>> +++ b/include/linux/mtd/nand_bch.h >>>>>>> @@ -15,7 +15,7 @@ struct mtd_info; >>>>>>> struct nand_chip; >>>>>>> struct nand_bch_control; >>>>>>> >>>>>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >>>>>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >>>>>>> >>>>>>> static inline int mtd_nand_has_bch(void) { return 1; } >>>>>>> >>>>>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >>>>>>> */ >>>>>>> void nand_bch_free(struct nand_bch_control *nbc); >>>>>>> >>>>>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >>>>>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>> >>>>>>> static inline int mtd_nand_has_bch(void) { return 0; } >>>>>>> >>>>>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >>>>>>> >>>>>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >>>>>>> >>>>>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >>>>>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>> >>>>>>> #endif /* __MTD_NAND_BCH_H__ */ >>>>>> >>>>>> Sorry, I didn't look at the right branch, this part of the code was >>>>>> correct, but we still have a problem to express the RAW_NAND(y) -> >>>>>> SW_BCH(y) dependency. >>>>> >>>>> It seems this dependency is not always need, >>>>> >>>>> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. >>>> >>>> Yes, I know, but forcing nand_bch to a be a boolean is not the right >>>> solution either, hence my suggestion to use 'imply'. >> >> Why exactly? SW BCH is going to be converted to the generic ECC engine >> abstraction and IIRC you told me that it would not be problematic to >> turn it into a boolean? > > Yes, I told you we should turn it into a boolean and embed the code in > the nandcore.o object which can itself be linked as a module or embedded > in the kernel image (see above for a version adapted for nand.o). That's > not exactly what Yue proposed. So is there anothor fix or just to pick my v2 ? https://patchwork.ozlabs.org/patch/1083480/ > > --->8--- > diff --git a/drivers/mtd/devices/Kconfig b/drivers/mtd/devices/Kconfig > index 7fcdaf6c279d..f9258d666846 100644 > --- a/drivers/mtd/devices/Kconfig > +++ b/drivers/mtd/devices/Kconfig > @@ -207,7 +207,7 @@ comment "Disk-On-Chip Device Drivers" > config MTD_DOCG3 > tristate "M-Systems Disk-On-Chip G3" > select BCH > - select BCH_CONST_PARAMS if !CONFIG_MTD_NAND_ECC_SW_BCH > + select BCH_CONST_PARAMS if !MTD_NAND_ECC_SW_BCH > select BITREVERSE > help > This provides an MTD device driver for the M-Systems DiskOnChip > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > index 615d738be411..0500c42f31cb 100644 > --- a/drivers/mtd/nand/raw/Kconfig > +++ b/drivers/mtd/nand/raw/Kconfig > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > if MTD_RAW_NAND > > config MTD_NAND_ECC_SW_BCH > - tristate "Support software BCH ECC" > + bool "Support software BCH ECC" > select BCH > default n > help > diff --git a/drivers/mtd/nand/raw/Makefile b/drivers/mtd/nand/raw/Makefile > index 8bc6faaa3bc7..d644185fa18d 100644 > --- a/drivers/mtd/nand/raw/Makefile > +++ b/drivers/mtd/nand/raw/Makefile > @@ -2,7 +2,6 @@ > > obj-$(CONFIG_MTD_RAW_NAND) += nand.o > obj-$(CONFIG_MTD_NAND_ECC_SW_HAMMING) += nand_ecc.o > -obj-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o > obj-$(CONFIG_MTD_SM_COMMON) += sm_common.o > > obj-$(CONFIG_MTD_NAND_CAFE) += cafe_nand.o > @@ -68,3 +67,5 @@ nand-objs += nand_macronix.o > nand-objs += nand_micron.o > nand-objs += nand_samsung.o > nand-objs += nand_toshiba.o > +nand-objs-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o > +nand-objs += $(nand-objs-y) > > > > . > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-12 14:28 ` YueHaibing 0 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-12 14:28 UTC (permalink / raw) To: Boris Brezillon, Miquel Raynal Cc: richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On 2019/4/11 1:03, Boris Brezillon wrote: > On Wed, 10 Apr 2019 18:18:43 +0200 > Miquel Raynal <miquel.raynal@bootlin.com> wrote: > >> Hi YueHaibing, >> >> YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 >> +0800: >> >>> On 2019/4/10 22:29, Boris Brezillon wrote: >>>> On Wed, 10 Apr 2019 22:22:16 +0800 >>>> YueHaibing <yuehaibing@huawei.com> wrote: >>>> >>>>> On 2019/4/10 21:58, Boris Brezillon wrote: >>>>>> On Wed, 10 Apr 2019 15:39:28 +0200 >>>>>> Boris Brezillon <boris.brezillon@collabora.com> wrote: >>>>>> >>>>>>> On Wed, 10 Apr 2019 21:07:47 +0800 >>>>>>> Yue Haibing <yuehaibing@huawei.com> wrote: >>>>>>> >>>>>>>> From: YueHaibing <yuehaibing@huawei.com> >>>>>>>> >>>>>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>>>>>>> is set to module: >>>>>>>> >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>>>>>>> (.text+0xef6): undefined reference to `nand_bch_free' >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>>>>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>>>>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>>>>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>>>>>>> >>>>>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>>>>>>> because MTD_RAW_NAND need it while linked. >>>>>>>> >>>>>>>> Reported-by: Hulk Robot <hulkci@huawei.com> >>>>>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >>>>>>> >>>>>>> Nope, it's not this one that introduced the regression. >>>>>>> >>>>>>> >>>>>>> >>>>>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>>>>>>> --- >>>>>>>> drivers/mtd/nand/raw/Kconfig | 2 +- >>>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>>> >>>>>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>>>>>>> index 615d738..0500c42 100644 >>>>>>>> --- a/drivers/mtd/nand/raw/Kconfig >>>>>>>> +++ b/drivers/mtd/nand/raw/Kconfig >>>>>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>>>>>>> if MTD_RAW_NAND >>>>>>>> >>>>>>>> config MTD_NAND_ECC_SW_BCH >>>>>>>> - tristate "Support software BCH ECC" >>>>>>>> + bool "Support software BCH ECC" >>>>>>>> select BCH >>>>>>>> default n >>>>>>>> help >>>>>>> >>>>>>> Should be fixed with the following diff squashed into: >>>>>>> >>>>>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >>>>>>> >>>>>>> --->8--- >>>>>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >>>>>>> index b8106651f807..06ce2b655c13 100644 >>>>>>> --- a/include/linux/mtd/nand_bch.h >>>>>>> +++ b/include/linux/mtd/nand_bch.h >>>>>>> @@ -15,7 +15,7 @@ struct mtd_info; >>>>>>> struct nand_chip; >>>>>>> struct nand_bch_control; >>>>>>> >>>>>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >>>>>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >>>>>>> >>>>>>> static inline int mtd_nand_has_bch(void) { return 1; } >>>>>>> >>>>>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >>>>>>> */ >>>>>>> void nand_bch_free(struct nand_bch_control *nbc); >>>>>>> >>>>>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >>>>>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>> >>>>>>> static inline int mtd_nand_has_bch(void) { return 0; } >>>>>>> >>>>>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >>>>>>> >>>>>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >>>>>>> >>>>>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >>>>>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>> >>>>>>> #endif /* __MTD_NAND_BCH_H__ */ >>>>>> >>>>>> Sorry, I didn't look at the right branch, this part of the code was >>>>>> correct, but we still have a problem to express the RAW_NAND(y) -> >>>>>> SW_BCH(y) dependency. >>>>> >>>>> It seems this dependency is not always need, >>>>> >>>>> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. >>>> >>>> Yes, I know, but forcing nand_bch to a be a boolean is not the right >>>> solution either, hence my suggestion to use 'imply'. >> >> Why exactly? SW BCH is going to be converted to the generic ECC engine >> abstraction and IIRC you told me that it would not be problematic to >> turn it into a boolean? > > Yes, I told you we should turn it into a boolean and embed the code in > the nandcore.o object which can itself be linked as a module or embedded > in the kernel image (see above for a version adapted for nand.o). That's > not exactly what Yue proposed. So is there anothor fix or just to pick my v2 ? https://patchwork.ozlabs.org/patch/1083480/ > > --->8--- > diff --git a/drivers/mtd/devices/Kconfig b/drivers/mtd/devices/Kconfig > index 7fcdaf6c279d..f9258d666846 100644 > --- a/drivers/mtd/devices/Kconfig > +++ b/drivers/mtd/devices/Kconfig > @@ -207,7 +207,7 @@ comment "Disk-On-Chip Device Drivers" > config MTD_DOCG3 > tristate "M-Systems Disk-On-Chip G3" > select BCH > - select BCH_CONST_PARAMS if !CONFIG_MTD_NAND_ECC_SW_BCH > + select BCH_CONST_PARAMS if !MTD_NAND_ECC_SW_BCH > select BITREVERSE > help > This provides an MTD device driver for the M-Systems DiskOnChip > diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > index 615d738be411..0500c42f31cb 100644 > --- a/drivers/mtd/nand/raw/Kconfig > +++ b/drivers/mtd/nand/raw/Kconfig > @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > if MTD_RAW_NAND > > config MTD_NAND_ECC_SW_BCH > - tristate "Support software BCH ECC" > + bool "Support software BCH ECC" > select BCH > default n > help > diff --git a/drivers/mtd/nand/raw/Makefile b/drivers/mtd/nand/raw/Makefile > index 8bc6faaa3bc7..d644185fa18d 100644 > --- a/drivers/mtd/nand/raw/Makefile > +++ b/drivers/mtd/nand/raw/Makefile > @@ -2,7 +2,6 @@ > > obj-$(CONFIG_MTD_RAW_NAND) += nand.o > obj-$(CONFIG_MTD_NAND_ECC_SW_HAMMING) += nand_ecc.o > -obj-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o > obj-$(CONFIG_MTD_SM_COMMON) += sm_common.o > > obj-$(CONFIG_MTD_NAND_CAFE) += cafe_nand.o > @@ -68,3 +67,5 @@ nand-objs += nand_macronix.o > nand-objs += nand_micron.o > nand-objs += nand_samsung.o > nand-objs += nand_toshiba.o > +nand-objs-$(CONFIG_MTD_NAND_ECC_SW_BCH) += nand_bch.o > +nand-objs += $(nand-objs-y) > > > > . > ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-12 14:28 ` YueHaibing @ 2019-04-12 15:04 ` Boris Brezillon -1 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-12 15:04 UTC (permalink / raw) To: YueHaibing Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, Miquel Raynal, computersforpeace On Fri, 12 Apr 2019 22:28:57 +0800 YueHaibing <yuehaibing@huawei.com> wrote: > On 2019/4/11 1:03, Boris Brezillon wrote: > > On Wed, 10 Apr 2019 18:18:43 +0200 > > Miquel Raynal <miquel.raynal@bootlin.com> wrote: > > > >> Hi YueHaibing, > >> > >> YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 > >> +0800: > >> > >>> On 2019/4/10 22:29, Boris Brezillon wrote: > >>>> On Wed, 10 Apr 2019 22:22:16 +0800 > >>>> YueHaibing <yuehaibing@huawei.com> wrote: > >>>> > >>>>> On 2019/4/10 21:58, Boris Brezillon wrote: > >>>>>> On Wed, 10 Apr 2019 15:39:28 +0200 > >>>>>> Boris Brezillon <boris.brezillon@collabora.com> wrote: > >>>>>> > >>>>>>> On Wed, 10 Apr 2019 21:07:47 +0800 > >>>>>>> Yue Haibing <yuehaibing@huawei.com> wrote: > >>>>>>> > >>>>>>>> From: YueHaibing <yuehaibing@huawei.com> > >>>>>>>> > >>>>>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > >>>>>>>> is set to module: > >>>>>>>> > >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > >>>>>>>> (.text+0xef6): undefined reference to `nand_bch_free' > >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > >>>>>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > >>>>>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > >>>>>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > >>>>>>>> > >>>>>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > >>>>>>>> because MTD_RAW_NAND need it while linked. > >>>>>>>> > >>>>>>>> Reported-by: Hulk Robot <hulkci@huawei.com> > >>>>>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > >>>>>>> > >>>>>>> Nope, it's not this one that introduced the regression. > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > >>>>>>>> --- > >>>>>>>> drivers/mtd/nand/raw/Kconfig | 2 +- > >>>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>>>>>> > >>>>>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > >>>>>>>> index 615d738..0500c42 100644 > >>>>>>>> --- a/drivers/mtd/nand/raw/Kconfig > >>>>>>>> +++ b/drivers/mtd/nand/raw/Kconfig > >>>>>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > >>>>>>>> if MTD_RAW_NAND > >>>>>>>> > >>>>>>>> config MTD_NAND_ECC_SW_BCH > >>>>>>>> - tristate "Support software BCH ECC" > >>>>>>>> + bool "Support software BCH ECC" > >>>>>>>> select BCH > >>>>>>>> default n > >>>>>>>> help > >>>>>>> > >>>>>>> Should be fixed with the following diff squashed into: > >>>>>>> > >>>>>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > >>>>>>> > >>>>>>> --->8--- > >>>>>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > >>>>>>> index b8106651f807..06ce2b655c13 100644 > >>>>>>> --- a/include/linux/mtd/nand_bch.h > >>>>>>> +++ b/include/linux/mtd/nand_bch.h > >>>>>>> @@ -15,7 +15,7 @@ struct mtd_info; > >>>>>>> struct nand_chip; > >>>>>>> struct nand_bch_control; > >>>>>>> > >>>>>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > >>>>>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > >>>>>>> > >>>>>>> static inline int mtd_nand_has_bch(void) { return 1; } > >>>>>>> > >>>>>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > >>>>>>> */ > >>>>>>> void nand_bch_free(struct nand_bch_control *nbc); > >>>>>>> > >>>>>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > >>>>>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>>>>> > >>>>>>> static inline int mtd_nand_has_bch(void) { return 0; } > >>>>>>> > >>>>>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > >>>>>>> > >>>>>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > >>>>>>> > >>>>>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > >>>>>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>>>>> > >>>>>>> #endif /* __MTD_NAND_BCH_H__ */ > >>>>>> > >>>>>> Sorry, I didn't look at the right branch, this part of the code was > >>>>>> correct, but we still have a problem to express the RAW_NAND(y) -> > >>>>>> SW_BCH(y) dependency. > >>>>> > >>>>> It seems this dependency is not always need, > >>>>> > >>>>> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > >>>> > >>>> Yes, I know, but forcing nand_bch to a be a boolean is not the right > >>>> solution either, hence my suggestion to use 'imply'. > >> > >> Why exactly? SW BCH is going to be converted to the generic ECC engine > >> abstraction and IIRC you told me that it would not be problematic to > >> turn it into a boolean? > > > > Yes, I told you we should turn it into a boolean and embed the code in > > the nandcore.o object which can itself be linked as a module or embedded > > in the kernel image (see above for a version adapted for nand.o). That's > > not exactly what Yue proposed. > > So is there anothor fix or just to pick my v2 ? > > https://patchwork.ozlabs.org/patch/1083480/ Looks like someone posted it already: https://patchwork.ozlabs.org/patch/1083603/ ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-12 15:04 ` Boris Brezillon 0 siblings, 0 replies; 24+ messages in thread From: Boris Brezillon @ 2019-04-12 15:04 UTC (permalink / raw) To: YueHaibing Cc: Miquel Raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On Fri, 12 Apr 2019 22:28:57 +0800 YueHaibing <yuehaibing@huawei.com> wrote: > On 2019/4/11 1:03, Boris Brezillon wrote: > > On Wed, 10 Apr 2019 18:18:43 +0200 > > Miquel Raynal <miquel.raynal@bootlin.com> wrote: > > > >> Hi YueHaibing, > >> > >> YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 > >> +0800: > >> > >>> On 2019/4/10 22:29, Boris Brezillon wrote: > >>>> On Wed, 10 Apr 2019 22:22:16 +0800 > >>>> YueHaibing <yuehaibing@huawei.com> wrote: > >>>> > >>>>> On 2019/4/10 21:58, Boris Brezillon wrote: > >>>>>> On Wed, 10 Apr 2019 15:39:28 +0200 > >>>>>> Boris Brezillon <boris.brezillon@collabora.com> wrote: > >>>>>> > >>>>>>> On Wed, 10 Apr 2019 21:07:47 +0800 > >>>>>>> Yue Haibing <yuehaibing@huawei.com> wrote: > >>>>>>> > >>>>>>>> From: YueHaibing <yuehaibing@huawei.com> > >>>>>>>> > >>>>>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH > >>>>>>>> is set to module: > >>>>>>>> > >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': > >>>>>>>> (.text+0xef6): undefined reference to `nand_bch_free' > >>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': > >>>>>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' > >>>>>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' > >>>>>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' > >>>>>>>> > >>>>>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, > >>>>>>>> because MTD_RAW_NAND need it while linked. > >>>>>>>> > >>>>>>>> Reported-by: Hulk Robot <hulkci@huawei.com> > >>>>>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" > >>>>>>> > >>>>>>> Nope, it's not this one that introduced the regression. > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > >>>>>>>> --- > >>>>>>>> drivers/mtd/nand/raw/Kconfig | 2 +- > >>>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>>>>>> > >>>>>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig > >>>>>>>> index 615d738..0500c42 100644 > >>>>>>>> --- a/drivers/mtd/nand/raw/Kconfig > >>>>>>>> +++ b/drivers/mtd/nand/raw/Kconfig > >>>>>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND > >>>>>>>> if MTD_RAW_NAND > >>>>>>>> > >>>>>>>> config MTD_NAND_ECC_SW_BCH > >>>>>>>> - tristate "Support software BCH ECC" > >>>>>>>> + bool "Support software BCH ECC" > >>>>>>>> select BCH > >>>>>>>> default n > >>>>>>>> help > >>>>>>> > >>>>>>> Should be fixed with the following diff squashed into: > >>>>>>> > >>>>>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") > >>>>>>> > >>>>>>> --->8--- > >>>>>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h > >>>>>>> index b8106651f807..06ce2b655c13 100644 > >>>>>>> --- a/include/linux/mtd/nand_bch.h > >>>>>>> +++ b/include/linux/mtd/nand_bch.h > >>>>>>> @@ -15,7 +15,7 @@ struct mtd_info; > >>>>>>> struct nand_chip; > >>>>>>> struct nand_bch_control; > >>>>>>> > >>>>>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) > >>>>>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) > >>>>>>> > >>>>>>> static inline int mtd_nand_has_bch(void) { return 1; } > >>>>>>> > >>>>>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); > >>>>>>> */ > >>>>>>> void nand_bch_free(struct nand_bch_control *nbc); > >>>>>>> > >>>>>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ > >>>>>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>>>>> > >>>>>>> static inline int mtd_nand_has_bch(void) { return 0; } > >>>>>>> > >>>>>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) > >>>>>>> > >>>>>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} > >>>>>>> > >>>>>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ > >>>>>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ > >>>>>>> > >>>>>>> #endif /* __MTD_NAND_BCH_H__ */ > >>>>>> > >>>>>> Sorry, I didn't look at the right branch, this part of the code was > >>>>>> correct, but we still have a problem to express the RAW_NAND(y) -> > >>>>>> SW_BCH(y) dependency. > >>>>> > >>>>> It seems this dependency is not always need, > >>>>> > >>>>> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. > >>>> > >>>> Yes, I know, but forcing nand_bch to a be a boolean is not the right > >>>> solution either, hence my suggestion to use 'imply'. > >> > >> Why exactly? SW BCH is going to be converted to the generic ECC engine > >> abstraction and IIRC you told me that it would not be problematic to > >> turn it into a boolean? > > > > Yes, I told you we should turn it into a boolean and embed the code in > > the nandcore.o object which can itself be linked as a module or embedded > > in the kernel image (see above for a version adapted for nand.o). That's > > not exactly what Yue proposed. > > So is there anothor fix or just to pick my v2 ? > > https://patchwork.ozlabs.org/patch/1083480/ Looks like someone posted it already: https://patchwork.ozlabs.org/patch/1083603/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module 2019-04-12 15:04 ` Boris Brezillon @ 2019-04-12 15:23 ` YueHaibing -1 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-12 15:23 UTC (permalink / raw) To: Boris Brezillon Cc: bbrezillon, richard, linux-kernel, marek.vasut, paul.burton, linux-mtd, Miquel Raynal, computersforpeace On 2019/4/12 23:04, Boris Brezillon wrote: > On Fri, 12 Apr 2019 22:28:57 +0800 > YueHaibing <yuehaibing@huawei.com> wrote: > >> On 2019/4/11 1:03, Boris Brezillon wrote: >>> On Wed, 10 Apr 2019 18:18:43 +0200 >>> Miquel Raynal <miquel.raynal@bootlin.com> wrote: >>> >>>> Hi YueHaibing, >>>> >>>> YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 >>>> +0800: >>>> >>>>> On 2019/4/10 22:29, Boris Brezillon wrote: >>>>>> On Wed, 10 Apr 2019 22:22:16 +0800 >>>>>> YueHaibing <yuehaibing@huawei.com> wrote: >>>>>> >>>>>>> On 2019/4/10 21:58, Boris Brezillon wrote: >>>>>>>> On Wed, 10 Apr 2019 15:39:28 +0200 >>>>>>>> Boris Brezillon <boris.brezillon@collabora.com> wrote: >>>>>>>> >>>>>>>>> On Wed, 10 Apr 2019 21:07:47 +0800 >>>>>>>>> Yue Haibing <yuehaibing@huawei.com> wrote: >>>>>>>>> >>>>>>>>>> From: YueHaibing <yuehaibing@huawei.com> >>>>>>>>>> >>>>>>>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>>>>>>>>> is set to module: >>>>>>>>>> >>>>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>>>>>>>>> (.text+0xef6): undefined reference to `nand_bch_free' >>>>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>>>>>>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>>>>>>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>>>>>>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>>>>>>>>> >>>>>>>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>>>>>>>>> because MTD_RAW_NAND need it while linked. >>>>>>>>>> >>>>>>>>>> Reported-by: Hulk Robot <hulkci@huawei.com> >>>>>>>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >>>>>>>>> >>>>>>>>> Nope, it's not this one that introduced the regression. >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>>>>>>>>> --- >>>>>>>>>> drivers/mtd/nand/raw/Kconfig | 2 +- >>>>>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>>>>> >>>>>>>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>>>>>>>>> index 615d738..0500c42 100644 >>>>>>>>>> --- a/drivers/mtd/nand/raw/Kconfig >>>>>>>>>> +++ b/drivers/mtd/nand/raw/Kconfig >>>>>>>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>>>>>>>>> if MTD_RAW_NAND >>>>>>>>>> >>>>>>>>>> config MTD_NAND_ECC_SW_BCH >>>>>>>>>> - tristate "Support software BCH ECC" >>>>>>>>>> + bool "Support software BCH ECC" >>>>>>>>>> select BCH >>>>>>>>>> default n >>>>>>>>>> help >>>>>>>>> >>>>>>>>> Should be fixed with the following diff squashed into: >>>>>>>>> >>>>>>>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >>>>>>>>> >>>>>>>>> --->8--- >>>>>>>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >>>>>>>>> index b8106651f807..06ce2b655c13 100644 >>>>>>>>> --- a/include/linux/mtd/nand_bch.h >>>>>>>>> +++ b/include/linux/mtd/nand_bch.h >>>>>>>>> @@ -15,7 +15,7 @@ struct mtd_info; >>>>>>>>> struct nand_chip; >>>>>>>>> struct nand_bch_control; >>>>>>>>> >>>>>>>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >>>>>>>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >>>>>>>>> >>>>>>>>> static inline int mtd_nand_has_bch(void) { return 1; } >>>>>>>>> >>>>>>>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >>>>>>>>> */ >>>>>>>>> void nand_bch_free(struct nand_bch_control *nbc); >>>>>>>>> >>>>>>>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >>>>>>>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>>>> >>>>>>>>> static inline int mtd_nand_has_bch(void) { return 0; } >>>>>>>>> >>>>>>>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >>>>>>>>> >>>>>>>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >>>>>>>>> >>>>>>>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >>>>>>>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>>>> >>>>>>>>> #endif /* __MTD_NAND_BCH_H__ */ >>>>>>>> >>>>>>>> Sorry, I didn't look at the right branch, this part of the code was >>>>>>>> correct, but we still have a problem to express the RAW_NAND(y) -> >>>>>>>> SW_BCH(y) dependency. >>>>>>> >>>>>>> It seems this dependency is not always need, >>>>>>> >>>>>>> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. >>>>>> >>>>>> Yes, I know, but forcing nand_bch to a be a boolean is not the right >>>>>> solution either, hence my suggestion to use 'imply'. >>>> >>>> Why exactly? SW BCH is going to be converted to the generic ECC engine >>>> abstraction and IIRC you told me that it would not be problematic to >>>> turn it into a boolean? >>> >>> Yes, I told you we should turn it into a boolean and embed the code in >>> the nandcore.o object which can itself be linked as a module or embedded >>> in the kernel image (see above for a version adapted for nand.o). That's >>> not exactly what Yue proposed. >> >> So is there anothor fix or just to pick my v2 ? >> >> https://patchwork.ozlabs.org/patch/1083480/ > > > Looks like someone posted it already: Thanks! > > https://patchwork.ozlabs.org/patch/1083603/ > > . > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module @ 2019-04-12 15:23 ` YueHaibing 0 siblings, 0 replies; 24+ messages in thread From: YueHaibing @ 2019-04-12 15:23 UTC (permalink / raw) To: Boris Brezillon Cc: Miquel Raynal, richard, computersforpeace, marek.vasut, paul.burton, bbrezillon, linux-kernel, linux-mtd On 2019/4/12 23:04, Boris Brezillon wrote: > On Fri, 12 Apr 2019 22:28:57 +0800 > YueHaibing <yuehaibing@huawei.com> wrote: > >> On 2019/4/11 1:03, Boris Brezillon wrote: >>> On Wed, 10 Apr 2019 18:18:43 +0200 >>> Miquel Raynal <miquel.raynal@bootlin.com> wrote: >>> >>>> Hi YueHaibing, >>>> >>>> YueHaibing <yuehaibing@huawei.com> wrote on Wed, 10 Apr 2019 23:03:24 >>>> +0800: >>>> >>>>> On 2019/4/10 22:29, Boris Brezillon wrote: >>>>>> On Wed, 10 Apr 2019 22:22:16 +0800 >>>>>> YueHaibing <yuehaibing@huawei.com> wrote: >>>>>> >>>>>>> On 2019/4/10 21:58, Boris Brezillon wrote: >>>>>>>> On Wed, 10 Apr 2019 15:39:28 +0200 >>>>>>>> Boris Brezillon <boris.brezillon@collabora.com> wrote: >>>>>>>> >>>>>>>>> On Wed, 10 Apr 2019 21:07:47 +0800 >>>>>>>>> Yue Haibing <yuehaibing@huawei.com> wrote: >>>>>>>>> >>>>>>>>>> From: YueHaibing <yuehaibing@huawei.com> >>>>>>>>>> >>>>>>>>>> Fix gcc build error while CONFIG_MTD_NAND_ECC_SW_BCH >>>>>>>>>> is set to module: >>>>>>>>>> >>>>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_cleanup': >>>>>>>>>> (.text+0xef6): undefined reference to `nand_bch_free' >>>>>>>>>> drivers/mtd/nand/raw/nand_base.o: In function `nand_scan_tail': >>>>>>>>>> nand_base.c:(.text+0xa101): undefined reference to `nand_bch_calculate_ecc' >>>>>>>>>> nand_base.c:(.text+0xa120): undefined reference to `nand_bch_correct_data' >>>>>>>>>> nand_base.c:(.text+0xa269): undefined reference to `nand_bch_init' >>>>>>>>>> >>>>>>>>>> CONFIG_MTD_NAND_ECC_SW_BCH should not be set to M, >>>>>>>>>> because MTD_RAW_NAND need it while linked. >>>>>>>>>> >>>>>>>>>> Reported-by: Hulk Robot <hulkci@huawei.com> >>>>>>>>>> Fixes: 193bd4002644 ("mtd: nand: add software BCH ECC support" >>>>>>>>> >>>>>>>>> Nope, it's not this one that introduced the regression. >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >>>>>>>>>> --- >>>>>>>>>> drivers/mtd/nand/raw/Kconfig | 2 +- >>>>>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>>>>> >>>>>>>>>> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig >>>>>>>>>> index 615d738..0500c42 100644 >>>>>>>>>> --- a/drivers/mtd/nand/raw/Kconfig >>>>>>>>>> +++ b/drivers/mtd/nand/raw/Kconfig >>>>>>>>>> @@ -22,7 +22,7 @@ menuconfig MTD_RAW_NAND >>>>>>>>>> if MTD_RAW_NAND >>>>>>>>>> >>>>>>>>>> config MTD_NAND_ECC_SW_BCH >>>>>>>>>> - tristate "Support software BCH ECC" >>>>>>>>>> + bool "Support software BCH ECC" >>>>>>>>>> select BCH >>>>>>>>>> default n >>>>>>>>>> help >>>>>>>>> >>>>>>>>> Should be fixed with the following diff squashed into: >>>>>>>>> >>>>>>>>> 51ef1d0b2095 ("mtd: nand: Clarify Kconfig entry for software BCH ECC algorithm") >>>>>>>>> >>>>>>>>> --->8--- >>>>>>>>> diff --git a/include/linux/mtd/nand_bch.h b/include/linux/mtd/nand_bch.h >>>>>>>>> index b8106651f807..06ce2b655c13 100644 >>>>>>>>> --- a/include/linux/mtd/nand_bch.h >>>>>>>>> +++ b/include/linux/mtd/nand_bch.h >>>>>>>>> @@ -15,7 +15,7 @@ struct mtd_info; >>>>>>>>> struct nand_chip; >>>>>>>>> struct nand_bch_control; >>>>>>>>> >>>>>>>>> -#if defined(CONFIG_MTD_NAND_ECC_BCH) >>>>>>>>> +#if defined(CONFIG_MTD_NAND_ECC_SW_BCH) >>>>>>>>> >>>>>>>>> static inline int mtd_nand_has_bch(void) { return 1; } >>>>>>>>> >>>>>>>>> @@ -39,7 +39,7 @@ struct nand_bch_control *nand_bch_init(struct mtd_info *mtd); >>>>>>>>> */ >>>>>>>>> void nand_bch_free(struct nand_bch_control *nbc); >>>>>>>>> >>>>>>>>> -#else /* !CONFIG_MTD_NAND_ECC_BCH */ >>>>>>>>> +#else /* !CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>>>> >>>>>>>>> static inline int mtd_nand_has_bch(void) { return 0; } >>>>>>>>> >>>>>>>>> @@ -64,6 +64,6 @@ static inline struct nand_bch_control *nand_bch_init(struct mtd_info *mtd) >>>>>>>>> >>>>>>>>> static inline void nand_bch_free(struct nand_bch_control *nbc) {} >>>>>>>>> >>>>>>>>> -#endif /* CONFIG_MTD_NAND_ECC_BCH */ >>>>>>>>> +#endif /* CONFIG_MTD_NAND_ECC_SW_BCH */ >>>>>>>>> >>>>>>>>> #endif /* __MTD_NAND_BCH_H__ */ >>>>>>>> >>>>>>>> Sorry, I didn't look at the right branch, this part of the code was >>>>>>>> correct, but we still have a problem to express the RAW_NAND(y) -> >>>>>>>> SW_BCH(y) dependency. >>>>>>> >>>>>>> It seems this dependency is not always need, >>>>>>> >>>>>>> case MTD_RAW_NAND set to y works well while CONFIG_MTD_NAND_ECC_SW_BCH is not set. >>>>>> >>>>>> Yes, I know, but forcing nand_bch to a be a boolean is not the right >>>>>> solution either, hence my suggestion to use 'imply'. >>>> >>>> Why exactly? SW BCH is going to be converted to the generic ECC engine >>>> abstraction and IIRC you told me that it would not be problematic to >>>> turn it into a boolean? >>> >>> Yes, I told you we should turn it into a boolean and embed the code in >>> the nandcore.o object which can itself be linked as a module or embedded >>> in the kernel image (see above for a version adapted for nand.o). That's >>> not exactly what Yue proposed. >> >> So is there anothor fix or just to pick my v2 ? >> >> https://patchwork.ozlabs.org/patch/1083480/ > > > Looks like someone posted it already: Thanks! > > https://patchwork.ozlabs.org/patch/1083603/ > > . > ^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2019-04-12 15:23 UTC | newest] Thread overview: 24+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-04-10 13:07 [PATCH] mtd: nand: Fix build error while CONFIG_MTD_NAND_ECC_SW_BCH is set to module Yue Haibing 2019-04-10 13:07 ` Yue Haibing 2019-04-10 13:39 ` Boris Brezillon 2019-04-10 13:39 ` Boris Brezillon 2019-04-10 13:58 ` Boris Brezillon 2019-04-10 13:58 ` Boris Brezillon 2019-04-10 14:10 ` Boris Brezillon 2019-04-10 14:10 ` Boris Brezillon 2019-04-10 14:22 ` YueHaibing 2019-04-10 14:22 ` YueHaibing 2019-04-10 14:29 ` Boris Brezillon 2019-04-10 14:29 ` Boris Brezillon 2019-04-10 15:03 ` YueHaibing 2019-04-10 15:03 ` YueHaibing 2019-04-10 16:18 ` Miquel Raynal 2019-04-10 16:18 ` Miquel Raynal 2019-04-10 17:03 ` Boris Brezillon 2019-04-10 17:03 ` Boris Brezillon 2019-04-12 14:28 ` YueHaibing 2019-04-12 14:28 ` YueHaibing 2019-04-12 15:04 ` Boris Brezillon 2019-04-12 15:04 ` Boris Brezillon 2019-04-12 15:23 ` YueHaibing 2019-04-12 15:23 ` YueHaibing
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.