Messages in this thread | | | Date | Mon, 11 Jul 2005 20:56:04 +0200 | From | Pavel Machek <> | Subject | Re: [PATCH][help?] Radeonfb acpi resume |
| |
Hi!
> >>Aww crap, thunderbird screwed up the white space... > >> > >>A usable version of the patch is attached, or here is a link: > >>http://dev.gentoo.org/~marineam/files/patch-radeonfb-2.6.12 > > > > > > Wrong indentation in acpi_vgapost; I remember there was better patch > > to fix this out there. > Ok, I'll go through and fix any coding style problems. I've only seen > older versions of this same patch, but if there is a better way I'd love > to hear it. I'll google around a little more just in case.
It *was* version of the same patch, but it had codingstyle fixed, IIRC.
> > Anyway, are you sure machine you have can't be fixed by any methods > > listed in Doc*/power/video.txt? I guess they are preferable to > > acpi_vgapost... > Actually, this is one of the metholds listed in video.txt. Take a look > at #7 ;-). I just tried acpi_sleep=s3_bios to see what that does, but > just caused an instant reboot on resume. The only other solutions that > works is to disable the frame buffer and use X or some other app to do > the job as listed in #5 and #6, but something in kernel like this patch > is required to be able to use the framebuffer.
I'd say that disabling framebuffer and going #5 or #6 is still prefered, but given nice patch, I'll probably accept it. Oh, and do note that (7) is listed near just one notebook.
Pavel -- teflon -- maybe it is a trademark, but it should not be. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |