On Sun, Aug 02, 2020 at 06:51:53PM +0000, mglockeropenbsd!org wrote: > On Mon, 3 Aug 2020 02:24:47 +1000 Jonathan Gray <j...@jsg.id.au> wrote: > > On Sun, Aug 02, 2020 at 01:05:37PM +0200, mgloc...@openbsd.org wrote: > > > System : OpenBSD 6.7 > > > Installing an OpenBSD snapshot the first time on this iMac Intel > > > Core i3 machine. After the radaeon firmware package has been > > > installed and you reboot the system, the screen remains black after > > > the radeondrm driver initializes. > > > > Try this. RV730 is DCE3.2 > > Thanks, that looked very promising. > > I've double checked that with your diff it reaches the desired branch > now, but unfortunately the screen still remains dark :-( enc_idx gets > set to 1.
OpenBSD 7.0 on an Intel iMac Core i3 (iMac11,2; RV730) had the same black screen behaviour, but setting enc_idx to 1 did restore a working screen with radeondrm: > > Index: sys/dev/pci/drm/radeon/atombios_encoders.c > > =================================================================== > > RCS file: /cvs/src/sys/dev/pci/drm/radeon/atombios_encoders.c,v > > retrieving revision 1.15 > > diff -u -p -U6 -r1.15 atombios_encoders.c > > --- sys/dev/pci/drm/radeon/atombios_encoders.c 8 Jun 2020 > > 04:48:15 -0000 1.15 +++ > > sys/dev/pci/drm/radeon/atombios_encoders.c 2 Aug 2020 16:18:23 > > -0000 @@ -2191,13 +2191,14 @@ int radeon_atom_pick_dig_encoder(struct > > /* > > * On DCE32 any encoder can drive any block so usually just > > use crtc id, > > * but Apple thinks different at least on iMac10,1, so there > > use linkb, > > * otherwise the internal eDP panel will stay dark. > > */ > > if (ASIC_IS_DCE32(rdev)) { > > - if (dmi_match(DMI_PRODUCT_NAME, "iMac10,1")) > > + if (dmi_match(DMI_PRODUCT_NAME, "iMac10,1") || > > + dmi_match(DMI_PRODUCT_NAME, "iMac11,2")) > > enc_idx = (dig->linkb) ? 1 : 0; > > else > > enc_idx = radeon_crtc->crtc_id; > > > > goto assigned; > > } I haven't extensively tested the system with the patch but DPMS is working and I played some bzflag with high graphics settings; have not seen any regressions yet.