drm/nouveau: fix GA100 issues
Status from this message: Applied, thanks!
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest10 patches 4 repliesApplied
- v16 patches 3 repliesSuperseded
Changes in v2 from the cover letter
1) Rebased to latest drm-misc-next
2) Added Lyude's reviewed-by
3) Reverted original commit and refactored on top
4) Added code for row remapping work-around
Patches in v2 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/10 | Revert "drm/nouveau/gsp: add support for GA100"drivers/ drivers/ drivers/ | +18 | 0 replies | |
| 2/10 | drm/nouveau/gsp: add SEC2 to GA100 chip tabledrivers/ | +1 | 0 replies | |
| 3/10 | drm/nouveau/gsp: use fb.bios.addr for gspFwWprEnd instead of vga_workspace.addrdrivers/ | +1 | 0 replies | |
| 4/10 | drm/nouveau/gsp: read MMU_LOCK to fix WPR placement on GA100drivers/ | +49 | 0 replies | |
| 5/10 | drm/nouveau: only boot FRTS if its region is allocateddrivers/ | +6 | 0 replies | |
| 6/10 | drm/nouveau: GA100 has an FRTS region size of zerodrivers/ | +7 | 0 replies | |
| 7/10 | drm/nouveau/bios: specify correct display fuse register for Ampere and Adadrivers/ | +2 | 0 replies | |
| 8/10 | drm/nouveau/bios: skip the IFR header if presentdrivers/ | +101 | 0 replies | |
| 9/10 | drm/nouveau/gsp: require GSP-RM for GA100 supportdrivers/ | +0 | 0 replies | |
| 10/10 | drm/nouveau/gsp: formally support GA100drivers/ | +1 | 0 replies |
Reviews trailers per patch
3 of 10 patches have a Reviewed-by or Acked-by.
| Patch | Lyude Paul |
|---|---|
| 1/10 | |
| 2/10 | |
| 3/10 | |
| 4/10 | |
| 5/10 | |
| 6/10 | |
| 7/10 | R-b |
| 8/10 | R-b |
| 9/10 | R-b |
| 10/10 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v2 00/10] drm/nouveau: fix GA100 issues
The previous commit that claimed to add GA100 support, 20e0c197802c ("drm/nouveau/gsp: add support for GA100"), actually had quite a few problems. It falsely claimed that there was no VBIOS. GA100 does have a VBIOS, but it has no display engine, so it cannot use the PRAMIN method the read VBIOS, so it has to fall back to PROM. For whatever reason, the VBIOS on GA100 has an "Init-from-ROM" (IFR) header where the PCI Expansion ROM would normally be found. So to find that ROM, Nouveau needs to parse the IFR header.
The previous commit also claimed that there is no graphics (GR) engine. That is also false.
This patch set adds the IFR header parsing and a few other missing pieces to allow GA100 to actually boot and shut down GSP-RM properly.
Changes in v2:
1) Rebased to latest drm-misc-next
2) Added Lyude's reviewed-by
3) Reverted original commit and refactored on top
4) Added code for row remapping work-around
Timur Tabi (10):
Revert "drm/nouveau/gsp: add support for GA100" drm/nouveau/gsp: add SEC2 to GA100 chip table drm/nouveau/gsp: use fb.bios.addr for gspFwWprEnd instead of
vga_workspace.addr
drm/nouveau/gsp: read MMU_LOCK to fix WPR placement on GA100 drm/nouveau: only boot FRTS if its region is allocated drm/nouveau: GA100 has an FRTS region size of zero drm/nouveau/bios: specify correct display fuse register for Ampere and
Ada
drm/nouveau/bios: skip the IFR header if present drm/nouveau/gsp: require GSP-RM for GA100 support drm/nouveau/gsp: formally support GA100
.../gpu/drm/nouveau/nvkm/engine/device/base.c | 1 + .../nouveau/nvkm/subdev/bios/shadowramin.c | 3 +- .../drm/nouveau/nvkm/subdev/bios/shadowrom.c | 110 ++++++++++++++++-- .../gpu/drm/nouveau/nvkm/subdev/gsp/ga100.c | 5 +- .../gpu/drm/nouveau/nvkm/subdev/gsp/tu102.c | 61 +++++++++- 5 files changed, 163 insertions(+), 17 deletions(-)
Signature · 1 line
2.54.0