On 2015/3/11 14:23, Ingo Molnar wrote: > > * Li, Aubrey <aubrey...@linux.intel.com> wrote: > >> On a platform in ACPI Hardware-reduced mode, the legacy PIC and PIT >> may not be initialized even though they may be present in silicon. >> Touching these legacy components causes unexpected result on system. > > s/causes unexpected result on system/ > causes unexpected results on the system > >> >> On Bay Trail-T(ASUS-T100) platform, touching these legacy components > > s/On Bay Trail-T(ASUS-T100) platform/ > On the Bay Trail-T(ASUS-T100) platform > > Ok, my final bike shed painting job would be to move the > 'acpi_gbl_reduced_hardware' flag check inside acpi_reduced_hw_init(): > that makes it nicely self-sustained and all in a single place. > > With that fixed it looks good to me.
Thanks, I'll cook v3 soon, :-) > > Should I merge it for v4.0 upstream merge, in tip:x86/urgent? > > The 'touches hardware in unexpected ways' aspect qualifies it for > urgent treatment IMO. Currently the unexpected result is that it blocks a low power idle during suspend. I'm not sure if it can be for x86/urgent. But it would be nice if it can be a bug-fix in v4.0. Thanks, -Aubrey > > Thanks, > > Ingo > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/