Mailinglist Archive: radeonhd (319 mails)

[bugzilla-daemon@xxxxxxxxxxxxxxx]

http://bugs.freedesktop.org/show_bug.cgi?id=18393





--- Comment #9 from Jung-uk Kim <jkim@xxxxxxxxxxx>  2008-11-17 10:11:14 PST ---
(In reply to comment #8)
> Kim, 
> a. on FreeBSD you don't get the sizes filled in when you call pci_fill_info()
> with   PCI_FILL_SIZES? bummer! I agree that this call needs to be added and we
> also need some fallback if the sizes are not filled in. So your rhd_conntest
> fix looks ok.
> I may probably wrap things with '#if defined (__FreeBSD__)' for now.

I see that you already checked it in.  Thanks.  However, I believe it is more
like this:

@@ -547,7 +547,7 @@ MapBar(struct pci_dev *device, int ioBar, int devMem)

     pci_fill_info(device, PCI_FILL_BASES | PCI_FILL_SIZES);
     if (!device->base_addr[ioBar]
-#if !defined (__FreeBSD__)
+#if defined (__linux__)
        || !device->size[ioBar]
 #endif
        )

or you don't want to check at all because only Linux with procfs or sysfs fills
it up by default, as I said.  For other platforms, libpci does/can not fill the
sizes according to libpci sources.

> b. Could you switch to use AtomBIOS and see if load detection works there?
> If it does i need to have a look at your BIOS.
> To do this please add the option: 
>     Option "UseAtomBIOS" "yes"
> to the device section of your xorg.conf and create a logverbose 7 log for the
> output.

Yes, it does.  However, it fails to restore previous text mode when it exits. 
I will attach two ROMs, one for HDMI and one for DVI.  This board has both HDMI
and DVI ports and one of them can be selected from BIOS.  By selecting the
option, corresponding VGA ROM is loaded when BIOS is POST'ed.

> I'm going to mark HPD as broken for your chip for now.

I see you also checked it in.  Thanks, the HPD is properly black-listed for
both HDMI and DVI cases.

> c.
> > > A note on the side: the change to rhd_randr.c should not matter. There is 
> > > a set
> > > of brackets missing, but this should be irrelevant, too.
> >
> > I know but it was confusing because of the indentation. ;-)
> Right. Saw that, should be fixed now :)

:-)

> Thanks for hanging in so patiently :)

Thank YOU for listening to my whining. ;-)


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
xorg-team mailing list
xorg-team@xxxxxxxxxxx
http://lists.x.org/mailman/listinfo/xorg-team
-- 
To unsubscribe, e-mail: radeonhd+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: radeonhd+help@xxxxxxxxxxxx