[PATCH v2 0/2]: Enable UBIFS support

[PATCH v2 0/2]: Enable UBIFS support

[kah.jing.lee]

From: Kah Jing Lee <kah.jing.lee@intel.com>

Hi,
The patchset is to update the qspi mtd partition range for UBIFS on
socfpga platform - Agilex and Stratix10 boards.
The subsequent patch is enable the UBIFS in defconfig.

Alif Zakuan Yuslaimi (2):
  arm64: dts: agilex/stratix10: Updated QSPI Flash layout for UBIFS
  arm64: defconfig: Enable UBI and UBIFS

---
v2: Update subject prefix and defconfig config place
---

 arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 8 ++++----
 arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts     | 8 ++++----
 arch/arm64/configs/defconfig                           | 2 ++
 3 files changed, 10 insertions(+), 8 deletions(-)


base-commit: dad9774deaf1cf8e8f7483310dfb2690310193d2
-- 
2.25.1

[PATCH v2 1/2] arm64: dts: agilex/stratix10: Updated QSPI Flash layout for UBIFS

[kah.jing.lee]

From: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@intel.com>

This patch is to update the QSPI flash layout to support UBIFS in mtd
root partition.

Signed-off-by: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@intel.com>
Signed-off-by: Kah Jing Lee <kah.jing.lee@intel.com>
---
v2: Update subject prefix
---
 arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 8 ++++----
 arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts     | 8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts b/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts
index 38ae674f2f02..12b960b9183c 100644
--- a/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts
+++ b/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts
@@ -202,12 +202,12 @@ partitions {
 
 			qspi_boot: partition@0 {
 				label = "Boot and fpga data";
-				reg = <0x0 0x03FE0000>;
+				reg = <0x0 0x04200000>;
 			};
 
-			qspi_rootfs: partition@3FE0000 {
-				label = "Root Filesystem - JFFS2";
-				reg = <0x03FE0000 0x0C020000>;
+			root: partition@4200000 {
+				label = "Root Filesystem - UBIFS";
+				reg = <0x04200000 0x0BE00000>;
 			};
 		};
 	};
diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts b/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts
index 07c3f8876613..608017e6e36a 100644
--- a/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts
+++ b/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts
@@ -128,12 +128,12 @@ partitions {
 
 			qspi_boot: partition@0 {
 				label = "Boot and fpga data";
-				reg = <0x0 0x03FE0000>;
+				reg = <0x0 0x04200000>;
 			};
 
-			qspi_rootfs: partition@3FE0000 {
-				label = "Root Filesystem - JFFS2";
-				reg = <0x03FE0000 0x0C020000>;
+			root: partition@4200000 {
+				label = "Root Filesystem - UBIFS";
+				reg = <0x04200000 0x0BE00000>;
 			};
 		};
 	};
-- 
2.25.1

RE: [PATCH v2 1/2] arm64: dts: agilex/stratix10: Updated QSPI Flash layout for UBIFS

[Lee, Kah Jing]

> -----Original Message-----
> From: Markus Elfring <Markus.Elfring@web.de>
> Sent: Thursday, 22 June, 2023 9:05 PM
> To: Yuslaimi, Alif Zakuan <alif.zakuan.yuslaimi@intel.com>; Lee, Kah Jing
> <kah.jing.lee@intel.com>; devicetree@vger.kernel.org; kernel-
> janitors@vger.kernel.org; Catalin Marinas <catalin.marinas@arm.com>;
> Conor Dooley <conor+dt@kernel.org>; Dinh Nguyen
> <dinguyen@kernel.org>; Rob Herring <robh+dt@kernel.org>; Krzysztof
> Kozlowski <krzysztof.kozlowski+dt@linaro.org>; Will Deacon
> <will@kernel.org>
> Cc: LKML <linux-kernel@vger.kernel.org>; cocci@inria.fr
> Subject: Re: [PATCH v2 1/2] arm64: dts: agilex/stratix10: Updated QSPI Flash
> layout for UBIFS
> 
> > This patch is to update the QSPI flash layout to support UBIFS in mtd
> > root partition.
> 
> Please replace this sentence by an imperative change suggestion.
> 
> See also:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Docu
> mentation/process/submitting-patches.rst?h=v6.4-rc7#n94
> 
> Regards,
> Markus

Updated in v3 - Commit message: 
Update the QSPI flash layout to support UBIFS in mtd root partition.
Flash layout is updated in order to store non-UBIFS related boot and fpga data in qspi_boot(mtd0) 
while keep the rootfs with UBIFS in the root partition(mtd1).

Thanks. 

Regards, 
Kah Jing

RE: [v2 1/2] arm64: dts: agilex/stratix10: Updated QSPI Flash layout for UBIFS

