On Sun, Aug 02, 2020 at 06:51:53PM +0000, mglockeropenbsd!org wrote:
> On Mon, 3 Aug 2020 02:24:47 +1000 Jonathan Gray <[email protected]> wrote:
> > On Sun, Aug 02, 2020 at 01:05:37PM +0200, [email protected] 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.