On Tuesday, 25 of March 2008, Carlos Corbacho wrote:
> > +static int __init asus_suspend_order(const struct dmi_system_id *d)
> > +{
> > + printk(KERN_WARNING PREFIX "%s detected, "
> > + "acpi_new_pts_ordering is force enabled\n", d->ident);
> > + new_pts_ordering = true;
> > + return 0;
>
> Given all you're doing here is just changing the ordering,
> 'asus_suspend_order' doesn't strike me as the right name for this function
> since:
>
> 1) This clearly isn't the case for all Asus boards.
>
> 2) Perhaps something like 'new_suspend_order' would be a better name for
> this function (e.g. if others want to use DMI to do this for non-Asus
> boards).
Agreed.
Thanks,
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
