On 29.11.2012 16:08, Philip, Avinash wrote:

On Thu, Nov 29, 2012 at 17:28:04, Daniel Mack wrote:

quoted

Hi Avinash,

On 29.11.2012 06:24, Philip, Avinash wrote:

quoted

On Wed, Nov 28, 2012 at 22:28:55, Daniel Mack wrote:

quoted

[Resending +devicetree-discuss, +Rob, +Grant]


This is a series of patches to support GPMC peripherals on OMAP boards.

Depends on Linus' master +
omap-next (branch omap-for-v3.8/cleanup-headers-gpmc)

Can you resend this series on top of linux_next?

The only branch these patches depend on is the
"omap-for-v3.8/cleanup-headers-gpmc" branch from omap_next. Are you
actually seeing any merge conflicts with my series? If so, which branch
are you referring to exactly?

quoted

Some of the missing items I seen 
1. Of_node not populated in  omap_nand_platform_data structure.

Hmm - gpmc_probe_nand_child() from 4/4 adds

  gpmc_nand_data->of_node = child;

Do I miss anything?

I didn't found definition for of_node member in omap_nand_platform_data
structure. This also might be missing in liniux-next?

Ah, right. I got that entry from your patch ("mtd: nand: omap2: Support
for hardware BCH error correction"). But you're right, it should be part
of my series. Will add that when resending, but note that your ELM
series will then cause a (trivial) merge conflict.

quoted

quoted

2. Remove platform device creation from hwmod as GPMC DT is populating.
   Currently GPMC device creaing from DT & HWMOD.

This is already addressed in cd00b0530 ("ARM: OMAP2+: gpmc: Fix kernel
BUG for DT boot mode") by Vaibhav Hiremath, which I got via Afzal's USB
branch:

Purpose of VH's patch was to get beagle bone booting, but Jon sent another
patch that made beagle bone boot and it has reached mainline by v3.7-rc2,
hence the patch you are referring to is currently not going upstream. Still to
have GPMC DT work, we need that patch as other wise there would be two GPMC
devices. In earlier versions of your series, you had a similar patch, it is
required, can you please add it to your series as otherwise with your series
GPMC DT won't work.

Ok, fine. Will do.


Daniel