Daniel Holmes wrote:
> > + printf("acpi_cpu: throttling enabled, %d steps from 100%% to %d.%d%%, "
> > + "currently %d.%d%%\n"
>
> Personally, rather than 'enabled', how about 'available'? Using the
> word enabled might give some newbies fits when they try to figure it
> out what it means. It sounds like the throttling is already turned on.
I made the same comment, originally; I think that the problem is
that "enabled" is more likely to have meaning for non-English
speakers than "available".
You missed the second "printf", which printed out the current
stepping level; I did too, when I first read it.
I think that changing the order from "100% to 10%" to "10% to 100%"
will, if people ignore the second printed line, imply that there
was a transition from 10% to 100%, rather than the reverse (that
was my response to the patch).
-- Terry
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
