[SAMSUNG N250P] acpi_video backlight doesn't work

Bug #1086921 reported by Martin Beltov
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Medium
Seth Forshee
Quantal
Fix Released
Undecided
Seth Forshee
Raring
Fix Released
Medium
Seth Forshee

Bug Description

== SRU Justification ==

Impact: The standard ACPI backlight interfaces on this machine fail to change the screen brightness. There is an alternate interface supported by the samsung-laptop module that does work, but since userspace prefers acpi_video the hotkeys and UI for adjusting screen brightness fail to work.

Fix: This problem is common enough on Samsung machines that samsung-laptop contains a dmi list of machines which need the acpi_video backlight disabled. Add this machine to that list.

Test Case: Using a driver with the quirk added results in working backlight controls. Tested on LP#1086921.

---

Hello, dear developers of Ubuntu! I want to ask something, that I think is relevant to the Ubuntu:
I have a small samsung netbook that requires a custom kernel module to be installed from a ppa (https://launchpad.net/~voria/+archive/ppa) so that fn keys work for backlight brightness adjustment. My question is: can this kernel module and script be installed in the main kernel, so that this annoyance can be avoided? From what I read this is an issue that has plagued samsung laptop owners for several years now. The installed samsung-tools and samsung-backlight fixes this issue.
I am willing to help if I can, I just want to have a truly out-of-the-box experience with Gnu/Linux. This problem is present in all distros (that's what I read from the Internet, I have only tried it on *buntu).
Thank you in advance.

ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: linux-image (not installed)
ProcVersionSignature: Ubuntu 3.2.0-34.53-generic 3.2.33
Uname: Linux 3.2.0-34-generic i686
AlsaVersion: Advanced Linux Sound Architecture Driver Version 1.0.24.
AplayDevices:
 **** List of PLAYBACK Hardware Devices ****
 card 0: Intel [HDA Intel], device 0: ALC269 Analog [ALC269 Analog]
   Subdevices: 1/1
   Subdevice #0: subdevice #0
ApportVersion: 2.0.1-0ubuntu15
Architecture: i386
ArecordDevices:
 **** List of CAPTURE Hardware Devices ****
 card 0: Intel [HDA Intel], device 0: ALC269 Analog [ALC269 Analog]
   Subdevices: 1/1
   Subdevice #0: subdevice #0
AudioDevicesInUse:
 USER PID ACCESS COMMAND
 /dev/snd/controlC0: martin 1564 F.... xfce4-volumed
                      martin 1571 F.... pulseaudio
Card0.Amixer.info:
 Card hw:0 'Intel'/'HDA Intel at 0xf0400000 irq 47'
   Mixer name : 'Realtek ALC269'
   Components : 'HDA:10ec0269,144dc08f,00100004'
   Controls : 18
   Simple ctrls : 10
Date: Wed Dec 5 20:07:50 2012
HibernationDevice: RESUME=UUID=1144cf36-8c2f-4124-943f-4684e409a192
InstallationMedia: Xubuntu 12.04.1 LTS "Precise Pangolin" - Release i386 (20120817.3)
MachineType: SAMSUNG ELECTRONICS CO., LTD. N250P
MarkForUpload: True
ProcEnviron:
 LANGUAGE=en_US:en
 TERM=xterm
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcFB: 0 inteldrmfb
ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.2.0-34-generic root=UUID=0379bcf6-d023-4729-b527-0d75ccfbe3e0 ro quiet splash vt.handoff=7
RelatedPackageVersions:
 linux-restricted-modules-3.2.0-34-generic N/A
 linux-backports-modules-3.2.0-34-generic N/A
 linux-firmware 1.79.1
RfKill:
 0: phy0: Wireless LAN
  Soft blocked: no
  Hard blocked: no
SourcePackage: linux
UpgradeStatus: No upgrade log present (probably fresh install)
dmi.bios.date: 12/09/2010
dmi.bios.vendor: Phoenix Technologies Ltd.
dmi.bios.version: 01HG.M011.20101209.RHU
dmi.board.asset.tag: SAMSUNG
dmi.board.name: N250P
dmi.board.vendor: SAMSUNG ELECTRONICS CO., LTD.
dmi.board.version: Not Applicable
dmi.chassis.asset.tag: No Asset Tag
dmi.chassis.type: 10
dmi.chassis.vendor: SAMSUNG ELECTRONICS CO., LTD.
dmi.chassis.version: N/A
dmi.modalias: dmi:bvnPhoenixTechnologiesLtd.:bvr01HG.M011.20101209.RHU:bd12/09/2010:svnSAMSUNGELECTRONICSCO.,LTD.:pnN250P:pvrNotApplicable:rvnSAMSUNGELECTRONICSCO.,LTD.:rnN250P:rvrNotApplicable:cvnSAMSUNGELECTRONICSCO.,LTD.:ct10:cvrN/A:
dmi.product.name: N250P
dmi.product.version: Not Applicable
dmi.sys.vendor: SAMSUNG ELECTRONICS CO., LTD.

Related branches

Revision history for this message
Martin Beltov (mbeltov) wrote :
Revision history for this message
Brad Figg (brad-figg) wrote : Status changed to Confirmed

This change was made by a bot.

Changed in linux (Ubuntu):
status: New → Confirmed
Revision history for this message
Joseph Salisbury (jsalisbury) wrote : Re: include samsung-backlight fix in main kernel

Would it be possible for you to test the latest upstream kernel? Refer to https://wiki.ubuntu.com/KernelMainlineBuilds . Please test the latest v3.7 kernel[0] (Not a kernel in the daily directory) and install both the linux-image and linux-image-extra .deb packages.

If this bug is fixed in the mainline kernel, please add the following tag 'kernel-fixed-upstream'.

If the mainline kernel does not fix this bug, please add the tag: 'kernel-bug-exists-upstream'.

If you are unable to test the mainline kernel, for example it will not boot, please add the tag: 'kernel-unable-to-test-upstream'.
Once testing of the upstream kernel is complete, please mark this bug as "Confirmed".

Thanks in advance.

[0] http://kernel.ubuntu.com/~kernel-ppa/mainline/v3.7-rc8-raring/

Changed in linux (Ubuntu):
importance: Undecided → Medium
status: Confirmed → Incomplete
Revision history for this message
Martin Beltov (mbeltov) wrote :

Should I uninstall the stuff from the ppa first ?

Revision history for this message
Seth Forshee (sforshee) wrote :

Yes, uninstall the stuff from the PPA for testing.

Revision history for this message
Martin Beltov (mbeltov) wrote :

Now running 3.7.0-030700rc8-generic, however that doesn't fix the issue

Changed in linux (Ubuntu):
status: Incomplete → Confirmed
tags: added: kernel-bug-exists-upstream
Revision history for this message
Seth Forshee (sforshee) wrote :

Can you now provide the following information, preferably when running the 3.7.0-030700rc8-generic kernel and definitely without the ppa packages installed? Please describe exactly what happens when you press the backlight Fn keys -- i.e., does nothing happen at all, or does the brightness indicator appear on the screen without the screen brightness actually changing, or something else? Please also provide the output of running 'ls /sys/class/backlight' in a terminal. Thanks!

Changed in linux (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
Martin Beltov (mbeltov) wrote :

When I press the backlight keys, I can see the OSD indicator changing, however the backlight brightness does not change. It always stays at the lowest value.
Output of ls /sys/class/backlight:
acpi_video0 intel_backlight

Revision history for this message
Seth Forshee (sforshee) wrote :

Okay, so Fn keys do work but the acpi_video0 backlight doesn't. That's not at all uncommon on Samsung laptops. But there isn't any samsung backlight.

Can you post the output of running lsmod in a terminal so we can verify that the samsung-laptop module is actually loaded?

Revision history for this message
Martin Beltov (mbeltov) wrote :

Attaching lsmod output

Revision history for this message
Seth Forshee (sforshee) wrote :

samsung-laptop is not loaded, although it should be. Just to verify there isn't some problem with the kernel, please run the following in a terminal.

  modprobe -nv dmi:bvnPhoenixTechnologiesLtd.:bvr01HG.M011.20101209.RHU:bd12/09/2010:svnSAMSUNGELECTRONICSCO.,LTD.:pnN250P:pvrNotApplicable:rvnSAMSUNGELECTRONICSCO.,LTD.:rnN250P:rvrNotApplicable:cvnSAMSUNGELECTRONICSCO.,LTD.:ct10:cvrN/A:

This should output something like 'insmod /lib/modules/.../samsung-laptop.ko'. Please verify this is the case. If so, then I suspect the packages you had installed may have put a blacklist on samsung-laptop and didn't remove it. To check for this run 'grep samsung /etc/modprobe.d/*'. If this finds any files containing a line that reads 'blacklist samsung-laptop' you can remove the blacklist by opening the affected file(s) as root and removing that line.

Please also try loading samsung-laptop by running 'sudo modprobe samsung-laptop'. Verify that it now appears in the lsmod output (if not please attach the output from dmesg). Then check whether or not the contents of /sys/class/backlight have changed. If there's now a "samsung" backlight present try adjusting the backlight with the Fn keys and see whether it works. If there's no samsung backlight even with samsung-laptop loaded, let me know and I can give you a test kernel that might help.

Thanks!

Revision history for this message
Martin Beltov (mbeltov) wrote :

output of : modprobe -nv dmi:bvnPhoenixTechnologiesLtd.:bvr01HG.M011.20101209.RHU:bd12/09/2010:svnSAMSUNGELECTRONICSCO.,LTD.:pnN250P:pvrNotApplicable:rvnSAMSUNGELECTRONICSCO.,LTD.:rnN250P:rvrNotApplicable:cvnSAMSUNGELECTRONICSCO.,LTD.:ct10:cvrN/A:

WARNING: Not loading blacklisted module samsung_laptop
FATAL: Module dmi:bvnPhoenixTechnologiesLtd.:bvr01HG.M011.20101209.RHU:bd12/09/2010:svnSAMSUNGELECTRONICSCO.,LTD.:pnN250P:pvrNotApplicable:rvnSAMSUNGELECTRONICSCO.,LTD.:rnN250P:rvrNotApplicable:cvnSAMSUNGELECTRONICSCO.,LTD.:ct10:cvrN/A: not found.

output of grep samsung /etc/modprobe.d/* :

/etc/modprobe.d/samsung-backlight.conf:blacklist samsung-laptop
/etc/modprobe.d/samsung-backlight.conf:options samsung-backlight use_sabi=0 levels=8 force=1

I have commented the blacklist. After I modprobed samsung-laptop, the contents is still:

acpi_video0 intel_backlight

No change

Revision history for this message
Seth Forshee (sforshee) wrote :

Please try the kernel at the link below. I think this should get you a samsung backlight and eliminate the acpi_video one. After booting to this kernel please verify that samsung-laptop is loaded, that you have /sys/class/backlight/samsung and no longer have /sys/class/backlight/acpi_video*, and that the Fn keys work for adjusting the backlight level.

http://people.canonical.com/~sforshee/lp1086921/linux-3.7.0-5.13~lp1086921v201212052012/

Revision history for this message
Martin Beltov (mbeltov) wrote :

Booting with your 3.7.0-5-generic, lsmod shows that samsung_laptop is running. lsmod output:
Module Size Used by
snd_hda_codec_realtek 63540 1
snd_hda_intel 32761 4
joydev 17098 0
snd_hda_codec 116443 2 snd_hda_codec_realtek,snd_hda_intel
coretemp 13098 0
rfcomm 37421 4
bnep 17737 2
parport_pc 31969 0
bluetooth 183825 10 rfcomm,bnep
snd_hwdep 13273 1 snd_hda_codec
ppdev 12818 0
snd_pcm 80891 2 snd_hda_intel,snd_hda_codec
gpio_ich 13237 0
snd_seq_midi 13133 0
snd_rawmidi 25383 1 snd_seq_midi
arc4 12544 2
ath9k 130503 0
snd_seq_midi_event 14476 1 snd_seq_midi
samsung_laptop 14101 0
snd_seq 51281 2 snd_seq_midi,snd_seq_midi_event
psmouse 75804 0
mac80211 470993 1 ath9k
serio_raw 13032 0
i915 508195 2
uvcvideo 71280 0
snd_timer 24412 2 snd_pcm,snd_seq
microcode 18287 0
snd_seq_device 14138 3 snd_seq_midi,snd_rawmidi,snd_seq
videobuf2_core 34625 1 uvcvideo
ath9k_common 13784 1 ath9k
videodev 95768 2 uvcvideo,videobuf2_core
drm_kms_helper 45273 1 i915
ath9k_hw 393937 2 ath9k,ath9k_common
videobuf2_vmalloc 12757 1 uvcvideo
drm 227450 3 i915,drm_kms_helper
ath 19188 3 ath9k,ath9k_common,ath9k_hw
mac_hid 13038 0
videobuf2_memops 13213 1 videobuf2_vmalloc
snd 62146 17 snd_hda_codec_realtek,snd_hda_intel,snd_hda_codec,snd_hwdep,snd_pcm,snd_rawmidi,snd_seq,snd_timer,snd_seq_device
cfg80211 184762 3 ath9k,mac80211,ath
hid_generic 12485 0
soundcore 14600 1 snd
i2c_algo_bit 13198 1 i915
snd_page_alloc 14231 2 snd_hda_intel,snd_pcm
video 18848 2 samsung_laptop,i915
lpc_ich 16926 0
lp 13300 0
parport 40754 3 parport_pc,ppdev,lp
usbhid 41806 0
hid 82179 2 hid_generic,usbhid
sky2 52927 0

output of ls /sys/class/backlight:
intel_backlight samsung

Samsung is now here and Fn keys work. This solved the problem. A million thanks !

Revision history for this message
Seth Forshee (sforshee) wrote :

Great! I'll work on getting the changes applied.

Unfortunately this isn't a fix for all Samsung laptops with broken backlight adjustment. The driver assumes by default that acpi_video should be used, and models with broken acpi_video have to be added individually. So this is only going to fix your particular model.

Revision history for this message
Martin Beltov (mbeltov) wrote :

Thank you. Will this work only on *buntu or will you submit the patch upstream?
I realise that it is not possbile to fix every laptop issue atm, but this is a promising start :)

Revision history for this message
Seth Forshee (sforshee) wrote :

I'll make sure it gets upstream.

Revision history for this message
Martin Beltov (mbeltov) wrote :

Thank you. Best of luck to you

Changed in linux (Ubuntu):
status: Incomplete → Fix Committed
Seth Forshee (sforshee)
summary: - include samsung-backlight fix in main kernel
+ [SAMSUNG N250P] acpi_video backlight doesn't work
Changed in linux (Ubuntu):
assignee: nobody → Seth Forshee (sforshee)
status: Fix Committed → In Progress
Seth Forshee (sforshee)
description: updated
Tim Gardner (timg-tpi)
Changed in linux (Ubuntu Raring):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Quantal):
assignee: nobody → Seth Forshee (sforshee)
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package linux - 3.8.0-1.5

---------------
linux (3.8.0-1.5) raring; urgency=low

  [Tim Gardner]

  * Release Tracking Bug
    - LP: #1101235

  [ Dudley Du ]

  * SAUCE: Input: add support for Cypress PS/2 Trackpads
    - LP: #978807

  [ Kamal Mostafa ]

  * SAUCE: Input: increase struct ps2dev cmdbuf[] to 8 bytes
  * SAUCE: Input: Cypress PS/2 Trackpad simulated multitouch
  * [Config] Add CONFIG_PS2_CYPRESS

  [ Tim Gardner ]

  * rebase to v3.8-rc4

  [ Upstream Kernel Changes ]

  * rebase to v3.8-rc4
    - LP: #1095315
    - LP: #886975
    - LP: #1086921
 -- Leann Ogasawara <email address hidden> Thu, 17 Jan 2013 10:50:22 -0800

Changed in linux (Ubuntu Raring):
status: Fix Committed → Fix Released
Revision history for this message
Luis Henriques (henrix) wrote :

This bug is awaiting verification that the kernel for Quantal in -proposed solves the problem (3.5.0-24.37). Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-quantal' to 'verification-done-quantal'.

If verification is not done by one week from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-quantal
Revision history for this message
Martin Beltov (mbeltov) wrote :

Verification done. Using the kernel 3.5.0-24-generic from quantal-proposed fixed the issue

tags: added: verification-done-quantal
removed: verification-needed-quantal
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.

Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (28.9 KiB)

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

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

  [Luis Henriques]

  * Release Tracking Bug
    - LP: #1117492

  [ Tim Gardner ]

  * [Config] CONFIG_ALX=m for x86 only
    - LP: #927782

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

  [Luis Henriques]

  * Release Tracking Bug
    - LP: #1116501

  [ Adam Lee ]

  * [Config] Enable RTSX_PCI modules
    - LP: #1057089

  [ Andy Whitcroft ]

  * [Config] enable various HVC consoles
    - LP: #1102206

  [ Brad Figg ]

  * Revert "SAUCE: samsung-laptop: disable in UEFI mode"
    - LP: #1111689

  [ Herton Ronaldo Krzesinski ]

  * [Config] updateconfigs for 3.5.7.3 stable update
  * d-i: Add mellanox ethernet drivers to nic-modules
    - LP: #1015339

  [ Kamal Mostafa ]

  * SAUCE: alx driver import script
    - LP: #927782

  [ Qualcomm Atheros, Inc ]

  * SAUCE: alx: Update to heads/master
    - LP: #927782

  [ Seth Forshee ]

  * SAUCE: samsung-laptop: Add quirk for broken acpi_video backlight on
    N250P
    - LP: #1086921

  [ Stefan Bader ]

  * (config) Move 9p modules into generic package
    - LP: #1107658

  [ Tim Gardner ]

  * [debian] Remove dangling symlink from headers package
    - LP: #1112442
  * [config] CONFIG_ALX=m
    - LP: #927782
  * [Config] Add alx to d-i nic-modules
    - LP: #927782

  [ Upstream Kernel Changes ]

  * Revert "8139cp: revert "set ring address before enabling receiver""
    - LP: #1102417
  * Revert "ath9k_hw: Update AR9003 high_power tx gain table"
    - LP: #1102417
  * Revert "drm/i915: no lvds quirk for Zotac ZDBOX SD ID12/ID13"
    - LP: #1102417
  * Revert "ALSA: hda - Shut up pins at power-saving mode with Conexnat
    codecs"
    - LP: #1106966, #886975
  * be2net: don't call vid_config() when there's no vlan config
    - LP: #1083088
  * be2net: cleanup be_vid_config()
    - LP: #1083088
  * be2net: do not modify PCI MaxReadReq size
    - LP: #1083088
  * be2net: fix reporting number of actual rx queues
    - LP: #1083088
  * be2net: do not use SCRATCHPAD register
    - LP: #1083088
  * be2net: Fix driver load for VFs for Lancer
    - LP: #1083088
  * be2net: Explicitly clear the reserved field in the Tx Descriptor
    - LP: #1083088
  * be2net: Regression bug wherein VFs creation broken for multiple cards.
    - LP: #1083088
  * be2net: Fix to trim skb for padded vlan packets to workaround an ASIC
    Bug
    - LP: #1083088
  * be2net: Fix Endian
    - LP: #1083088
  * be2net: Fix error while toggling autoneg of pause parameters
    - LP: #1083088
  * be2net : Fix die temperature stat for Lancer
    - LP: #1083088
  * be2net: Fix initialization sequence for Lancer
    - LP: #1083088
  * be2net: Activate new FW after FW download for Lancer
    - LP: #1083088
  * be2net: Fix cleanup path when EQ creation fails
    - LP: #1083088
  * be2net: Enable RSS UDP hashing for Lancer and Skyhawk
    - LP: #1083088
  * be2net: dont pull too much data in skb linear part
    - LP: #1083088
  * be2net: Fix VF driver load for Lancer
    - LP: #1083088
  * be2net: Ignore physical link async event for Lancer
    - LP: #1083088
  * be2net: Fix to parse RSS hash from Receive compl...

Changed in linux (Ubuntu Quantal):
status: Fix Committed → Fix Released
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.