On Mon, 2018-07-23 at 11:38 -0700, Andrey Smirnov wrote:

On Mon, Jul 23, 2018 at 5:41 AM Leonard Crestez [off-list ref] wrote:

quoted

On Fri, 2018-07-20 at 08:33 -0700, Andrey Smirnov wrote:

quoted

On Fri, Jul 20, 2018 at 5:48 AM Leonard Crestez [off-list ref] wrote:

quoted

This reverts commit 1c86c9dd82f859b474474a7fee0d5195da2c9c1d.

That commit followed the reference manual but unfortunately the imx7d
manual is incorrect.

I'd also add similar comment to DT file to prevent people from trying
to "fix" this in the future.

I'll try to see if I can follow up internally with docs team to get
this updated in the next revision of the reference manual.

Not sure if we are on the same page here, but what I meant is adding
the same explanation that is in your commit message to Device Tree
file as well so that if anyone looks at DT code and goes "Huh?" in the
future, they wouldn't have to research commit history to see the
reason why things the way they are.

OK, I will add a comment on irq mappings in the next version explaining
that the reference manual is incorrect.

quoted

quoted

Also, this change is going to break QEMU's mapping found here:

I had no idea that existed, I guess somebody needs to fix that as well.

I take it it's a no to my "any chance you can fix that as well?" ;-).
I'll see if I can find time to do that this week.

Sorry, I'm not familiar with qemu source at all.