Re: [PATCH 2/2] ACPI: update Kconfig help texts (no functional changes)

[Len Brown <lenb@kernel.org>]

>  drivers/acpi/Kconfig |  148 +++++++++++++++++++++++++++++---------------------
>  1 files changed, 86 insertions(+), 62 deletions(-)
> 
> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
> index 8a851d0..7a3f52c 100644
> --- a/drivers/acpi/Kconfig
> +++ b/drivers/acpi/Kconfig
> @@ -11,9 +11,9 @@ menuconfig ACPI
>  	select PNP
>  	select CPU_IDLE
>  	default y
> -	---help---
> +	help
>  	  Advanced Configuration and Power Interface (ACPI) support for 
> -	  Linux requires an ACPI compliant platform (hardware/firmware),
> +	  Linux requires an ACPI-compliant platform (hardware/firmware),
>  	  and assumes the presence of OS-directed configuration and power
>  	  management (OSPM) software.  This option will enlarge your 
>  	  kernel by about 70K.
> @@ -23,20 +23,18 @@ menuconfig ACPI
>  	  the Plug-and-Play BIOS specification (PnP BIOS), the 
>  	  MultiProcessor Specification (MPS), and the Advanced Power 
>  	  Management (APM) specification.  If both ACPI and APM support 
> -	  are configured, whichever is loaded first shall be used.
> +	  are configured, ACPI is used.
>  
> -	  The ACPI SourceForge project contains the latest source code, 
> -	  documentation, tools, mailing list subscription, and other 
> -	  information.  This project is available at:
> -	  <http://sourceforge.net/projects/acpi>
> +	  The mailing list for ACPI in Linux is here:
> +	  <http://vger.kernel.org/vger-lists.html#linux-acpi>

per MAINTAINERS, the project home page is 
http://www.lesswatts.org/projects/acpi/

Probably sufficient to just point there...

>  	  Linux support for ACPI is based on Intel Corporation's ACPI
>  	  Component Architecture (ACPI CA).  For more information see:
>  	  <http://developer.intel.com/technology/iapc/acpi>

ACPICA's home page moved here:
http://acpica.org/

> -	  ACPI is an open industry specification co-developed by Compaq, 
> -	  Intel, Microsoft, Phoenix, and Toshiba.  The specification is 
> -	  available at:
> +	  ACPI is an open industry specification co-developed by
> +	  Hewlett-Packard, Intel, Microsoft, Phoenix, and Toshiba.
> +	  The specification is available at:
>  	  <http://www.acpi.info>
...

>  config ACPI_VIDEO
>  	tristate "Video"
> @@ -141,38 +149,48 @@ config ACPI_VIDEO
>  	depends on INPUT
>  	select THERMAL
>  	help
> -	  This driver implement the ACPI Extensions For Display Adapters
> +	  This driver implements the ACPI Extensions For Display Adapters
>  	  for integrated graphics devices on motherboard, as specified in
> -	  ACPI 2.0 Specification, Appendix B, allowing to perform some basic
> -	  control like defining the video POST device, retrieving EDID information
> -	  or to setup a video output, etc.
> -	  Note that this is an ref. implementation only.  It may or may not work
> -	  for your integrated video device.
> +	  ACPI 2.0 Specification, Appendix B.  This supports basic operations
> +	  such as defining the video POST device, retrieving EDID information,
> +	  and setting up a video output.
> +
> +	  Note that this is an reference implementation only.  It may not
> +	  work for your integrated video device.

we can probably strike the two lines above entirely.
It should work for everything that implements the spec,
and it is a bug if it does not.

> +
> +	  To compile this driver as a module, choose M here:
> +	  the module will be called video.
>  

thanks,
Len Brown, Intel Open Source Technology Center