From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Vignesh Raghavendra <vigneshr@ti.com>,
Arnd Bergmann <arnd@arndb.de>,
Tudor Ambarus <tudor.ambarus@microchip.com>,
Richard Weinberger <richard@nod.at>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Boris Brezillon <boris.brezillon@collabora.com>,
linux-mtd@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
Mike Rapoport <mike@compulab.co.il>,
Daniel Mack <daniel@zonque.org>
Subject: Re: [PATCH 0/3] mtd: rawnand: Get rid of the cmx270 driver
Date: Fri, 8 May 2020 12:10:41 +0200 [thread overview]
Message-ID: <20200508121041.64f91276@xps13> (raw)
In-Reply-To: <20200429223134.789322-1-boris.brezillon@collabora.com>
Hi Robert,
Boris Brezillon <boris.brezillon@collabora.com> wrote on Thu, 30 Apr
2020 00:31:31 +0200:
> Hello,
>
> As part of my attempt to convert all NAND drivers to exec_op() I noticed
> the cmx270 board didn't really deserve a custom driver since other boards
> using the same SoC (em-x270 to name one) are using the gen_nand driver.
> I think the only issue with the CM-X270 is that the chip is connected
> to D[16:23] (or D[16:31] if it's a 16bit bus). Adjusting the mem
> resource offset should do the trick.
>
> I hope someone still has a board to test that.
>
> Regards,
>
> Boris
>
> Boris Brezillon (3):
> ARM: pxa: cm-x270: Use gen_nand to expose the NAND device
> ARM: pxa: Stop selecting CONFIG_MTD_NAND_CM_X270
> mtd: rawnand: Remove the cmx270 NAND controller driver
>
> arch/arm/configs/cm_x2xx_defconfig | 1 -
> arch/arm/configs/pxa_defconfig | 1 -
> arch/arm/mach-pxa/cm-x270.c | 131 ++++++++++++++++
> drivers/mtd/nand/raw/Kconfig | 4 -
> drivers/mtd/nand/raw/Makefile | 1 -
> drivers/mtd/nand/raw/cmx270_nand.c | 236 -----------------------------
> 6 files changed, 131 insertions(+), 243 deletions(-)
> delete mode 100644 drivers/mtd/nand/raw/cmx270_nand.c
>
Any chance you give this series a try? I plan to merge several series
like that during this release, but of course if you can test it
beforehands it's even better!
Thanks,
Miquèl
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
WARNING: multiple messages have this Message-ID (diff)
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Vignesh Raghavendra <vigneshr@ti.com>,
Arnd Bergmann <arnd@arndb.de>,
Tudor Ambarus <tudor.ambarus@microchip.com>,
Richard Weinberger <richard@nod.at>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Boris Brezillon <boris.brezillon@collabora.com>,
linux-mtd@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
Mike Rapoport <mike@compulab.co.il>,
Daniel Mack <daniel@zonque.org>
Subject: Re: [PATCH 0/3] mtd: rawnand: Get rid of the cmx270 driver
Date: Fri, 8 May 2020 12:10:41 +0200 [thread overview]
Message-ID: <20200508121041.64f91276@xps13> (raw)
In-Reply-To: <20200429223134.789322-1-boris.brezillon@collabora.com>
Hi Robert,
Boris Brezillon <boris.brezillon@collabora.com> wrote on Thu, 30 Apr
2020 00:31:31 +0200:
> Hello,
>
> As part of my attempt to convert all NAND drivers to exec_op() I noticed
> the cmx270 board didn't really deserve a custom driver since other boards
> using the same SoC (em-x270 to name one) are using the gen_nand driver.
> I think the only issue with the CM-X270 is that the chip is connected
> to D[16:23] (or D[16:31] if it's a 16bit bus). Adjusting the mem
> resource offset should do the trick.
>
> I hope someone still has a board to test that.
>
> Regards,
>
> Boris
>
> Boris Brezillon (3):
> ARM: pxa: cm-x270: Use gen_nand to expose the NAND device
> ARM: pxa: Stop selecting CONFIG_MTD_NAND_CM_X270
> mtd: rawnand: Remove the cmx270 NAND controller driver
>
> arch/arm/configs/cm_x2xx_defconfig | 1 -
> arch/arm/configs/pxa_defconfig | 1 -
> arch/arm/mach-pxa/cm-x270.c | 131 ++++++++++++++++
> drivers/mtd/nand/raw/Kconfig | 4 -
> drivers/mtd/nand/raw/Makefile | 1 -
> drivers/mtd/nand/raw/cmx270_nand.c | 236 -----------------------------
> 6 files changed, 131 insertions(+), 243 deletions(-)
> delete mode 100644 drivers/mtd/nand/raw/cmx270_nand.c
>
Any chance you give this series a try? I plan to merge several series
like that during this release, but of course if you can test it
beforehands it's even better!
Thanks,
Miquèl
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-05-08 10:11 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-29 22:31 [PATCH 0/3] mtd: rawnand: Get rid of the cmx270 driver Boris Brezillon
2020-04-29 22:31 ` Boris Brezillon
2020-04-29 22:31 ` [PATCH 1/3] ARM: pxa: cm-x270: Use gen_nand to expose the NAND device Boris Brezillon
2020-04-29 22:31 ` Boris Brezillon
2020-04-29 22:31 ` [PATCH 2/3] ARM: pxa: Stop selecting CONFIG_MTD_NAND_CM_X270 Boris Brezillon
2020-04-29 22:31 ` Boris Brezillon
2020-04-29 22:31 ` [PATCH 3/3] mtd: rawnand: Remove the cmx270 NAND controller driver Boris Brezillon
2020-04-29 22:31 ` Boris Brezillon
2020-05-08 10:10 ` Miquel Raynal [this message]
2020-05-08 10:10 ` [PATCH 0/3] mtd: rawnand: Get rid of the cmx270 driver Miquel Raynal
2020-05-13 12:55 ` Robert Jarzmik
2020-05-13 12:55 ` Robert Jarzmik
2020-05-13 13:17 ` Boris Brezillon
2020-05-13 13:17 ` Boris Brezillon
2020-05-13 13:23 ` Miquel Raynal
2020-05-13 13:23 ` Miquel Raynal
2020-05-13 13:58 ` Arnd Bergmann
2020-05-13 13:58 ` Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200508121041.64f91276@xps13 \
--to=miquel.raynal@bootlin.com \
--cc=arnd@arndb.de \
--cc=boris.brezillon@collabora.com \
--cc=daniel@zonque.org \
--cc=haojian.zhuang@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-mtd@lists.infradead.org \
--cc=mike@compulab.co.il \
--cc=richard@nod.at \
--cc=robert.jarzmik@free.fr \
--cc=tudor.ambarus@microchip.com \
--cc=vigneshr@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.