kern/127581: [patch] [acpi_sony] Add support for more Sony features

[Gavin Atkinson]

>Number:         127581
>Category:       kern
>Synopsis:       [patch] [acpi_sony] Add support for more Sony features
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Sep 23 15:50:02 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator:     Gavin Atkinson
>Release:        FreeBSD 8.0-CURRENT amd64
>Organization:
>Environment:
System: FreeBSD blade32.york.ac.uk 8.0-CURRENT FreeBSD 8.0-CURRENT #1: Mon Aug 18 14:12:25 BST 2008 root at blade32.york.ac.uk:/usr/obj/usr/src/sys/B32 amd64
>Description:
	Add support for a few more Sony-specific ACPI features (default
display brightness, wired LAN power and bass gain), and update the description
of one previously unknown feature.  While here, expand on a comment and
remove two unused defines.
>How-To-Repeat:
	N/A
>Fix:

--- sony_acpi_5001.diff begins here ---
Index: src/sys/dev/acpi_support/acpi_sony.c
===================================================================
RCS file: /home/ncvs/src/sys/dev/acpi_support/acpi_sony.c,v
retrieving revision 1.11
diff -u -r1.11 acpi_sony.c
--- src/sys/dev/acpi_support/acpi_sony.c	20 May 2008 06:32:07 -0000	1.11
+++ src/sys/dev/acpi_support/acpi_sony.c	23 Sep 2008 15:42:20 -0000
@@ -40,14 +40,21 @@
 #define _COMPONENT	ACPI_OEM
 ACPI_MODULE_NAME("Sony")
 
-#define ACPI_SONY_GET_BRIGHTNESS "GBRT"
-#define ACPI_SONY_SET_BRIGHTNESS "SBRT"
 #define ACPI_SONY_GET_PID "GPID"
 
 /*
  * SNY5001
+ *   This is the ACPI handle for the "Sony Notebook Control" driver under
+ *   Windows.
+ *   It provides several methods within the ACPI namespace, including:
  *  [GS]BRT [GS]PBR [GS]CTR [GS]PCR [GS]CMI [CDPW GCDP]? GWDP PWAK PWRN 
  *
+ * SNY6001
+ *   This is the ACPI handle for the "Sony Programmable I/O" driver under
+ *   Windows.
+ *   It is not yet supported by this driver, but provides control over the
+ *   power to the bluetooth, built-in camera and HSDPA modem devices in some
+ *   laptops, and also allows some control of the fan speed.
  */
 
 struct acpi_sony_softc {
@@ -61,14 +68,17 @@
 	char *comment;
 } acpi_sony_oids[] = {
 	{ "brightness", "GBRT", "SBRT", "Display Brightness"},
-	{ "ctr", "GCTR", "SCTR", "??"},
+	{ "brightness_default", "GPBR", "SPBR", "Default Display Brightness"},
+	{ "contrast", "GCTR", "SCTR", "Display Contrast"},
+	{ "bass_gain", "GMGB", "SMGB", "Multimedia Bass Gain"},
 	{ "pcr", "GPCR", "SPCR", "???"},
 #if 0
-	{ "cmi", "GCMI", "SCMI", "????"},
+	{ "cmi", "GCMI", "SCMI", "???"},
 #endif
-	{ "wdp", "GWDP", NULL, "?????"},
+	{ "wdp", "GWDP", NULL, "???"},
 	{ "cdp", "GCDP", "CDPW", "CD Power"},  /*shares [\GL03]&0x8 flag*/
 	{ "azp", "GAZP", "AZPW", "Audio Power"}, 
+	{ "lnp", "GLNP", "LNPW", "LAN Power"},
 	{ NULL, NULL, NULL }
 };
 
--- sony_acpi_5001.diff ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted: