Accelerometer non functional (acer-wmi) in Acer W500/501

Bug #1055433 reported by Sean DS
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Medium
Tim Gardner
Quantal
Fix Released
Medium
Tim Gardner

Bug Description

The accelerometer is not detected as an acpi event. There is a patch (attached) which when applied to the acer-wmi.c source, allows the accelerometer to work.

It would be nice if the patch could be added upstream so us early ubuntu tab adopters can rotate at our leisure!

I found the patch in this link: http://www.linuxforums.org/forum/hardware-peripherals/189360-accelerometer-help-needed-arch-linux-kernel-3-4-a.html

I did not make the patch myself.
---
AlsaVersion: Advanced Linux Sound Architecture Driver Version 1.0.24.
ApportVersion: 2.0.1-0ubuntu12
Architecture: i386
ArecordDevices:
 **** List of CAPTURE Hardware Devices ****
 card 1: SB [HDA ATI SB], device 0: ALC269VB Analog [ALC269VB Analog]
   Subdevices: 1/1
   Subdevice #0: subdevice #0
AudioDevicesInUse:
 USER PID ACCESS COMMAND
 /dev/snd/controlC1: sean 1594 F.... pulseaudio
 /dev/snd/controlC0: sean 1594 F.... pulseaudio
Card0.Amixer.info:
 Card hw:0 'Generic'/'HD-Audio Generic at 0x90144000 irq 40'
   Mixer name : 'ATI R6xx HDMI'
   Components : 'HDA:1002aa01,00aa0100,00100200'
   Controls : 6
   Simple ctrls : 1
Card0.Amixer.values:
 Simple mixer control 'IEC958',0
   Capabilities: pswitch pswitch-joined penum
   Playback channels: Mono
   Mono: Playback [on]
Card1.Amixer.info:
 Card hw:1 'SB'/'HDA ATI SB at 0x90140000 irq 16'
   Mixer name : 'Realtek ALC269VB'
   Components : 'HDA:10ec0269,10250577,00100100'
   Controls : 14
   Simple ctrls : 8
DistroRelease: Ubuntu 12.04
HibernationDevice: RESUME=UUID=007e4666-f036-4990-8da8-b9fb9c949999
MachineType: acer ICONIA Tab W501
NonfreeKernelModules: fglrx
Package: linux (not installed)
ProcEnviron:
 LANGUAGE=en_GB:en
 TERM=xterm
 PATH=(custom, no user)
 LANG=en_GB.UTF-8
 SHELL=/bin/bash
ProcFB: 0 VESA VGA
ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.2.0-29-generic root=UUID=47430b6c-e9cd-400a-9b83-004bd4f6b348 ro quiet splash vt.handoff=7
ProcVersionSignature: Ubuntu 3.2.0-29.46-generic 3.2.24
RelatedPackageVersions:
 linux-restricted-modules-3.2.0-29-generic N/A
 linux-backports-modules-3.2.0-29-generic N/A
 linux-firmware 1.79
Tags: precise
Uname: Linux 3.2.0-29-generic i686
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo
dmi.bios.date: 07/08/2011
dmi.bios.vendor: Insyde Corp.
dmi.bios.version: V1.13
dmi.board.asset.tag: Base Board Asset Tag
dmi.board.name: WT1
dmi.board.vendor: acer
dmi.board.version: Base Board Version
dmi.chassis.type: 10
dmi.chassis.vendor: Chassis Manufacturer
dmi.chassis.version: Chassis Version
dmi.modalias: dmi:bvnInsydeCorp.:bvrV1.13:bd07/08/2011:svnacer:pnICONIATabW501:pvrV1.13:rvnacer:rnWT1:rvrBaseBoardVersion:cvnChassisManufacturer:ct10:cvrChassisVersion:
dmi.product.name: ICONIA Tab W501
dmi.product.version: V1.13
dmi.sys.vendor: acer

Revision history for this message
Sean DS (se4n-1) wrote :

Here is the patch.

affects: grub2 (Ubuntu) → acerhk (Ubuntu)
tags: added: acer-wmi kernel-bug
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "acer-wmi patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Benjamin Kerensa (bkerensa) wrote :

This package does not contain acer-wmi.c so that patch cannot be applied.

tags: removed: patch
Revision history for this message
Sean DS (se4n-1) wrote :

Changed the bug report to linux. acer-wmi.c is in drivers/platform/x86/

affects: acerhk (Ubuntu) → linux (Ubuntu)
Revision history for this message
Brad Figg (brad-figg) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. From a terminal window please run:

apport-collect 1055433

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Revision history for this message
Sean DS (se4n-1) wrote : AcpiTables.txt

apport information

tags: added: apport-collected precise
description: updated
Revision history for this message
Sean DS (se4n-1) wrote : AlsaDevices.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : AplayDevices.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : BootDmesg.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : CRDA.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : Card0.Codecs.codec.0.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : Card1.Amixer.values.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : Card1.Codecs.codec.0.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : CurrentDmesg.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : IwConfig.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : Lspci.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : Lsusb.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : PciMultimedia.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : ProcCpuinfo.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : ProcInterrupts.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : ProcModules.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : PulseList.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : RfKill.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : UdevDb.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : UdevLog.txt

apport information

Revision history for this message
Sean DS (se4n-1) wrote : WifiSyslog.txt

apport information

Changed in linux (Ubuntu):
status: Incomplete → Confirmed
Sean DS (se4n-1)
tags: added: patch
Revision history for this message
Joseph Salisbury (jsalisbury) wrote :

Can you provide some information on the status of the patch with regards to getting it merged upstream? Has it been sent upstream, what sort of feedback has it received, is it getting applied to a subsystem maintainer's tree, etc?

People affected by this bug are probably wondering why the kernel team doesn't just apply the patch and fix it. The reason is that the kernel team is reluctant (not opposed) to apply any patch to a stable kernel that is not from upstream. Applying patches that don't come from upstream add greatly to the support of the kernel as other upstream patches may touch the same area as the non-upstream patch and may prevent them from applying cleanly.

To submit your patch, send your patch with the detailed description/changelog and your Signoff (ending with Signed-off-by: your name <email>), to the emails listed from ./scripts/get_maintainer.pl drivers/SUBSYSTEM-DETAILS (the get_maintainer.pl is from the kernel sources). Once you have sent the patch upstream and it's accepted, please drop a note here so that we can cherry-pick/include the patch into Ubuntu kernel.

Changed in linux (Ubuntu):
importance: Undecided → Medium
status: Confirmed → Triaged
Revision history for this message
Sean DS (se4n-1) wrote :

All I know is the information from:
http://www.linuxforums.org/forum/hardware-peripherals/189360-accelerometer-help-needed-arch-linux-kernel-3-4-a.html
and
blog.gmane.org/gmane.linux.drivers.platform.x86.devel/3350

The patch originated from a kernel developer but as of 3.5.rc6 it still has not been committed. I'll see what its status upstream is and drop a not here when I hear back.

Revision history for this message
Sean DS (se4n-1) wrote :

The patch has made its way into the linux 3.6rc7 kernel and therefore has been merged upstream.

Thanks, I learnt a lot about kernel development today.

Revision history for this message
Sean DS (se4n-1) wrote :

not sure what kernel version we're aiming for in Quantal, so will mark as fix committed.

Changed in linux (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Tim Gardner (timg-tpi) wrote :

ACER: Add support for accelerometer sensor
(cherry picked from commit 1eb3fe1d3b6b9bf6045eb12f0c3ac12569169870)

Changed in linux (Ubuntu Quantal):
assignee: nobody → Tim Gardner (timg-tpi)
Revision history for this message
Tim Gardner (timg-tpi) wrote :

ACER: Fix Smatch double-free issue
(cherry picked from commit 1bfaf1d5bcfb2ae0b7b90c5aed86909ee4f3f099)

The patch 6ae3a0876185: "ACER: Add support for accelerometer sensor"
    from Jun 1, 2012, leads to the following Smatch warning:
    drivers/platform/x86/acer-wmi.c:1886 acer_wmi_accel_destroy()
             error: don't call input_free_device() after input_unregister_device()

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package linux - 3.5.0-16.25

---------------
linux (3.5.0-16.25) quantal-proposed; urgency=low

  [ <email address hidden> ]

  * SAUCE: input: Cypress PS/2 Trackpad fix multi-source, double-click
    - LP: #1055788

  [ Tim Gardner ]

  * [Config] revert '[Config] enable CONFIG_X86_X32=y'
    - LP: #1041883

  [ Upstream Kernel Changes ]

  * vmwgfx: corruption in vmw_event_fence_action_create()
  * drm/nvd0/disp: hopefully fix selection of 6/8bpc mode on DP outputs
    - LP: #1058088
  * drm/nv50-/gpio: initialise to vbios defaults during init
    - LP: #1058088
  * igb: A fix to VF TX rate limit
    - LP: #1058188
  * igb: Add switch case for supported hardware to igb_ptp_remove.
    - LP: #1058188
  * igb: Support the get_ts_info ethtool method.
    - LP: #1058188
  * igb: Streamline RSS queue and queue pairing assignment logic.
    - LP: #1058188
  * igb: Update firmware info output
    - LP: #1058188
  * igb: Version bump
    - LP: #1058188
  * igb: reset PHY in the link_up process to recover PHY setting after
    power down.
    - LP: #1058188
  * igb: Fix for failure to init on some 82576 devices.
    - LP: #1058188
  * igb: correct hardware type (i210/i211) check in igb_loopback_test()
    - LP: #1058188
  * igb: don't break user visible strings over multiple lines in
    igb_ethtool.c
    - LP: #1058188
  * igb: add delay to allow igb loopback test to succeed on 8086:10c9
    - LP: #1058188
  * igb: fix panic while dumping packets on Tx hang with IOMMU
    - LP: #1058188
  * igb: Fix register defines for all non-82575 hardware
    - LP: #1058188
  * e1000e: use more informative logging macros when netdev not yet
    registered
    - LP: #1058219
  * e1000e: Cleanup code logic in e1000_check_for_serdes_link_82571()
    - LP: #1058219
  * e1000e: Program the correct register for ITR when using MSI-X.
    - LP: #1058219
  * e1000e: advertise transmit time stamping
    - LP: #1058219
  * e1000e: 82571 Tx Data Corruption during Tx hang recovery
    - LP: #1058219
  * e1000e: fix panic while dumping packets on Tx hang with IOMMU
    - LP: #1058219
  * e1000: Combining Bitwise OR in one expression.
    - LP: #1058221
  * e1000: advertise transmit time stamping
    - LP: #1058221
  * e1000: Small packets may get corrupted during padding by HW
    - LP: #1058221
  * sched: Fix migration thread runtime bogosity
    - LP: #1057593
  * ACER: Add support for accelerometer sensor
    - LP: #1055433
  * ACER: Fix Smatch double-free issue
    - LP: #1055433

  [ Wen-chien Jesse Sung ]

  * SAUCE: HID: ntrig: change default value of logical/physical
    width/height to 1
    - LP: #1044248
 -- Leann Ogasawara <email address hidden> Fri, 28 Sep 2012 14:07:41 -0700

Changed in linux (Ubuntu Quantal):
status: Fix Committed → Fix Released
Revision history for this message
Adam Conrad (adconrad) wrote : Update Released

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.