[oi-dev] [developer] acpi webrevs: 1, 2, 3

[Yuri Pankov]

On Wed, Nov 30, 2011 at 04:22:37PM -0500, Gordon Ross wrote:
> On Tue, Nov 29, 2011 at 11:37 PM, Gordon Ross <gordon.w.ross at gmail.com> wrote:
> > OK, Yuri helped get his regressions fixed.
> >
> > I've updated these webrevs:
> >   http://www.yalms.org/cr/acpi-gwr1/
> >   http://www.yalms.org/cr/acpi-gwr2/
> >   http://www.yalms.org/cr/acpi-gwr3/
> >
> > There's now also an "all in one" webrev:
> >   http://www.yalms.org/cr/acpi-11/
> >
> > I have a build running.  When it finishes,
> > I'll update the test bits here:
> >  http://yalms.org/test/
> >
> > Any additional test results are appreciated.
> > If you see anything unusual, please note
> > whether it's a regression vs oi151a
> >
> > Thanks again to everyone who provided
> > test results and feedback.  Almost ready...
>
> After the most recent problem investigations, it
> became apparent that I did not need some of the
> changes in EC initialization, so I backed those out.
> In particular, EC initialization once again happens
> after AcpiEnableSubsystem, same as before.
> I kept the EC code organization improvements
> as those will help with any further work on this.
> 
> There's still more work needed for systems,
> but no regressions that I'm aware of.
> 
> Please have another look at the webrevs.
> (changed are: acpi-gwr3, acpi-11)
> 
> I believe this work is  ready to integrate.

LGTM (at least, testing-wise, if not code-wise... :-)


Thanks,
Yuri