Lv Zheng | 20 Jun 2013 02:01
Picon
Favicon

[PATCH 0/3] ACPI: Documentation udpates

This patch set adds 2 ACPI documentation files with MAINTAINERS updated.

Lv Zheng (3):
  ACPI: Update MAINTAINERS file to include Documentation/acpi
  ACPI: Add sysfs ABI documentation
  ACPI: Add ACPI namespace documentation

 Documentation/ABI/testing/sysfs-bus-acpi    |   58 ++++
 Documentation/ABI/testing/sysfs-devices-sun |    2 +-
 Documentation/acpi/namespace.txt            |  395 +++++++++++++++++++++++++++
 MAINTAINERS                                 |    2 +
 4 files changed, 456 insertions(+), 1 deletion(-)
 create mode 100644 Documentation/ABI/testing/sysfs-bus-acpi
 create mode 100644 Documentation/acpi/namespace.txt

--

-- 
1.7.10

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Naveen N. Rao | 19 Jun 2013 19:57
Picon

[PATCH v2 1/2] mce: acpi/apei: Honour Firmware First for MCA banks listed in APEI HEST CMC

The Corrected Machine Check structure (CMC) in HEST has a flag which can be
set by the firmware to indicate to the OS that it prefers to process the
corrected error events first. In this scenario, the OS is expected to not
monitor for corrected errors (through CMCI/polling). Instead, the firmware
notifies the OS on corrected error events through GHES.

Linux already has support for GHES. This patch adds support for parsing CMC
structure and to disable CMCI/polling if the firmware first flag is set.

Further, the list of machine check bank structures at the end of CMC is used
to determine which MCA banks function in FF mode, so that we continue to
monitor error events on the other banks.

- Naveen

--
Changes:
- Incorporated comments from Boris and Tony from the previous thread at
  http://thread.gmane.org/gmane.linux.acpi.devel/60802
- Added patch to disable firmware first mode through a boot option.

Signed-off-by: Naveen N. Rao <naveen.n.rao <at> linux.vnet.ibm.com>
---
 arch/x86/include/asm/mce.h                |    3 ++
 arch/x86/kernel/cpu/mcheck/mce-internal.h |    3 ++
 arch/x86/kernel/cpu/mcheck/mce.c          |   25 ++++++++++++++++++
 arch/x86/kernel/cpu/mcheck/mce_intel.c    |   40 +++++++++++++++++++++++------
 drivers/acpi/apei/hest.c                  |   36 ++++++++++++++++++++++++++
 5 files changed, 99 insertions(+), 8 deletions(-)

(Continue reading)

John B. Wyatt IV | 19 Jun 2013 05:17
Picon

[PATCH] Acpi: acpica: acmacros: fixed a semicolon formatting issue.

From: "John B. Wyatt IV" <sageofredondo <at> gmail.com>

Formatting patch: fixes all "space required after that ';'" errors in
acmarcos.h.

Please note this only fixes 12 out of 64 errors as reported by
./scripts/checkpatch.pl
---
 drivers/acpi/acpica/acmacros.h | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/drivers/acpi/acpica/acmacros.h b/drivers/acpi/acpica/acmacros.h
index 53666bd..b916eed 100644
--- a/drivers/acpi/acpica/acmacros.h
+++ b/drivers/acpi/acpica/acmacros.h
 <at>  <at>  -93,15 +93,15  <at>  <at> 
 /* 16-bit source, 16/32/64 destination */

 #define ACPI_MOVE_16_TO_16(d, s)        {((  u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[1];\
-			  ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[0];}
+			  ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[0]; }

 #define ACPI_MOVE_16_TO_32(d, s)        {(*(u32 *)(void *)(d))=0;\
 					  ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\
-					  ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0];}
+					  ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0]; }

 #define ACPI_MOVE_16_TO_64(d, s)        {(*(u64 *)(void *)(d))=0;\
 							   ((u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[1];\
-							   ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0];}
(Continue reading)

Mika Westerberg | 18 Jun 2013 15:51
Picon

[PATCH] ACPI / LPSS: override SDIO device's private register space size

The SDIO device in Lynxpoint has its LTR registers allocated for a WiFi
device (a child of the SDIO device) instead in ACPI namespace even though
they physically belong to the SDIO device. In order to be able to access
the LTR registers from the ACPI LPSS driver for diagnostic purposes we need
to enlarge size of this private register space.

Work around this by adding a possibility to override size of the private
register space in the ACPI LPSS driver and set correct size for the SDIO
device.

Signed-off-by: Mika Westerberg <mika.westerberg <at> linux.intel.com>
---
Applies on top of https://patchwork.kernel.org/patch/2742521/ (ACPI / LPSS:
Power up LPSS devices during enumeration)

 drivers/acpi/acpi_lpss.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
