mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Ladislav Michl <ladis@linux-mips.org>
Cc: Barebox List <barebox@lists.infradead.org>
Subject: Re: [PATCH 00/23] AM3517 support
Date: Fri, 14 Dec 2018 20:32:39 +0100	[thread overview]
Message-ID: <20181214193239.mtxrzarszquoixxp@pengutronix.de> (raw)
In-Reply-To: <20181214162249.GA19868@lenoch>

On Fri, Dec 14, 2018 at 05:22:49PM +0100, Ladislav Michl wrote:
> Hello Sascha,
> 
> On Fri, Dec 14, 2018 at 03:17:07PM +0100, Sascha Hauer wrote:
> > This series adds support for the TI AM3517 SoC along with Wago PFC-200
> > board support. Also in this series some TI related driver patches. Most
> > notably some significant chages to the GPMC Nand driver which was not
> > really working anymore.
> > 
> > Sascha
> > 
> > Heinrich Toews (3):
> >   net: davinci-emac: fix buggy channel tear down
> >   net: davinci-emac: switch to device tree support
> >   ARM: omap: AM3517: Change default clock rate for AM3517
> > 
> > Oleg Karfich (1):
> >   ARM: omap3: Add support for reset reason detection
> > 
> > Sascha Hauer (19):
> >   net: davinci-emac: Add timeout to polling loop
> >   mci: omap: Improve error handling
> >   mci: omap: use IS_ENABLED() rather than #ifdef
> 
> >   mtd: nand: omap: Use dev_dbg when a struct device * is available
> >   mtd: nand: omap: Fix hamming correct return values
> >   mtd: nand: omap: Disable subpage reads in hardware ecc mode
> >   mtd: nand: omap: remove unused function argument
> >   mtd: nand: omap: fix bch8_hw_romcode ecc layout
> >   mtd: nand: omap: set eccbytes correctly
> >   mtd: nand: omap: unbreak BCH8 support
> 
> may I point there is a MTD update pending on mailing list which,
> once applied, would make merging with upstream drivers easier?
> Yes, that upstream, which already fixed those bugs...

Hm, somehow I assumed you would send a v2. I applied the series now
along with the fixup you sent.

However, the barebox OMAP nand driver has the special feature that we
can switch the ECC mode during runtime, so even with a uptodate NAND
stack we would still have to make adjustments to the Linux driver.

You are right, a nand stack better in sync with Linux would make things
easier, your series is a start.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      reply	other threads:[~2018-12-14 19:32 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-14 14:17 Sascha Hauer
2018-12-14 14:17 ` [PATCH 01/23] net: davinci-emac: fix buggy channel tear down Sascha Hauer
2018-12-14 14:17 ` [PATCH 02/23] net: davinci-emac: Add timeout to polling loop Sascha Hauer
2018-12-14 14:17 ` [PATCH 03/23] net: davinci-emac: switch to device tree support Sascha Hauer
2018-12-14 14:17 ` [PATCH 04/23] mci: omap: Improve error handling Sascha Hauer
2018-12-14 14:17 ` [PATCH 05/23] mci: omap: use IS_ENABLED() rather than #ifdef Sascha Hauer
2018-12-14 14:17 ` [PATCH 06/23] mtd: nand: omap: Use dev_dbg when a struct device * is available Sascha Hauer
2018-12-14 14:17 ` [PATCH 07/23] mtd: nand: omap: Fix hamming correct return values Sascha Hauer
2018-12-14 14:17 ` [PATCH 08/23] mtd: nand: omap: Disable subpage reads in hardware ecc mode Sascha Hauer
2018-12-14 14:17 ` [PATCH 09/23] mtd: nand: omap: remove unused function argument Sascha Hauer
2018-12-14 14:17 ` [PATCH 10/23] mtd: nand: omap: fix bch8_hw_romcode ecc layout Sascha Hauer
2018-12-14 14:17 ` [PATCH 11/23] mtd: nand: omap: set eccbytes correctly Sascha Hauer
2018-12-14 14:17 ` [PATCH 12/23] mtd: nand: omap: unbreak BCH8 support Sascha Hauer
2018-12-14 14:17 ` [PATCH 13/23] ARM: omap: Add missing include Sascha Hauer
2018-12-14 14:17 ` [PATCH 14/23] ARM: omap: dmtimer: Turn into a driver Sascha Hauer
2018-12-17 13:37   ` Teresa Remmet
2018-12-18  7:46     ` Sascha Hauer
2018-12-14 14:17 ` [PATCH 15/23] ARM: omap: 32ktimer: " Sascha Hauer
2018-12-14 14:17 ` [PATCH 16/23] ARM: omap: Add AM35XX support Sascha Hauer
2018-12-14 14:17 ` [PATCH 17/23] ARM: omap: enable am33xx_uart_soft_reset for AM35xx Sascha Hauer
2018-12-14 14:17 ` [PATCH 18/23] ARM: omap3: Add support for reset reason detection Sascha Hauer
2018-12-14 14:17 ` [PATCH 19/23] ARM: omap: Add AM3517 specific mux configuration Sascha Hauer
2018-12-14 14:17 ` [PATCH 20/23] ARM: omap: Add emif4 support for AM3517 Sascha Hauer
2018-12-14 14:17 ` [PATCH 21/23] ARM: omap: AM3517: Change default clock rate " Sascha Hauer
2018-12-14 14:17 ` [PATCH 22/23] ARM: omap: Add board support for WAGO pfc200 platform Sascha Hauer
2019-01-07 11:40   ` Heinrich.Toews
2019-01-07 11:46     ` Sascha Hauer
2018-12-14 14:17 ` [PATCH 23/23] ARM: omap: am33xx_defconfig: Enable more boards and rename Sascha Hauer
2018-12-14 16:22 ` [PATCH 00/23] AM3517 support Ladislav Michl
2018-12-14 19:32   ` Sascha Hauer [this message]

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=20181214193239.mtxrzarszquoixxp@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=ladis@linux-mips.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox