Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757355AbcJTJEp (ORCPT ); Thu, 20 Oct 2016 05:04:45 -0400 Received: from mail-lf0-f41.google.com ([209.85.215.41]:35645 "EHLO mail-lf0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754190AbcJTJEl (ORCPT ); Thu, 20 Oct 2016 05:04:41 -0400 Subject: Re: [PATCH v2] mtd: nand: Add OX820 NAND Support To: Boris Brezillon References: <20161020084901.6486-1-narmstrong@baylibre.com> <20161020110025.11b06090@bbrezillon> Cc: dwmw2@infradead.org, computersforpeace@gmail.com, richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-oxnas@lists.tuxfamily.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, daniel@makrotopia.org From: Neil Armstrong Organization: Baylibre Message-ID: Date: Thu, 20 Oct 2016 11:04:29 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: <20161020110025.11b06090@bbrezillon> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2865 Lines: 80 On 10/20/2016 11:00 AM, Boris Brezillon wrote: > On Thu, 20 Oct 2016 10:49:01 +0200 > Neil Armstrong wrote: > >> Add NAND driver to support the Oxford Semiconductor OX820 NAND Controller. >> This is a simple memory mapped NAND controller with single chip select and >> software ECC. >> >> Acked-by: Rob Herring >> Signed-off-by: Neil Armstrong >> --- >> .../devicetree/bindings/mtd/oxnas-nand.txt | 41 +++++ >> drivers/mtd/nand/Kconfig | 5 + >> drivers/mtd/nand/Makefile | 1 + >> drivers/mtd/nand/oxnas_nand.c | 196 +++++++++++++++++++++ >> 4 files changed, 243 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/mtd/oxnas-nand.txt >> create mode 100644 drivers/mtd/nand/oxnas_nand.c >> >> Changes since v1 http://lkml.kernel.org/r/20161019145523.6763-1-narmstrong@baylibre.com : >> - Simplify cmd_ctrl command and drop the ctrl address offset >> - Change oxnas_nand struct name to oxnas_nand_ctrl >> - Update DT-Bindings example to reflect the ctrl->chip->partitions hierarchy >> >> Changes since RFC http://lkml.kernel.org/r/20161018090927.1990-1-narmstrong@baylibre.com : >> - Avoid using chip->IO_ADDR* >> - Use new DT structure >> - Assign a chip for the subnode >> - Use the nand_hw_control structure >> - Cleanup probe >> - Cleanup cmd_ctrl by using a context ctrl offset used in write_bytes >> >> diff --git a/Documentation/devicetree/bindings/mtd/oxnas-nand.txt b/Documentation/devicetree/bindings/mtd/oxnas-nand.txt >> new file mode 100644 >> index 0000000..33a77b8 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mtd/oxnas-nand.txt >> @@ -0,0 +1,41 @@ >> +* Oxford Semiconductor OXNAS NAND Controller >> + >> +Please refer to nand.txt for generic information regarding MTD NAND bindings. >> + >> +Required properties: >> + - compatible: "oxsemi,ox820-nand" >> + - reg: Base address and length for NAND mapped memory. >> + >> +Optional Properties: >> + - clocks: phandle to the NAND gate clock if needed. >> + - resets: phandle to the NAND reset control if needed. >> + >> +Example: >> + >> +nand: nand-controller@41000000 { > > 'nandc:' or 'nand_ctrl:' > > Otherwise it may conflict with a NAND chip alias. > The rest looks good, so no need to send a new version (I can fix it when > applying the patch). OK, thanks, I will fix in the dtsi accordingly. > > One last thing, I saw there was other people owning boards with this > controller. Can I get one or two Tested-by? I would like too, but it seems I'm the only one working upstream for now.... I hope it will change in near future once there is enough upstream for booting the platform. > > Thanks taking my reviews into account. Thanks for reviewing ! Neil > > Regards, > > Boris >