mbox

[U-Boot] Pull request: nand flash master

Message ID 20150826035426.GA30213@home.buserror.net
State Accepted
Delegated to: Tom Rini
Headers show

Pull-request

git://git.denx.de/u-boot-nand-flash.git

Message

Scott Wood Aug. 26, 2015, 3:54 a.m. UTC
The following changes since commit c851a2458fbc12495f4f786d4eabb612850a5143:

  Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-nand-flash.git 

for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543:

  mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -0500)

----------------------------------------------------------------
Ezequiel Garcia (1):
      mtd: Introduce mtd_block_isreserved()

Peng Fan (2):
      mtd: nand: mxs support oobsize bigger than 512
      mtd: nand: mxs invalidate dcache before DMA read

Scott Wood (2):
      nand: Remove __UBOOT__ ifdefs
      nand: Sync with Linux v4.1

Siva Durga Prasad Paladugu (1):
      mtd: nand: Increase max sizes of OOB and Page size

Stefan Roese (2):
      mtd: nand: Add mvebu (PXA / AXP / A38x) NAND device driver
      arm: mvebu: Enable NAND on db-mv784mp-gp

 arch/arm/include/asm/imx-common/regs-bch.h |    2 +
 configs/db-mv784mp-gp_defconfig            |    1 +
 drivers/mtd/mtdcore.c                      |   14 +-
 drivers/mtd/mtdpart.c                      |    9 +
 drivers/mtd/nand/Kconfig                   |    7 +
 drivers/mtd/nand/Makefile                  |    2 +
 drivers/mtd/nand/atmel_nand_ecc.h          |    3 +
 drivers/mtd/nand/denali.c                  |  209 ++--
 drivers/mtd/nand/denali.h                  |    8 +-
 drivers/mtd/nand/docg4.c                   |    6 +-
 drivers/mtd/nand/fsl_elbc_nand.c           |   14 +
 drivers/mtd/nand/fsl_ifc_nand.c            |   33 +-
 drivers/mtd/nand/mxs_nand.c                |   38 +-
 drivers/mtd/nand/nand_base.c               |  615 +++--------
 drivers/mtd/nand/nand_bbt.c                |   75 +-
 drivers/mtd/nand/nand_ids.c                |   10 +-
 drivers/mtd/nand/nand_timings.c            |  252 +++++
 drivers/mtd/nand/pxa3xx_nand.c             | 1621 ++++++++++++++++++++++++++++
 drivers/mtd/nand/pxa3xx_nand.h             |   64 ++
 include/configs/db-mv784mp-gp.h            |    5 +
 include/fsl_ifc.h                          |    2 +
 include/linux/mtd/mtd.h                    |    4 +-
 include/linux/mtd/nand.h                   |  126 ++-
 23 files changed, 2448 insertions(+), 672 deletions(-)
 create mode 100644 drivers/mtd/nand/nand_timings.c
 create mode 100644 drivers/mtd/nand/pxa3xx_nand.c
 create mode 100644 drivers/mtd/nand/pxa3xx_nand.h

Comments

Tom Rini Aug. 26, 2015, 5:53 p.m. UTC | #1
On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote:

> The following changes since commit c851a2458fbc12495f4f786d4eabb612850a5143:
> 
>   Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-nand-flash.git 
> 
> for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543:
> 
>   mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -0500)
> 

Applied to u-boot/master, thanks!

But please note that I now see:
+(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too big

Which I'm hoping can be tuned to re-fit size constraints again easily
enough.
Scott Wood Aug. 26, 2015, 5:57 p.m. UTC | #2
On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote:
> On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote:
> 
> > The following changes since commit 
> > c851a2458fbc12495f4f786d4eabb612850a5143:
> > 
> >   Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400)
> > 
> > are available in the git repository at:
> > 
> >   git://git.denx.de/u-boot-nand-flash.git 
> > 
> > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543:
> > 
> >   mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -
> > 0500)
> > 
> 
> Applied to u-boot/master, thanks!
> 
> But please note that I now see:
> +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too 
> big
> 
> Which I'm hoping can be tuned to re-fit size constraints again easily
> enough.

They built OK for me, so they must be close enough that the toolchain makes a 
difference.

-Scott
Tom Rini Aug. 26, 2015, 6:06 p.m. UTC | #3
On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote:
> On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote:
> > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote:
> > 
> > > The following changes since commit 
> > > c851a2458fbc12495f4f786d4eabb612850a5143:
> > > 
> > >   Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400)
> > > 
> > > are available in the git repository at:
> > > 
> > >   git://git.denx.de/u-boot-nand-flash.git 
> > > 
> > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543:
> > > 
> > >   mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -
> > > 0500)
> > > 
> > 
> > Applied to u-boot/master, thanks!
> > 
> > But please note that I now see:
> > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too 
> > big
> > 
> > Which I'm hoping can be tuned to re-fit size constraints again easily
> > enough.
> 
> They built OK for me, so they must be close enough that the toolchain makes a 
> difference.