[Lee, Kah Jing]

> -----Original Message-----
> From: Markus Elfring <Markus.Elfring@web.de>
> Sent: Friday, 23 June, 2023 7:04 PM
> To: Lee, Kah Jing <kah.jing.lee@intel.com>; Yuslaimi, Alif Zakuan
> <alif.zakuan.yuslaimi@intel.com>; devicetree@vger.kernel.org; kernel-
> janitors@vger.kernel.org; Catalin Marinas <catalin.marinas@arm.com>;
> Conor Dooley <conor+dt@kernel.org>; Dinh Nguyen <dinguyen@kernel.org>;
> Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@linaro.org>; Will Deacon <will@kernel.org>
> Cc: LKML <linux-kernel@vger.kernel.org>; cocci@inria.fr
> Subject: Re: [v2 1/2] arm64: dts: agilex/stratix10: Updated QSPI Flash layout
> for UBIFS
> 
> > Updated in v3 - Commit message:
> > Update the QSPI flash layout to support UBIFS in mtd root partition.
> > Flash layout is updated in order to store non-UBIFS related boot and
> > fpga data in qspi_boot(mtd0) while keep the rootfs with UBIFS in the root
> partition(mtd1).
> 
> I got the impression that this change description is also improvable a bit.
> How do you think about a wording variant like the following?
> 
> 
> Non-UBIFS related boot and fpga data should be stored in qspi_boot (mtd0)
> while keeping the rootfs with UBIFS in the root partition “mtd1”.
> Thus update the QSPI flash layout to support UBIFS in the mtd root partition.
> 
> 
> Will similar adjustments become helpful for subsequent update steps?
Sounds good to me. Will update the commit message for v3.
Thanks.
> 
> Regards,
> Markus

Regards, 
Lee, Kah Jing

[PATCH v2 2/2] arm64: defconfig: Enable UBI and UBIFS

[kah.jing.lee]

From: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@intel.com>

This patch is to enable UBI and UBIFS in Linux defconfig for socfpga
Agilex and Stratix10 platform.

Signed-off-by: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@intel.com>
Signed-off-by: Kah Jing Lee <kah.jing.lee@intel.com>
---
v2: Update subject prefix and align defconfig config place
---
 arch/arm64/configs/defconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index a24609e14d50..8c3c6fd20d77 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -267,6 +267,7 @@ CONFIG_MTD_NAND_BRCMNAND=m
 CONFIG_MTD_NAND_FSL_IFC=y
 CONFIG_MTD_NAND_QCOM=y
 CONFIG_MTD_SPI_NOR=y
+CONFIG_MTD_UBI=y
 CONFIG_BLK_DEV_LOOP=y
 CONFIG_BLK_DEV_NBD=m
 CONFIG_VIRTIO_BLK=y
@@ -1444,6 +1445,7 @@ CONFIG_TMPFS_POSIX_ACL=y
 CONFIG_HUGETLBFS=y
 CONFIG_CONFIGFS_FS=y
 CONFIG_EFIVAR_FS=y
+CONFIG_UBIFS_FS=y
 CONFIG_SQUASHFS=y
 CONFIG_NFS_FS=y
 CONFIG_NFS_V4=y
-- 
2.25.1

Re: [PATCH v2 2/2] arm64: defconfig: Enable UBI and UBIFS

[Krzysztof Kozlowski]

On 22/06/2023 14:36, kah.jing.lee@intel.com wrote:
> From: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@intel.com>
> 
> This patch is to enable UBI and UBIFS in Linux defconfig for socfpga
> Agilex and Stratix10 platform.
> 
> Signed-off-by: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@intel.com>
> Signed-off-by: Kah Jing Lee <kah.jing.lee@intel.com>
> ---
> v2: Update subject prefix and align defconfig config place
> ---
>  arch/arm64/configs/defconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index a24609e14d50..8c3c6fd20d77 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -267,6 +267,7 @@ CONFIG_MTD_NAND_BRCMNAND=m
>  CONFIG_MTD_NAND_FSL_IFC=y
>  CONFIG_MTD_NAND_QCOM=y
>  CONFIG_MTD_SPI_NOR=y
> +CONFIG_MTD_UBI=y

=m

(and wait till discussion finished)

>  CONFIG_BLK_DEV_LOOP=y
>  CONFIG_BLK_DEV_NBD=m
>  CONFIG_VIRTIO_BLK=y
> @@ -1444,6 +1445,7 @@ CONFIG_TMPFS_POSIX_ACL=y
>  CONFIG_HUGETLBFS=y
>  CONFIG_CONFIGFS_FS=y
>  CONFIG_EFIVAR_FS=y
> +CONFIG_UBIFS_FS=y

Also =m



Best regards,
Krzysztof