> -----Original Message----- > From: Dinh Nguyen <dingu...@kernel.org> > Sent: Monday, October 21, 2019 11:57 PM > To: Ooi, Joyce <joyce....@intel.com>; Vladimir Murzin > <vladimir.mur...@arm.com>; Catalin Marinas <catalin.mari...@arm.com>; Will > Deacon <w...@kernel.org> > Cc: Shawn Guo <shawn...@kernel.org>; Olof Johansson <o...@lixom.net>; > Maxime Ripard <mrip...@kernel.org>; Bjorn Andersson > <bjorn.anders...@linaro.org>; Arnd Bergmann <a...@arndb.de>; Jagan Teki > <ja...@amarulasolutions.com>; Anson Huang <anson.hu...@nxp.com>; > Leonard Crestez <leonard.cres...@nxp.com>; Marcin Juszkiewicz > <marcin.juszkiew...@linaro.org>; linux-arm-ker...@lists.infradead.org; linux- > ker...@vger.kernel.org; Ong, Hean Loong <hean.loong....@intel.com>; See, > Chin Liang <chin.liang....@intel.com>; Tan, Ley Foon > <ley.foon....@intel.com> > Subject: Re: [PATCHv2] arm64: defconfig: add JFFS FS support in defconfig > > > > On 10/17/19 1:28 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....@intel.com> > > --- > > v2: disable CONFIG_MTD_SPI_NOR_USE_4K_SECTORS using the correct syntax > > --- > > 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 is not set > > > > Can you explain why you're removing > CONFIG_MTD_SPI_NOR_USE_4K_SECTORS? Removing CONFIG_MTD_SPI_NOR_USE_4K_SECTORS will fix errors like this: [ 1.905772] jffs2: jffs2_scan_eraseblock(): Magic bitmask 0x1985 not found at 0x00000f90: 0x00bf instead [ 1.915275] jffs2: jffs2_scan_eraseblock(): Magic bitmask 0x1985 not found at 0x00000f94: 0x6abb instead
I'll add this explanation in the commit message. > > Dinh
