Skip to content

ARM: dts: zynq: parallella: Fix up reserved-memory and memory nodes#11

Merged
olajep merged 1 commit intoparallella:parallella-linux-2016.3from
nathanrossi:nrossi/parallella-fixup-dt-reservedmem
May 31, 2016
Merged

ARM: dts: zynq: parallella: Fix up reserved-memory and memory nodes#11
olajep merged 1 commit intoparallella:parallella-linux-2016.3from
nathanrossi:nrossi/parallella-fixup-dt-reservedmem

Conversation

@nathanrossi
Copy link
Copy Markdown

This pull request resolves issues where the reserved memory region is not available to the elink driver.

[    1.777966] elink 81000000.elink0: reserved-mem: failed requesting mem region
[    1.785062] elink 81000000.elink0: reserved mem probe failed
[    1.790665] elink 81000000.elink0: Failed parsing device tree
[    1.796490] elink: probe of 81000000.elink0 failed with error -12

With these changes the reserved memory is now available for request by the driver, and allows for the driver to probe correctly.

[    1.853077] epiphany elink0: elink: version field empty. Using default platform.
[    1.853095] epiphany elink0: Epiphany FPGA elink at address 0x81000000
[    1.853095] epiphany elink0: platform 01 revision 00
root@parallella:~# cat /proc/iomem
00000000-3dffffff : System RAM
  00208000-00e5bf37 : Kernel code
  00f64000-010b1b57 : Kernel data
3e000000-3fffffff : 81000000.elink0
80000000-8fffffff : 81000000.elink0
...

Setup the memory node so that it covers all the available memory. This
adds the chunk back that is specifically used for the elink as the
kernel will handle reserving it.

Fix the reserved-memory node for emem so that it has the 'no-map'
property as the property 'no-map' is not inherited from the parent node.
Also fix up the node name (missing a 0) to match the reg property.

Signed-off-by: Nathan Rossi <[email protected]>
@olajep olajep merged commit 18c456a into parallella:parallella-linux-2016.3 May 31, 2016
@olajep
Copy link
Copy Markdown
Member

olajep commented May 31, 2016

Thank you Nathan!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants