Re: IPMI

From: Rusty Russell (rusty@rustcorp.com.au)
Date: Tue Jan 14 2003 - 22:49:55 EST


In message <3E241ECD.6000108@mvista.com> you write:
> Does this work, or would you like more detail?
>
> config IPMI_HANDLER
> tristate 'IPMI top-level message handler'
> help
> This enables the central IPMI message handler, required for IPMI
> to work. Note that you must have this enabled to enable any
> other IPMI things. IPMI is a standard for managing sensors
> (temperature, voltage, etc.) in a system. If you don't know
> what it is, your system probably doesn't have it and you can
> ignore this option. See Documentation/IPMI.txt for more
> details on the driver.

I don't mean to nitpick, but I want to know whether I need it or not.

Now, if I read this correctly, CONFIG_IPMI would be a more appropriate
name. But changing the message at least would make it clearer:

        config IPMI_HANDLER
               tristate 'IPMI support'

Now, the message. How about this:
       help
          Intelligent Peripheral Management Interface (IPMI) is a
          standard for managing sensors (temperature, voltage, etc.)
          in a system. Most IA64 and x86 servers shipped since 2002
          have support for it. See Documentation/IPMI.txt for more
          details. Say "N" unless configuring for a recent x86 or
          IA64 machine.

Tweak details to suit: maybe IPMI goes furthur back, maybe it's also
on desktops, maybe it's so widespread that the suggested default
should be "Y" for all IA64 and x86.

But the "what to do if you know nothing" is important: people who know
what IPMI is don't need the help message.

Hope that clarifies?
Rusty.

--
  Anyone who quotes me in their sig is an idiot. -- Rusty Russell.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed Jan 15 2003 - 22:00:53 EST