Hi Baruch,

On 29.10.20 07:52, Baruch Siach wrote:
This series adds NAND flash support to Aramda 8k systems. Patches make the
necessary changes to the pxa3xx_nand driver and DT files.

v3:
   Address Stefan's comments on patch #3

   Move hunks from patch #3 to #5 to fix bisectability

Thanks. I'll look into it soonish.

One general comment though. You missed the v3 in the patch subject this
time and you also missed adding the patch revision history to the
patches:

http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions

No need to re-send now. But please make sure to send with a correct
history next time.

Thanks,
Stefan


v2:
   Rebase on current master. Fixes conflict with commit 661c98121d4 ("mtd: nand:
   pxa3xx: Fix not calling dev_xxx with a device")

Baruch Siach (2):
   arm: dts: armada-cp110-master: update nand-controller
   mtd: pxa3xx_nand: remove dead code

Shmuel Hazan (3):
   arm: dts: armada-cp110-slave: add missing cps_nand
   mtd: pxa3xx_nand: port to use driver model
   mtd: nand: pxa3xx: enable NAND controller if the SoC needs it

  arch/arm/dts/armada-cp110-master.dtsi |  15 ++-
  arch/arm/dts/armada-cp110-slave.dtsi  |  16 +++
  drivers/mtd/nand/raw/Kconfig          |   2 +
  drivers/mtd/nand/raw/pxa3xx_nand.c    | 180 ++++++++++++++------------
  4 files changed, 125 insertions(+), 88 deletions(-)



Viele Grüße,
Stefan

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de

Reply via email to