Indeed.  I'm using arm-none-eabi from Debian on that test box (since it
will also build the armv7m targets).
Masahiro Yamada Aug. 27, 2015, 7:06 a.m. UTC | #4
2015-08-27 3:06 GMT+09:00 Tom Rini <trini@konsulko.com>:
> On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote:
>> On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote:
>> > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote:
>> >
>> > > The following changes since commit
>> > > c851a2458fbc12495f4f786d4eabb612850a5143:
>> > >
>> > >   Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400)
>> > >
>> > > are available in the git repository at:
>> > >
>> > >   git://git.denx.de/u-boot-nand-flash.git
>> > >
>> > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543:
>> > >
>> > >   mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -
>> > > 0500)
>> > >
>> >
>> > Applied to u-boot/master, thanks!
>> >
>> > But please note that I now see:
>> > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too
>> > big
>> >


These are my boards.

I will do something with them.
Masahiro Yamada Aug. 27, 2015, 9:57 a.m. UTC | #5
Hi Tom, Scott,


2015-08-27 3:06 GMT+09:00 Tom Rini <trini@konsulko.com>:
> On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote:
>> On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote:
>> > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote:
>> >
>> > > The following changes since commit
>> > > c851a2458fbc12495f4f786d4eabb612850a5143:
>> > >
>> > >   Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400)
>> > >
>> > > are available in the git repository at:
>> > >
>> > >   git://git.denx.de/u-boot-nand-flash.git
>> > >
>> > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543:
>> > >
>> > >   mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -
>> > > 0500)
>> > >
>> >
>> > Applied to u-boot/master, thanks!
>> >
>> > But please note that I now see:
>> > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too
>> > big
>> >
>> > Which I'm hoping can be tuned to re-fit size constraints again easily
>> > enough.
>>
>> They built OK for me, so they must be close enough that the toolchain makes a
>> difference.
>
> Indeed.  I'm using arm-none-eabi from Debian on that test box (since it
> will also build the armv7m targets).


I've sent a patch to fix the root cause of this problem.
http://patchwork.ozlabs.org/patch/511211/

Only my boards (ph1_ld4, ph1_sld8, ph1_pro4) define CONFIG_SPL_NAND_DENALI,
and I tested the patch on them.

So, this patch should be safe.


Is it OK to apply it quickly?
Tom Rini Aug. 27, 2015, 11:17 a.m. UTC | #6
On Thu, Aug 27, 2015 at 06:57:13PM +0900, Masahiro Yamada wrote:
> Hi Tom, Scott,
> 
> 
> 2015-08-27 3:06 GMT+09:00 Tom Rini <trini@konsulko.com>:
> > On Wed, Aug 26, 2015 at 12:57:14PM -0500, Scott Wood wrote:
> >> On Wed, 2015-08-26 at 13:53 -0400, Tom Rini wrote:
> >> > On Tue, Aug 25, 2015 at 10:54:26PM -0500, Scott Wood wrote:
> >> >
> >> > > The following changes since commit
> >> > > c851a2458fbc12495f4f786d4eabb612850a5143:
> >> > >
> >> > >   Merge git://git.denx.de/u-boot-socfpga (2015-08-23 20:44:25 -0400)
> >> > >
> >> > > are available in the git repository at:
> >> > >
> >> > >   git://git.denx.de/u-boot-nand-flash.git
> >> > >
> >> > > for you to fetch changes up to ecfb8768b16ebf6781ee0df45742674bda8a9543:
> >> > >
> >> > >   mtd: nand: mxs invalidate dcache before DMA read (2015-08-25 22:53:59 -
> >> > > 0500)
> >> > >
> >> >
> >> > Applied to u-boot/master, thanks!
> >> >
> >> > But please note that I now see:
> >> > +(ph1_sld8,ph1_ld4,ph1_pro4) arm-none-eabi-ld.bfd: SPL image plus BSS too
> >> > big
> >> >
> >> > Which I'm hoping can be tuned to re-fit size constraints again easily
> >> > enough.
> >>
> >> They built OK for me, so they must be close enough that the toolchain makes a
> >> difference.
> >
> > Indeed.  I'm using arm-none-eabi from Debian on that test box (since it
> > will also build the armv7m targets).
> 
> 
> I've sent a patch to fix the root cause of this problem.
> http://patchwork.ozlabs.org/patch/511211/
> 
> Only my boards (ph1_ld4, ph1_sld8, ph1_pro4) define CONFIG_SPL_NAND_DENALI,
> and I tested the patch on them.
> 
> So, this patch should be safe.
> 
> 
> Is it OK to apply it quickly?

Yes it is, thanks!