Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build on aarch64 #89

Closed
wants to merge 8 commits into from
Closed

Conversation

valpackett
Copy link
Contributor

This PR adds some basic portability and makes radeon and amdgpu build on aarch64. Does not include openfirmware stuff (damn that's a lot of API surface), only works with the existing PCIe support.

radeon loads and says this:

[drm] radeon kernel modesetting enabled.

amdgpu doesn't load (this is a soft FPU function, I guess we need a flag other than -mhard-float??):

link_elf: symbol __eqdf2 undefined

Whether it actually works is not tested… porting rkpcie would be the first requirement on my hardware (RK3399), and then there's the issue of whether u-boot configures pci correctly. Marvell Armada 8k (MACCHIATObin) is at least known to work with GPUs on Linux, but we don't yet have pcie on that either. I guess if someone has physical access to a ThunderX box, they can test it :D Jetson TX1 might have PCIe working too?

@iotamudelta
Copy link
Member

@johalun any opinions on this?

If it looks good to you, I'm happy to test it for the port.

@johalun
Copy link
Member

johalun commented Oct 1, 2018

No makefiles in the drivers/ subfolders are used so no need changing them. We have to remember to pull this in to 4.17 later also.

@johalun
Copy link
Member

johalun commented Oct 1, 2018

@myfreeweb Can you revert the changes to makefiles in drivers/ so we don't get merge fail on those later?

@evadot
Copy link
Contributor

evadot commented Oct 29, 2018

I don't see the point of merging something that isn't tested.
Loading module is one thing, using them is another thing (trust me on this).

@mattmacy
Copy link
Member

Is there anyone with an arm64 with a radeon or amdgpu graphics card to test? Otherwise I'm inclined to agree with @evadot I didn't push my power changes until I had confirmed that the driver at least attached.

@mattmacy
Copy link
Member

mattmacy commented Dec 3, 2018

@myfreeweb Can we close this? The POWER support changes should make this easier to add in the future if someone has appropriate hardware.

@valpackett
Copy link
Contributor Author

Yes, I'll redo this on top of the POWER changes when I get hardware to test on. (btw, did you find anything from this PR useful for the POWER changes?)

@valpackett valpackett closed this Dec 4, 2018
@valpackett valpackett deleted the aarch64-pr branch December 4, 2018 20:13
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.

None yet

5 participants