From: Tony Lindgren <tony@atomide.com>
To: "Gupta, Pekon" <pekon@ti.com>
Cc: linux-omap <linux-omap@vger.kernel.org>,
"bcousson@baylibre.com" <bcousson@baylibre.com>
Subject: Re: [PATCH v2 0/2] add parallel NAND support for TI's new OMAPx and AMxx platforms
Date: Fri, 18 Apr 2014 09:33:16 -0700 [thread overview]
Message-ID: <20140418163316.GD5354@atomide.com> (raw)
In-Reply-To: <20980858CB6D3A4BAE95CA194937D5E73EAB5FA6@DBDE04.ent.ti.com>
* Gupta, Pekon <pekon@ti.com> [140327 00:22]:
> HI Tony,
>
> >From: Gupta, Pekon
> [...]
> >I see following patches in your tree/branch "omap-for-v3.15/dt"
> >But probably they were omitted in the pull request. Is there something, I need to do for these ?
> >
> > f68e355 2014-03-02 ARM: dts: am43xx: add support for parallel NAND flash
> > c06c527 2014-03-02 ARM: dts: AM33xx: updated default ECC scheme in nand-ecc-opt
> > 91994fa 2014-03-02 ARM: dts: am335x-evm: NAND: update MTD partition table
> > 0611c41 2014-03-02 ARM: OMAP2+: gpmc: update gpmc_hwecc_bch_capable() for new platforms and ECC schemes
> >
> >
> >Also, I had sent a [v2] for one of the patches, dropped by you. Please see if that can be picked
> > [Patch v2 1/2] ARM: dts: dra7: add support for parallel NAND flash
> > [Patch v2 1/2] ARM: dts: am43xx: fix starting offset of NAND.filesystem MTD partition
> >
> >
> Please let me know, if I need to rebase any of the patches in follow series
> against your for_3.15/dt or any other branch.
Yes please rebase against v3.15-rc1 and repost, otherwise I'll have hard
time keeping track of what's still missing.
BTW, I think we have few more GPMC issue to debug before we can drop omap3
legacy booting support:
1. At least the gpmc_probe_generic_child fails if the device is initialized
by the bootloader, and then initialized by the kernel based on the .dts
entries. This works just fine when booted in legacy mode.
2. Also gpmc_cs_remap hangs the system if the bootloader address for the
device was different from the address in the .dts file.
3. The gpmc_cs_remap should use the ranges property instead of the child
reg property. The child reg property should be the offset from the
GPMC ranges property as at least smc91x wants 0x300 as the offset to
avoid extra warnings from the driver when it checks the address space.
Regards,
Tony
> (1) [Partly Merged] "add parallel NAND support for TI's new OMAPx and AMxx platforms"
> http://www.spinics.net/lists/linux-omap/msg104305.html
>
> (2) [New Patches] "add parallel NAND support for TI's new OMAPx and AMxx platforms (part-2)"
> http://www.spinics.net/lists/linux-omap/msg104668.html
prev parent reply other threads:[~2014-04-18 16:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-12 9:45 [PATCH v2 0/2] add parallel NAND support for TI's new OMAPx and AMxx platforms Pekon Gupta
2014-03-12 9:45 ` [PATCH v2 1/2] ARM: dts: dra7: add support for parallel NAND flash Pekon Gupta
2014-03-12 9:45 ` [PATCH v2 2/2] ARM: dts: am43xx: fix starting offset of NAND.filesystem MTD partition Pekon Gupta
2014-03-19 9:17 ` [PATCH v2 0/2] add parallel NAND support for TI's new OMAPx and AMxx platforms Gupta, Pekon
2014-03-27 7:19 ` Gupta, Pekon
2014-04-18 16:33 ` Tony Lindgren [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=20140418163316.GD5354@atomide.com \
--to=tony@atomide.com \
--cc=bcousson@baylibre.com \
--cc=linux-omap@vger.kernel.org \
--cc=pekon@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).