On Thu, Dec 18, 2008 at 3:04 PM, Corey Osgood <[email protected]>wrote:
> On Thu, Dec 18, 2008 at 2:56 PM, Corey Osgood <[email protected]>wrote: > >> On Thu, Dec 18, 2008 at 2:38 PM, Corey Osgood <[email protected]>wrote: >> >>> On Thu, Dec 18, 2008 at 2:28 PM, Myles Watson <[email protected]> wrote: >>> >>>> >>>> >>>> On Thu, Dec 18, 2008 at 12:24 PM, Corey Osgood >>>> <[email protected]>wrote: >>>> >>>>> On Thu, Dec 18, 2008 at 11:37 AM, Corey Osgood <[email protected] >>>>> > wrote: >>>>> >>>>>> On Thu, Dec 18, 2008 at 10:58 AM, ron minnich <[email protected]>wrote: >>>>>> >>>>>>> I like this idea a lot, I have no idea why we did not have that >>>>>>> enabled before ... I'm glad I'm on vacation and you all can fix it >>>>>>> :-) >>>>>> >>>>>> >>>>>> I'm all for it. CN700 should be all ready for the change. >>>>>> >>>>> >>>>> I retract that, I was thinking of v3. Attached patch fixes v2. >>>>> >>>> Index: src/northbridge/via/cn700/vgachip.h >>>> =================================================================== >>>> --- src/northbridge/via/cn700/vgachip.h (revision 3818) >>>> +++ src/northbridge/via/cn700/vgachip.h (working copy) >>>> @@ -1,6 +1,6 @@ >>>> /* >>>> * This file is part of the coreboot project. >>>> - * >>>> + *; >>>> >>>> I'm curious about this extra semicolon. Besides that: >>>> >>>> Acked-by: Myles Watson <[email protected]> >>> >>> >>> Ah, I was looking for that! I have a bad habit of hitting my touchpad >>> while typing and moving the cursor. Thanks, r3819 >>> >> >> I just got another implicit declaration in i82810, and i82801xx is clean. >> Someone ought to check i440bx, i830, and the other via chipsets, vt8623 will >> probably have the same ones cn700 did, but I've got to get back to work. >> > > Here's another patch to clear up a bunch more, not all of them by far. I'm > for the most part avoiding k8 and sticking with the simpler ones. > Acked-by: Myles Watson <[email protected]> Thanks, Myles
-- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
