Matthew Garrett wrote:
> Micheal Marineau <[EMAIL PROTECTED]> wrote:
> 
> 
>>+     if (pdev->dev.power.power_state != 4)
>>+     {
>>+             pci_restore_state (pdev);
>>+             acpi_vgapost (pdev->devfn);
>>+     }
> 
> 
> Please *don't* make this unconditional. There's no guarantee that the
> video BIOS is postable after initial system boot. A boot-time parameter
> to enable it seems reasonable, but in general we're likely to stand a
> much better chance getting it working in userspace.

A kernel parameter seems reasonable, I assume that pci_restore_state
shoule remain unconditional right?  Userspace works just fine if
the frame buffer is not used, but it has to be done in kernel when
using a frame buffer.


-- 
Michael Marineau
[EMAIL PROTECTED]
Oregon State University

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to