linux-kernel - Re: [v7, 1/3] nand: pl353: Add basic driver for arm pl353 smc nand interface

Open Source and information security mailing list archives

Date:   Fri, 21 Oct 2016 23:46:11 +0200
From:   Boris Brezillon <boris.brezillon@...e-electrons.com>
To:     Jason Gunthorpe <jgunthorpe@...idianresearch.com>
Cc:     Punnaiah Choudary Kalluri <punnaiah.choudary.kalluri@...inx.com>,
        mark.rutland@....com, jussi.kivilinna@....fi,
        linux-doc@...r.kernel.org, artem.bityutskiy@...ux.intel.com,
        linux-mtd@...ts.infradead.org, punnaia@...inx.com, arnd@...db.de,
        michal.simek@...inx.com, ezequiel.garcia@...e-electrons.com,
        grant.likely@...aro.org, devicetree@...r.kernel.org,
        jason@...edaemon.net, pawel.moll@....com,
        ijc+devicetree@...lion.org.uk, joern@...fs.org, kpc528@...il.com,
        robh+dt@...nel.org, acourbot@...dia.com,
        kalluripunnaiahchoudary@...il.com, gregkh@...uxfoundation.org,
        linux-kernel@...r.kernel.org, rob@...dley.net,
        galak@...eaurora.org, ivan.khoronzhuk@...com,
        computersforpeace@...il.com, dwmw2@...radead.org
Subject: Re: [v7, 1/3] nand: pl353: Add basic driver for arm pl353 smc nand
 interface

On Fri, 21 Oct 2016 14:33:22 -0600
Jason Gunthorpe <jgunthorpe@...idianresearch.com> wrote:

> On Mon, Jun 08, 2015 at 11:38:36PM +0530, Punnaiah Choudary Kalluri wrote:
> > Add driver for arm pl353 static memory controller nand interface with
> > HW ECC support. This controller is used in xilinx zynq soc for interfacing
> > the nand flash memory.
> > 
> > Signed-off-by: Punnaiah Choudary Kalluri <punnaia@...inx.com>
> > Changes in v7:
> >  - Currently not implemented the memclk rate adjustments. I will
> >    look into this later and once the basic driver is accepted.
> >  - Fixed GPL licence ident
> > Changes in v6:
> >  - Fixed the checkpatch.pl reported warnings
> >  - Using the address cycles information from the onfi param page
> >    earlier it is hardcoded to 5 in driver
> > Changes in v5:
> >  - Configure the nand timing parameters as per the onfi spec
> > Changes in v4:
> >  - Updated the driver to sync with pl353_smc driver APIs
> > Changes in v3:
> >  - implemented the proper error codes
> >  - further breakdown this patch to multiple sets
> >  - added the controller and driver details to Documentation section
> >  - updated the licenece to GPLv2
> >  - reorganized the pl353_nand_ecc_init function
> > Changes in v2:
> >  - use "depends on" rather than "select" option in kconfig
> >  - remove unused variable parts
> >  - remove dummy helper and use writel_relaxed directly  
> 
> What is the status of getting this driver merged? I am interested in
> this driver and can provide some help if there are still TODO items.

I'll try to review it next week.

> 
> At the very least I will test it, is there something newer than v7 out
> there?
> 
> Regards,
> Jason
> 
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives