From: Greg KH <gregkh@suse.de>
To: David Herrmann <dh.herrmann@googlemail.com>
Cc: linux-kernel@vger.kernel.org, mjg@redhat.com
Subject: Re: samsung-laptop backlight control not working
Date: Thu, 19 May 2011 11:27:53 -0700 [thread overview]
Message-ID: <20110519182753.GA23865@suse.de> (raw)
In-Reply-To: <BANLkTikwjL-3m110CcRs+-KLdZpH0rOTsQ@mail.gmail.com>
On Thu, May 19, 2011 at 08:18:16PM +0200, David Herrmann wrote:
> Hi
>
> I tried the new samsung-laptop driver with samsung backlight support.
What kernel are you trying? We have some recent patches to the driver
to try to resolve some of these issues on your laptop.
> It is detected as:
> [109282.931433] samsung_laptop: found laptop model 'N150P/N210P/N220P'
>
> However, backlight support is kind of weird:
> "echo 1 >/sys/class/backlight/samsung/brightness"
> does not set brightness to 1 but instead reduces brightness for 1.
> That is, if my current brightness setting is 8, I need to call "echo 1
> >..." 7 times to get brightness value 1.
>
> "cat /sys/class/backlight...."
> returns 1 after the first try, though.
>
> "echo 3 >..."
> reduces brightness also for 1 but stops at brightness level 3 if I
> repeat the command.
>
> dmesg shows the following for every "echo" I call:
> [110690.073783] ACPI: Failed to switch the brightness
>
> Furthermore "echo 0 >..." doesn't disable backlight but instead
> behaves like a brightness setting below 1.
Yeah, don't echo 0 there, that doesn't work to well. I have some
patches here that you can try if you want, just let me know.
> However, I know the following works on my machine:
> setpci -s 00:02.0 F4.B=255
> It accepts a setting between 0-255 and 255 is full brightness and 0 is
> backlight off.
Don't do that, the video driver will not like you as you are changing
things without telling the bios that something is really happening.
Of course you can continue to do this, it's just strongly not
recommended.
thanks,
greg k-h
next prev parent reply other threads:[~2011-05-19 18:28 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-19 18:18 samsung-laptop backlight control not working David Herrmann
2011-05-19 18:27 ` Greg KH [this message]
2011-05-19 18:53 ` David Herrmann
2011-06-01 14:50 ` David Herrmann
2011-06-13 23:50 ` Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110519182753.GA23865@suse.de \
--to=gregkh@suse.de \
--cc=dh.herrmann@googlemail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox