Attention is currently required from: Jason Glenesk, Furquan Shaikh, Marshall Dawson, Paul Menzel, Tim Wawrzynczak, Karthik Ramasubramanian, Felix Held.
5 comments:
Commit Message:
I decided
to copy and modify it because the Intel driver has a lot of Intel
specific code.
Thanks Raul!
Done
File src/soc/amd/common/block/rtd3/rtd3.c:
Name (_PR2, Package (0x01) // _PR2: Power Resources for D2
* {
* RTD3
* })
Please also mention this difference in the commit message.
Removed _PR2, so no longer relevant.
Thanks Raul!
Looks like the upstream patch has been tested on the original hardware. I'll get that pulled into our tree.
Patch Set #1, Line 210: pci_dev_read_resources
No worries, glad we hashed it out. At least we know why we are doing it this way now :)
So is the driver good as is then?
Probably make a note that the driver does not handle any downstream graphics devices?
Should I just call pci_rom_ssdt from the rtd3 ssdt method?
Patch Set #1, Line 221: drivers_nvme_generic_config
Is it soc_amd_common_block_rtd3_config?
Done
To view, visit change 54966. To unsubscribe, or for help writing mail filters, visit settings.