[resend v2 00/19] Add Cadence NAND Driver support
Maniyam, Dinesh
dinesh.maniyam at intel.com
Wed Mar 5 17:04:49 CET 2025
From: Michael Nazzareno Trimarchi <michael at amarulasolutions.com>
Sent: Wednesday, 5 March 2025 5:27 pm
To: Maniyam, Dinesh <dinesh.maniyam at intel.com>
Cc: u-boot at lists.denx.de; Marek <marex at denx.de>; Simon <simon.k.r.goldschmidt at gmail.com>; Tom Rini <trini at konsulko.com>; Dario Binacchi <dario.binacchi at amarulasolutions.com>; Johan Jonker <jbx6244 at gmail.com>; Michal Simek <michal.simek at amd.com>; Arseniy Krasnov <avkrasnov at salutedevices.com>; Alexander Dahl <ada at thorsis.com>; William Zhang <william.zhang at broadcom.com>; Igor Prusov <ivprusov at salutedevices.com>; Meng, Tingting <tingting.meng at intel.com>; Chee, Tien Fong <tien.fong.chee at intel.com>; Hea, Kok Kiang <kok.kiang.hea at intel.com>; Ng, Boon Khai <boon.khai.ng at intel.com>; Yuslaimi, Alif Zakuan <alif.zakuan.yuslaimi at intel.com>; Zamri, Muhammad Hazim Izzat <muhammad.hazim.izzat.zamri at intel.com>; Lim, Jit Loon <jit.loon.lim at intel.com>; Tang, Sieu Mun <sieu.mun.tang at intel.com>
Subject: Re: [resend v2 00/19] Add Cadence NAND Driver support
Hi
I have pushed a rebase of your patches on nand-next on my tree
I have dropped one patch for SPL_FRAMEWORK, drop the same in config of the board. Move the config on the board as last patch, adjust one patch for a code style kisses
and push here
https://source.denx.de/u-boot/custodians/u-boot-nand-flash/-/commits/nand-next?ref_type=heads
Michael
Noted with thanks.
Dinesh
On Wed, Feb 26, 2025 at 5:36 PM Maniyam, Dinesh <dinesh.maniyam at intel.com<mailto:dinesh.maniyam at intel.com>> wrote:
> -----Original Message-----
> From: Maniyam, Dinesh
> Sent: Wednesday, 26 February 2025 11:37 am
> To: 'Michael Nazzareno Trimarchi' <michael at amarulasolutions.com<mailto:michael at amarulasolutions.com>>
> Cc: u-boot at lists.denx.de<mailto:u-boot at lists.denx.de>; Marek <marex at denx.de<mailto:marex at denx.de>>; Simon
> <simon.k.r.goldschmidt at gmail.com<mailto:simon.k.r.goldschmidt at gmail.com>>; Tom Rini <trini at konsulko.com<mailto:trini at konsulko.com>>; Dario
> Binacchi <dario.binacchi at amarulasolutions.com<mailto:dario.binacchi at amarulasolutions.com>>; Johan Jonker
> <jbx6244 at gmail.com<mailto:jbx6244 at gmail.com>>; Michal Simek <michal.simek at amd.com<mailto:michal.simek at amd.com>>; Arseniy
> Krasnov <avkrasnov at salutedevices.com<mailto:avkrasnov at salutedevices.com>>; Alexander Dahl <ada at thorsis.com<mailto:ada at thorsis.com>>;
> William Zhang <william.zhang at broadcom.com<mailto:william.zhang at broadcom.com>>; Igor Prusov
> <ivprusov at salutedevices.com<mailto:ivprusov at salutedevices.com>>; Meng, Tingting <tingting.meng at intel.com<mailto:tingting.meng at intel.com>>;
> Chee, Tien Fong <tien.fong.chee at intel.com<mailto:tien.fong.chee at intel.com>>; Hea, Kok Kiang
> <kok.kiang.hea at intel.com<mailto:kok.kiang.hea at intel.com>>; Ng, Boon Khai <Boon.Khai.Ng at intel.com<mailto:Boon.Khai.Ng at intel.com>>; Yuslaimi,
> Alif Zakuan <alif.zakuan.yuslaimi at intel.com<mailto:alif.zakuan.yuslaimi at intel.com>>; Zamri, Muhammad Hazim Izzat
> <muhammad.hazim.izzat.zamri at intel.com<mailto:muhammad.hazim.izzat.zamri at intel.com>>; Lim, Jit Loon
> <jit.loon.lim at intel.com<mailto:jit.loon.lim at intel.com>>; Tang, Sieu Mun <Sieu.Mun.Tang at intel.com<mailto:Sieu.Mun.Tang at intel.com>>
> Subject: RE: [resend v2 00/19] Add Cadence NAND Driver support
>
>
>
> > -----Original Message-----
> > From: Michael Nazzareno Trimarchi <michael at amarulasolutions.com<mailto:michael at amarulasolutions.com>>
> > Sent: Saturday, 25 January 2025 6:40 pm
> > To: Maniyam, Dinesh <dinesh.maniyam at intel.com<mailto:dinesh.maniyam at intel.com>>
> > Cc: u-boot at lists.denx.de<mailto:u-boot at lists.denx.de>; Marek <marex at denx.de<mailto:marex at denx.de>>; Simon
> > <simon.k.r.goldschmidt at gmail.com<mailto:simon.k.r.goldschmidt at gmail.com>>; Tom Rini <trini at konsulko.com<mailto:trini at konsulko.com>>;
> > Dario Binacchi <dario.binacchi at amarulasolutions.com<mailto:dario.binacchi at amarulasolutions.com>>; Johan Jonker
> > <jbx6244 at gmail.com<mailto:jbx6244 at gmail.com>>; Michal Simek <michal.simek at amd.com<mailto:michal.simek at amd.com>>; Arseniy
> > Krasnov <avkrasnov at salutedevices.com<mailto:avkrasnov at salutedevices.com>>; Alexander Dahl
> > <ada at thorsis.com<mailto:ada at thorsis.com>>; William Zhang <william.zhang at broadcom.com<mailto:william.zhang at broadcom.com>>; Igor
> > Prusov <ivprusov at salutedevices.com<mailto:ivprusov at salutedevices.com>>; Meng, Tingting
> > <tingting.meng at intel.com<mailto:tingting.meng at intel.com>>; Chee, Tien Fong <tien.fong.chee at intel.com<mailto:tien.fong.chee at intel.com>>;
> > Hea, Kok Kiang <kok.kiang.hea at intel.com<mailto:kok.kiang.hea at intel.com>>; Ng, Boon Khai
> > <boon.khai.ng at intel.com<mailto:boon.khai.ng at intel.com>>; Yuslaimi, Alif Zakuan
> > <alif.zakuan.yuslaimi at intel.com<mailto:alif.zakuan.yuslaimi at intel.com>>; Zamri, Muhammad Hazim Izzat
> > <muhammad.hazim.izzat.zamri at intel.com<mailto:muhammad.hazim.izzat.zamri at intel.com>>; Lim, Jit Loon
> > <jit.loon.lim at intel.com<mailto:jit.loon.lim at intel.com>>; Tang, Sieu Mun <sieu.mun.tang at intel.com<mailto:sieu.mun.tang at intel.com>>
> > Subject: Re: [resend v2 00/19] Add Cadence NAND Driver support
> >
> > Hi Dinesh
> >
> > On Tue, Jan 21, 2025 at 9:39 AM Maniyam, Dinesh
> > <dinesh.maniyam at intel.com<mailto:dinesh.maniyam at intel.com>>
> > wrote:
> > >
> > > Hi Michael,
> > >
> > > Any update on my NAND patchset ?. I can see it is still under review.
> > > Is there anything I can do from my side to speed up the review process?
> > > Thanks for your time.
> > >
> >
> > Well, last time was not paassing the CI. I will test it again today
> >
> > Michael
> >
>
> Hope you already run the CI test again.
> If failing, may I get the link to the failing list of tests?
>
> Dinesh
>
Apologies!
I think you mentioned that you were unable to build the code.
I resubmitted v3 of this patchset, which will resolve warning/
compilation issue. You can look at the cover patch to see
the changes made for v3 in detail.
Please help to use this v3 patchset in running CI test. Please let me know
if you face any difficulties.
Thanks for your time and consideration.
Dinesh
> > > > -----Original Message-----
> > > > From: Maniyam, Dinesh
> > > > Sent: Tuesday, 17 December 2024 3:41 pm
> > > > To: Michael Nazzareno Trimarchi <michael at amarulasolutions.com<mailto:michael at amarulasolutions.com>>
> > > > Cc: u-boot at lists.denx.de<mailto:u-boot at lists.denx.de>; Marek <marex at denx.de<mailto:marex at denx.de>>; Simon
> > > > <simon.k.r.goldschmidt at gmail.com<mailto:simon.k.r.goldschmidt at gmail.com>>; Tom Rini <trini at konsulko.com<mailto:trini at konsulko.com>>;
> > > > Dario Binacchi <dario.binacchi at amarulasolutions.com<mailto:dario.binacchi at amarulasolutions.com>>; Johan Jonker
> > > > <jbx6244 at gmail.com<mailto:jbx6244 at gmail.com>>; Michal Simek <michal.simek at amd.com<mailto:michal.simek at amd.com>>; Arseniy
> > > > Krasnov <avkrasnov at salutedevices.com<mailto:avkrasnov at salutedevices.com>>; Alexander Dahl
> > > > <ada at thorsis.com<mailto:ada at thorsis.com>>; William Zhang <william.zhang at broadcom.com<mailto:william.zhang at broadcom.com>>;
> > > > Igor Prusov <ivprusov at salutedevices.com<mailto:ivprusov at salutedevices.com>>; Meng, Tingting
> > > > <tingting.meng at intel.com<mailto:tingting.meng at intel.com>>; Chee, Tien Fong
> > > > <tien.fong.chee at intel.com<mailto:tien.fong.chee at intel.com>>; Hea, Kok Kiang
> > > > <kok.kiang.hea at intel.com<mailto:kok.kiang.hea at intel.com>>; Ng, Boon Khai <Boon.Khai.Ng at intel.com<mailto:Boon.Khai.Ng at intel.com>>;
> > > > Yuslaimi, Alif Zakuan <alif.zakuan.yuslaimi at intel.com<mailto:alif.zakuan.yuslaimi at intel.com>>; Zamri,
> > > > Muhammad Hazim Izzat <muhammad.hazim.izzat.zamri at intel.com<mailto:muhammad.hazim.izzat.zamri at intel.com>>; Lim,
> > > > Jit Loon <jit.loon.lim at intel.com<mailto:jit.loon.lim at intel.com>>; Tang, Sieu Mun
> > > > <Sieu.Mun.Tang at intel.com<mailto:Sieu.Mun.Tang at intel.com>>
> > > > Subject: RE: [resend v2 00/19] Add Cadence NAND Driver support
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Michael Nazzareno Trimarchi <michael at amarulasolutions.com<mailto:michael at amarulasolutions.com>>
> > > > > Sent: Friday, 13 December 2024 5:16 pm
> > > > > To: Maniyam, Dinesh <dinesh.maniyam at intel.com<mailto:dinesh.maniyam at intel.com>>
> > > > > Cc: u-boot at lists.denx.de<mailto:u-boot at lists.denx.de>; Marek <marex at denx.de<mailto:marex at denx.de>>; Simon
> > > > > <simon.k.r.goldschmidt at gmail.com<mailto:simon.k.r.goldschmidt at gmail.com>>; Tom Rini
> > > > > <trini at konsulko.com<mailto:trini at konsulko.com>>; Dario Binacchi
> > > > > <dario.binacchi at amarulasolutions.com<mailto:dario.binacchi at amarulasolutions.com>>; Johan Jonker
> > > > > <jbx6244 at gmail.com<mailto:jbx6244 at gmail.com>>; Michal Simek <michal.simek at amd.com<mailto:michal.simek at amd.com>>;
> > > > > Arseniy Krasnov <avkrasnov at salutedevices.com<mailto:avkrasnov at salutedevices.com>>; Alexander Dahl
> > > > > <ada at thorsis.com<mailto:ada at thorsis.com>>; William Zhang <william.zhang at broadcom.com<mailto:william.zhang at broadcom.com>>;
> > > > > Igor Prusov <ivprusov at salutedevices.com<mailto:ivprusov at salutedevices.com>>; Meng, Tingting
> > > > > <tingting.meng at intel.com<mailto:tingting.meng at intel.com>>; Chee, Tien Fong
> > > > > <tien.fong.chee at intel.com<mailto:tien.fong.chee at intel.com>>; Hea, Kok Kiang
> > > > > <kok.kiang.hea at intel.com<mailto:kok.kiang.hea at intel.com>>; Ng, Boon Khai
> > > > > <boon.khai.ng at intel.com<mailto:boon.khai.ng at intel.com>>; Yuslaimi, Alif Zakuan
> > > > > <alif.zakuan.yuslaimi at intel.com<mailto:alif.zakuan.yuslaimi at intel.com>>; Zamri, Muhammad Hazim Izzat
> > > > > <muhammad.hazim.izzat.zamri at intel.com<mailto:muhammad.hazim.izzat.zamri at intel.com>>; Lim, Jit Loon
> > > > > <jit.loon.lim at intel.com<mailto:jit.loon.lim at intel.com>>; Tang, Sieu Mun
> > > > > <sieu.mun.tang at intel.com<mailto:sieu.mun.tang at intel.com>>
> > > > > Subject: Re: [resend v2 00/19] Add Cadence NAND Driver support
> > > > >
> > > > > Hi
> > > > >
> > > > > On Thu, Dec 5, 2024 at 10:23 AM <dinesh.maniyam at intel.com<mailto:dinesh.maniyam at intel.com>> wrote:
> > > > > >
> > > > > > From: Dinesh Maniyam <dinesh.maniyam at intel.com<mailto:dinesh.maniyam at intel.com>>
> > > > > >
> > > > > > This patchset add Cadence NAND driver support for Intel
> > > > > > Agilex5 devices.
> > > > > >
> > > > > > The NAND driver is leveraged from the
> > > > > > cadence-nand-controller.c from Linux version 6.11.2. U-Boot
> > > > > > will support read, write and erase NAND with Cadence driver.
> > > > > > The driver further enhanced in U-Boot to support NAND booting
> > > > > > from FSBL and support boot to kernel via
> > UBIFS.
> > > > > >
> > > > > > Patch status:
> > > > > > Have changes: All patches expect Patch 19
> > > > > >
> > > > > > Detail changelog can find in commit message.
> > > > > >
> > > > > > v1->v2:
> > > > > > --------
> > > > > > Patch 1 - 18:
> > > > > >
> > > > > > - remove the "this patch is to" commit phrases
> > > > > >
> > > > > > Patch 4;
> > > > > > - minor refactoring of code to match the stable version 6.11.2.
> > > > > >
> > > > > > Patch 14:
> > > > > > - Add maintainer and "#include" mechanism for new variant
> > > > > >
> > > > > > Patch 16:
> > > > > > - leverage the existing nand_spl_load_image
> > > > > >
> > > > >
> > > > > I have tried to build but it does not work. Have you tested in github?
> > > > >
> > > > > Michael
> > > >
> > > > Yet to deploy github test on our current platform.
> > > > Only checkpatch.pl<http://checkpatch.pl> and local built has been used as a metric for
> > > > each commit before submission.
> > > > And, all these commits are part of the firmware currently running
> > > > on real hardware.
> > > >
> > > > >
> > > > > > History:
> > > > > > --------
> > > > > > [v1]:
> > > > > > https://patchwork.ozlabs.org/project/uboot/cover/20240919035512.
> > > > > > 1385
> > > > > > 4-
> > > > > > 1-dinesh.maniyam at intel.com/<http://1-dinesh.maniyam@intel.com/>
> > > > > >
> > > > > > Dinesh Maniyam (19):
> > > > > > dt: nand: add cadence nand dt-bindings
> > > > > > arm: dts: agilex5: Enabled cdns-nand dts setting
> > > > > > include: asm: Add support to read/write 64-bit
> > > > > > drivers: mtd: nand: Add driver for Cadence Nand
> > > > > > drivers: mtd: nand: cadence: Add support for read status command
> > > > > > drivers: mtd: nand: cadence: Add support for readid command
> > > > > > drivers: mtd: nand: cadence: Add support for NAND_CMD_PARAM
> > > > > > drivers: mtd: nand: cadence: Add support for NAND_CMD_RESET
> > > > > > drivers: mtd: nand: cadence: Support cmd SET_FEATURES &
> > GET_FEATURES
> > > > > > drivers: mtd: nand: cadence: Flush & invalidate dma descriptor
> > > > > > drivers: mtd: nand: cadence: Poll for desc complete status
> > > > > > drivers: mtd: nand: cadence: Use bounce buffer
> > > > > > drivers: nand: Enabled Kconfig and Makefile for cdns-nand
> > > > > > configs: nand2_defconfig: Enable configs for nand boot
> > > > > > drivers: mtd: nand: base: Add support for Hardware ECC for check bad
> > > > > > block
> > > > > > drivers: mtd: nand: spl: Add support for nand SPL load image
> > > > > > drivers: mtd: nand: Enabled Kconfig and Makefile for Cadence-SPL
> > > > > > drivers: mtd: nand: Kconfig: Remove SYS_NAND_BLOCK_SIZE
> > dependency
> > > > > > drivers: mtd: nand: Kconfig: Enabled self-init for cdns-nand
> > > > > > SPL
> > > > > >
> > > > > > arch/arm/dts/socfpga_agilex5.dtsi | 14 +
> > > > > > .../arm/dts/socfpga_agilex5_socdk-u-boot.dtsi | 28 +
> > > > > > arch/arm/include/asm/io.h | 25 +
> > > > > > board/intel/agilex5-socdk/MAINTAINERS | 2 +
> > > > > > configs/socfpga_agilex5_nand2_defconfig | 33 +
> > > > > > .../mtd/cadence,nand.yaml | 98 +
> > > > > > drivers/mtd/nand/raw/Kconfig | 19 +-
> > > > > > drivers/mtd/nand/raw/Makefile | 2 +
> > > > > > drivers/mtd/nand/raw/cadence_nand.c | 2424
> > +++++++++++++++++
> > > > > > drivers/mtd/nand/raw/cadence_spl.c | 59 +
> > > > > > drivers/mtd/nand/raw/nand_base.c | 71 +-
> > > > > > include/cadence-nand.h | 529 ++++
> > > > > > include/linux/mtd/rawnand.h | 14 +
> > > > > > 13 files changed, 3293 insertions(+), 25 deletions(-) create
> > > > > > mode
> > > > > > 100644 configs/socfpga_agilex5_nand2_defconfig
> > > > > > create mode 100644
> > > > > > doc/device-tree-bindings/mtd/cadence,nand.yaml
> > > > > > create mode 100644 drivers/mtd/nand/raw/cadence_nand.c
> > > > > > create mode 100644 drivers/mtd/nand/raw/cadence_spl.c
> > > > > > create mode 100644 include/cadence-nand.h
> > > > > >
> > > > > > --
> > > > > > 2.19.0
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Michael Nazzareno Trimarchi
> > > > > Co-Founder & Chief Executive Officer M. +39 347 913 2170
> > > > > michael at amarulasolutions.com<mailto:michael at amarulasolutions.com>
> __________________________________
> > > > >
> > > > > Amarula Solutions BV
> > > > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111
> > > > > 9172 info at amarulasolutions.com<mailto:info at amarulasolutions.com> www.amarulasolutions.com<http://www.amarulasolutions.com>
> >
> >
> >
> > --
> > Michael Nazzareno Trimarchi
> > Co-Founder & Chief Executive Officer
> > M. +39 347 913 2170
> > michael at amarulasolutions.com<mailto:michael at amarulasolutions.com>
> > __________________________________
> >
> > Amarula Solutions BV
> > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172
> > info at amarulasolutions.com<mailto:info at amarulasolutions.com> www.amarulasolutions.com<http://www.amarulasolutions.com>
--
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com<mailto:michael at amarulasolutions.com>
__________________________________
Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com<mailto:info at amarulasolutions.com>
www.amarulasolutions.com<http://www.amarulasolutions.com>
More information about the U-Boot
mailing list