Re: [PATCH 2/2] ARM: dts: add AM33XX MMC support

Felipe,

On Friday, June 14, 2013, Felipe Balbi wrote:

Hi,

On Fri, Jun 14, 2013 at 02:54:33PM -0500, Joel A Fernandes wrote:

quoted

Hi Tony, Vaibhav,

quoted

quoted

I just doublechecked MMC rootfs on bone and evmsk as it's the standard
smoke test. My EVM is intermittent now so trying to coax it to power

up

quoted

quoted

quoted

to reverify.

Matt,

Your branch is working for me, I tested it on EVM. Not sure what is

wrong with manual rebasing

quoted

quoted

I did yesterday. I will try to nail-down root-cause here.

And you can submit the next version fixing the comments I have given.

The only comment I could find in the whole discussion was adding of
the interrupts property. I tested I don't have any problem booting
with Matt's patch, without having the interrupts property.

might very well be, but we want to figure out why adding "intrrupts"
causes issues. Why does it have to come through hwmod ? You might have
uncovered a real problem which needs fixing. Besides, at the end of the

No actually I think I wasn't very descriptive in my post. There is no
problem here. Whether it comes from DT or not, it works in both cases . (I
haven't tried adding interrupt node to dts but what I gathered from
Vaibhav's post is that it works also when added to dts)

But I can revisit Vaibhav's suggestion of adding it to dts for completeness
sake. My tree currently works and picks up the interrupt info from hwmod. I
just didn't see any activity on this thread for a while so I thought of
enquiring it was ok to pull it in for the 3.11 merge window or if the
consensus was that interrupt node be added to dts even though it works
without it.

Thanks,

-Joel

day, we want to remove the redundant hwmod data if we already have DTS
bindings covering the same requirements.
--
balbi