index a14d997..539aeae 100644
--- a/drivers/acpi/acpi_lpss.c
+++ b/drivers/acpi/acpi_lpss.c
 <at>  <at>  -44,6 +44,7  <at>  <at>  struct lpss_device_desc {
 	const char *clkdev_name;
 	bool ltr_required;
 	unsigned int prv_offset;
+	size_t prv_size_override;
 	bool clk_gate;
 	struct lpss_shared_clock *shared_clock;
 };
(Continue reading)

Rafael J. Wysocki | 18 Jun 2013 15:41
Picon
Gravatar

[PATCH] ACPI / LPSS: Power up LPSS devices during enumeration

From: Rafael J. Wysocki <rafael.j.wysocki <at> intel.com>

Commit 7cd8407 (ACPI / PM: Do not execute _PS0 for devices without
_PSC during initialization) introduced a regression on some systems
with Intel Low-Power Subsystem (LPSS) where some devices need to be
powered up during initialization, but their device objects in the
ACPI namespace have _PS0 and _PS3 only (without _PSC or power
resources).

To work around this problem, make the ACPI LPSS driver power up
devices it knows about by using a new helper function
acpi_device_fix_up_power() that does all of the necessary
sanity checks and calls acpi_dev_pm_explicit_set() to put the
device into D0.

Reported-and-tested-by: Mika Westerberg <mika.westerberg <at> linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki <at> intel.com>
---
 drivers/acpi/acpi_lpss.c |   21 +++++++++++++++------
 drivers/acpi/device_pm.c |   20 ++++++++++++++++++++
 include/acpi/acpi_bus.h  |    1 +
 3 files changed, 36 insertions(+), 6 deletions(-)

