* [PATCH] arm64: defconfig: add JFFS FS support in defconfig
@ 2019-10-16 9:35 Ooi, Joyce
2019-10-16 11:46 ` Vladimir Murzin
0 siblings, 1 reply; 3+ messages in thread
From: Ooi, Joyce @ 2019-10-16 9:35 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Dinh Nguyen
Cc: Tan Ley Foon, Anson Huang, Arnd Bergmann, Ong Hean Loong,
Joyce Ooi, See Chin Liang, linux-kernel, Maxime Ripard,
Bjorn Andersson, Marcin Juszkiewicz, Jagan Teki, Olof Johansson,
Leonard Crestez, Shawn Guo, linux-arm-kernel
This patch adds JFFS2 FS support and remove QSPI Sector 4K size force in
the default defconfig
Signed-off-by: Ooi, Joyce <joyce.ooi@intel.com>
---
arch/arm64/configs/defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index c9adae4..6080c6e 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -860,3 +860,5 @@ CONFIG_DEBUG_KERNEL=y
# CONFIG_DEBUG_PREEMPT is not set
# CONFIG_FTRACE is not set
CONFIG_MEMTEST=y
+CONFIG_JFFS2_FS=y
+CONFIG_MTD_SPI_NOR_USE_4K_SECTORS=n
--
1.9.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: defconfig: add JFFS FS support in defconfig
2019-10-16 9:35 [PATCH] arm64: defconfig: add JFFS FS support in defconfig Ooi, Joyce
@ 2019-10-16 11:46 ` Vladimir Murzin
2019-10-17 6:09 ` Ooi, Joyce
0 siblings, 1 reply; 3+ messages in thread
From: Vladimir Murzin @ 2019-10-16 11:46 UTC (permalink / raw)
To: Ooi, Joyce, Catalin Marinas, Will Deacon, Dinh Nguyen
Cc: Anson Huang, Arnd Bergmann, Ong Hean Loong, See Chin Liang,
linux-kernel, Tan Ley Foon, Bjorn Andersson, Marcin Juszkiewicz,
Maxime Ripard, Olof Johansson, Leonard Crestez, Shawn Guo,
linux-arm-kernel, Jagan Teki
On 10/16/19 10:35 AM, Ooi, Joyce wrote:
> This patch adds JFFS2 FS support and remove QSPI Sector 4K size force in
> the default defconfig
>
> Signed-off-by: Ooi, Joyce <joyce.ooi@intel.com>
> ---
> arch/arm64/configs/defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index c9adae4..6080c6e 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -860,3 +860,5 @@ CONFIG_DEBUG_KERNEL=y
> # CONFIG_DEBUG_PREEMPT is not set
> # CONFIG_FTRACE is not set
> CONFIG_MEMTEST=y
> +CONFIG_JFFS2_FS=y
> +CONFIG_MTD_SPI_NOR_USE_4K_SECTORS=n
^^^^
This is incorrect syntax for disabling config option. Correct one is
# CONFIG_MTD_SPI_NOR_USE_4K_SECTORS is not set
However, it looks to me you want to remove it from defconfig rather than
force it to be unset.
Cheers
Vladimir
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread* RE: [PATCH] arm64: defconfig: add JFFS FS support in defconfig
2019-10-16 11:46 ` Vladimir Murzin
@ 2019-10-17 6:09 ` Ooi, Joyce
0 siblings, 0 replies; 3+ messages in thread
From: Ooi, Joyce @ 2019-10-17 6:09 UTC (permalink / raw)
To: Vladimir Murzin, Catalin Marinas, Will Deacon, Dinh Nguyen
Cc: Anson Huang, Arnd Bergmann, Ong, Hean Loong, See, Chin Liang,
linux-kernel@vger.kernel.org, Tan, Ley Foon, Bjorn Andersson,
Marcin Juszkiewicz, Maxime Ripard, Olof Johansson,
Leonard Crestez, Shawn Guo, linux-arm-kernel@lists.infradead.org,
Jagan Teki
> -----Original Message-----
> From: Vladimir Murzin [mailto:vladimir.murzin@arm.com]
> Sent: Wednesday, October 16, 2019 7:46 PM
> To: Ooi, Joyce <joyce.ooi@intel.com>; Catalin Marinas
> <catalin.marinas@arm.com>; Will Deacon <will@kernel.org>; Dinh Nguyen
> <dinguyen@kernel.org>
> Cc: Tan, Ley Foon <ley.foon.tan@intel.com>; Anson Huang
> <Anson.Huang@nxp.com>; Arnd Bergmann <arnd@arndb.de>; Ong, Hean Loong
> <hean.loong.ong@intel.com>; See, Chin Liang <chin.liang.see@intel.com>;
> linux-kernel@vger.kernel.org; Maxime Ripard <mripard@kernel.org>; Bjorn
> Andersson <bjorn.andersson@linaro.org>; Marcin Juszkiewicz
> <marcin.juszkiewicz@linaro.org>; Jagan Teki <jagan@amarulasolutions.com>;
> Olof Johansson <olof@lixom.net>; Leonard Crestez
> <leonard.crestez@nxp.com>; Shawn Guo <shawnguo@kernel.org>; linux-arm-
> kernel@lists.infradead.org
> Subject: Re: [PATCH] arm64: defconfig: add JFFS FS support in defconfig
>
> On 10/16/19 10:35 AM, Ooi, Joyce wrote:
> > This patch adds JFFS2 FS support and remove QSPI Sector 4K size force
> > in the default defconfig
> >
> > Signed-off-by: Ooi, Joyce <joyce.ooi@intel.com>
> > ---
> > arch/arm64/configs/defconfig | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/arm64/configs/defconfig
> > b/arch/arm64/configs/defconfig index c9adae4..6080c6e 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -860,3 +860,5 @@ CONFIG_DEBUG_KERNEL=y #
> CONFIG_DEBUG_PREEMPT is
> > not set # CONFIG_FTRACE is not set CONFIG_MEMTEST=y
> > +CONFIG_JFFS2_FS=y
> > +CONFIG_MTD_SPI_NOR_USE_4K_SECTORS=n
> ^^^^
> This is incorrect syntax for disabling config option. Correct one is
>
> # CONFIG_MTD_SPI_NOR_USE_4K_SECTORS is not set
Okay, will do that.
>
> However, it looks to me you want to remove it from defconfig rather than force
> it to be unset.
>
> Cheers
> Vladimir
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-10-17 6:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-16 9:35 [PATCH] arm64: defconfig: add JFFS FS support in defconfig Ooi, Joyce
2019-10-16 11:46 ` Vladimir Murzin
2019-10-17 6:09 ` Ooi, Joyce
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).