[oi-dev] [developer] acpi webrevs: 1, 2, 3
yuri.pankov at gmail.com
Wed Nov 30 22:20:16 UTC 2011
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... :-)
More information about the oi-dev