Index: linux-pm/drivers/acpi/acpi_lpss.c
===================================================================
--- linux-pm.orig/drivers/acpi/acpi_lpss.c
+++ linux-pm/drivers/acpi/acpi_lpss.c
 <at>  <at>  -280,15 +280,24  <at>  <at>  static int acpi_lpss_create_device(struc
 	if (dev_desc->clk_required) {
 		ret = register_device_clock(adev, pdata);
(Continue reading)

Heikki Krogerus | 17 Jun 2013 12:25
Picon

[PATCH] ACPI / LPSS: mask the UART TX completion interrupt

Intel LPSS provides an extra TX byte counter and an extra TX
completion interrupt for some of it's bus controllers. This
masks out the extra UART interrupt as there is no use for it.

Otherwise, if the firmware does not mask the interrupt and
the driver does not clear it, it may cause interrupt flood
that freezes the board.

Signed-off-by: Heikki Krogerus <heikki.krogerus <at> linux.intel.com>
---
 drivers/acpi/acpi_lpss.c | 30 ++++++++++++++++++++++++++++--
 1 file changed, 28 insertions(+), 2 deletions(-)

diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
index f6d7605..bd9867f 100644
--- a/drivers/acpi/acpi_lpss.c
+++ b/drivers/acpi/acpi_lpss.c
 <at>  <at>  -32,6 +32,8  <at>  <at>  ACPI_MODULE_NAME("acpi_lpss");
 #define LPSS_GENERAL_LTR_MODE_SW	BIT(2)
 #define LPSS_SW_LTR			0x10
 #define LPSS_AUTO_LTR			0x14
+#define LPSS_TX_INT			0x20
+#define LPSS_TX_INT_MASK		BIT(1)

 struct lpss_shared_clock {
 	const char *name;
 <at>  <at>  -39,6 +41,8  <at>  <at>  struct lpss_shared_clock {
 	struct clk *clk;
 };

(Continue reading)

Aaron Lu | 17 Jun 2013 03:01
Picon
Favicon

[PATCH] acpi: video: no automatic brightness changes by firmware

Starting from win8, MS backlight control driver will set bit 2 of the
parameter of control method _DOS, to inform firmware it should not
perform any automatic brightness changes. This mostly affects hotkey
notification deliver - if we do not set this bit, on hotkey press,
firmware may choose to adjust brightness level instead of sending out
notification and doing nothing.

So this patch sets bit 2 when calling _DOS so that GUIs can show the
notification window on hotkey press.

The MS document on win8 backlight control is here:
http://msdn.microsoft.com/en-US/library/windows/hardware/jj159305

This patch solves problem for ASUS N56VZ and ASUS N56VJ, both for
notification delivery and for brightness control.

Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=52951
Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=56711
Reported-and-tested-by: Micael Dias <kam1kaz3 <at> gmail.com>
Reported-and-tested-by: Dan Garton <dan.garton <at> gmail.com>
Reported-and-tested-by: Bob Ziuchkovski <bob.ziuchkovski <at> gmail.com>
Signed-off-by: Aaron Lu <aaron.lu <at> intel.com>
---
 drivers/acpi/video.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
index b8299f2..f8fa541 100644
--- a/drivers/acpi/video.c
+++ b/drivers/acpi/video.c
(Continue reading)

Larry Baker | 15 Jun 2013 00:40
Picon
Favicon

Re: Wishlist: Disable C6 in intel_idle for Model 44 processors

Rob,

I think I found the description of the "Driver Impedance" fix in the IBM uEFI firmware at
http://www-947.ibm.com/support/entry/portal/docdisplay?lndocid=migr-5091950.  I'm puzzled by
the Workaround section.  I still can't tell from this if the "Driver Impedance" setting is or is not ignored
by intel_idle.  Anyway, I hope the hardware boys can say if intel_idle should force this "Driver
Impedance" setting on these processors (assuming that is possible) when it wants to use C states.

Larry Baker
US Geological Survey
650-329-5608
baker <at> usgs.gov

On 14 Jun 2013, at 2:39 PM, Rob E Russell wrote:

> Larry, 
> 
> The fix from an IBM uEFI standpoint was to include an option in system settings called "Driver Impedance"
in uEFI v1.16.  If the user wants C-states to work, then driver impedance should also be enabled to avoid
this package C6 transition issue.   If the user does not care about C-states, then on Linux, the intel_idle
driver should be disabled.   C-states are disabled by default in uEFI system settings.
> 
> Regards, 
> Rob Russell
> Technical Sales Support, IBM System x, US Federal Government
> Phone: (720) 396-2235, TL 938-2235, Cell: 919-389-4874
> <Mail Attachment.jpeg>	Phone: 1-720-396-2235 | Phone: 1-919-651-1294 | Mobile: 1-919-389-4874
> E-mail: robr <at> us.ibm.com	
> <Mail Attachment.gif>
> 
(Continue reading)

Thomas Koch | 14 Jun 2013 18:21
Picon

[PATCH] ACPI: Add missing Thinkpad models to OSI(Linux) white list


acpi_osi=Linux helps the mute button work properly by sending Linux a
mute key press.

Add missing Thinkpad models X61[st], X200[st], R400, R500, W500, W700,
X300, X301 to the list.

Signed-off-by: Thomas Koch <linrunner <at> gmx.net>
---
 drivers/acpi/blacklist.c |   58
+++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 57 insertions(+), 1 deletion(-)

diff --git a/drivers/acpi/blacklist.c b/drivers/acpi/blacklist.c
index cb96296..65dfe0e 100644
--- a/drivers/acpi/blacklist.c
+++ b/drivers/acpi/blacklist.c
 <at>  <at>  -305,7 +305,7  <at>  <at>  static struct dmi_system_id acpi_osi_dmi_table[]
__initdata = {
 	},
 	{
 	.callback = dmi_enable_osi_linux,
-	.ident = "Lenovo ThinkPad X61",
+	.ident = "Lenovo ThinkPad X61[s][t]",
 	.matches = {
 		     DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
 		     DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X61"),
 <at>  <at>  -327,6 +327,62  <at>  <at>  static struct dmi_system_id acpi_osi_dmi_table[]
__initdata = {
 		     DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T500"),
(Continue reading)

Rafael J. Wysocki | 14 Jun 2013 00:29
Picon
Gravatar

[PATCH] PCI / ACPI / PM: Use correct power state strings in messages

From: Rafael J. Wysocki <rafael.j.wysocki <at> intel.com>

Make acpi_pci_set_power_state() print the name of the ACPI device
power state the device has been actually put into instead of printing
the name of the requested PCI device power state, which need not be
the same.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki <at> intel.com>
---

For 3.11.

Thanks,
Rafael

---
 drivers/pci/pci-acpi.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-pm/drivers/pci/pci-acpi.c
===================================================================
--- linux-pm.orig/drivers/pci/pci-acpi.c
+++ linux-pm/drivers/pci/pci-acpi.c
 <at>  <at>  -211,7 +211,7  <at>  <at>  static int acpi_pci_set_power_state(stru

 	if (!error)
 		dev_info(&dev->dev, "power state changed by ACPI to %s\n",
-			 pci_power_name(state));
+			 acpi_power_state_string(state_conv[state]));

(Continue reading)

Rafael J. Wysocki | 13 Jun 2013 20:50
Picon
Gravatar

[GIT PULL] ACPI fix for v3.10-rc6

Hi Linus,

Please pull from the git repository at

  git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git acpi-3.10-rc6

to receive one ACPI fix for v3.10-rc6 which is
commit 8c9b7a7b2fc2750af418ddc28e707c42e78aa0bf

  ACPI / video: Do not bind to device objects with a scan handler

on top of commit 317ddd256b9c24b0d78fa8018f80f1e495481a10

  Linux 3.10-rc5

This is an alternative fix for the regression introduced in 3.9 whose
previous fix had to be reverted right before 3.10-rc5, because it broke
one of the Tony's machines.  In this one the check is confined to the
ACPI video driver (which is the only one causing the problem to happen
in the first place) and the Tony's box shouldn't even notice it.

Thanks!

---------------

Rafael J. Wysocki (1):
      ACPI / video: Do not bind to device objects with a scan handler

---------------

(Continue reading)


Gmane