public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Nikolai Kondrashov <spbnick@gmail.com>
To: Greg KH <greg@kroah.com>
Cc: Matthew Garrett <mjg@redhat.com>,
	Randy Dunlap <rdunlap@xenotime.net>,
	linux-kernel@vger.kernel.org,
	platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH] Platform: add Samsung Laptop platform driver
Date: Sun, 13 Mar 2011 12:03:20 +0200	[thread overview]
Message-ID: <4D7C9668.3000902@gmail.com> (raw)
In-Reply-To: <20110311203032.GA14384@kroah.com>

Hi Greg,

On 03/11/2011 10:30 PM, Greg KH wrote:
> You can also go into the /sys/class/backlight/samsung/ directory and
> mess with the backlight values in the files to see if it gets brighter
> and dimmer.  If that works, then the driver is working properly.
Thanks :)
I see some strange behavior, though.

Every time I execute this in the /sys/class/backlight/samsung directory:
sudo bash -c 'echo 1 > brightness'
brightness is reduced a little until it reaches the minimum.

After that, every time I execute this:
sudo bash -c 'echo 8 > brightness'
it increases a little until it reaches the maximum.

And each time a message appears in dmesg: "ACPI: Failed to switch the
brightness", until the brightness stops changing.

At the same time, the little Gnome brightness adjustment window appears,
showing the percentage of the brightness, which drops/raises to
minimum/maximum in only four executions of these commands. After that, while
brightness drops/raises with each execution, the percentage stays at
minimum/maximum.

Is this all normal?

Also, brightness buttons seem to invoke the Gnome brightness percentage
window, which shows it changing, but the brightness itself doesn't change.
Is it also normal? If so, is it solved in the newer Gnome and is it what you
tried to tell me before?

Sometime ago I've found this hack on some forum, which I use currently
to set the brightness:

setpci -s 00:02.0 f4.b="$HEX_BYTE"

It allows finer control of the brightness, but it seems it operates directly
on the video card PCI register. I assume there is no sane way to use
this interface from the platform driver, right?

Then, does the wireless on/off button require gnome support too?

Thank you very much :)

Sincerely,
Nick

  reply	other threads:[~2011-03-13 10:03 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-09 22:40 [PATCH] Platform: add Samsung Laptop platform driver Greg KH
2011-02-09 22:44 ` Randy Dunlap
2011-02-09 22:50   ` Greg KH
2011-02-09 22:58     ` Matthew Garrett
2011-02-09 22:50 ` Matthew Garrett
2011-02-09 23:02   ` Greg KH
2011-02-11 11:37 ` Richard Schütz
2011-02-11 15:22   ` Greg KH
2011-02-11 16:27     ` Richard Schütz
2011-02-25 21:01       ` Greg KH
2011-02-11 15:45 ` Matthew Garrett
2011-02-11 18:42   ` Greg KH
2011-02-16 10:06 ` Nikolai Kondrashov
2011-02-25 21:03   ` Greg KH
2011-02-25 21:49     ` Nikolai Kondrashov
2011-02-25 22:50       ` Greg KH
2011-03-10  7:00         ` Nikolai Kondrashov
2011-03-10 17:01           ` Greg KH
2011-03-10 17:13             ` Nikolai Kondrashov
2011-03-10 17:19               ` Greg KH
2011-03-10 17:28                 ` Nikolai Kondrashov
2011-03-10 18:09                   ` Greg KH
2011-03-10 21:05                     ` Nikolai Kondrashov
2011-03-10 21:15                       ` Greg KH
2011-03-11  7:12                         ` Nikolai Kondrashov
2011-03-11 17:03                           ` Greg KH
2011-03-11 19:07                             ` Nikolai Kondrashov
2011-03-11 19:34                               ` Greg KH
2011-03-11 20:20                                 ` Nikolai Kondrashov
2011-03-11 20:23                                   ` Nikolai Kondrashov
2011-03-11 20:35                                     ` Greg KH
2011-03-11 20:30                                   ` Greg KH
2011-03-13 10:03                                     ` Nikolai Kondrashov [this message]
2011-03-13 10:55                                       ` Richard Schütz
2011-03-13 13:02                                         ` Nikolai Kondrashov
2011-03-13 16:18                                           ` Greg KH
2011-03-13 17:49                                             ` Nikolai Kondrashov
2011-03-13 17:58                                               ` Greg KH
2011-03-13 16:17                                       ` Greg KH
2011-03-13 16:28                                         ` Matthew Garrett
2011-03-13 17:51                                           ` Nikolai Kondrashov
2011-03-13 17:50                                         ` Nikolai Kondrashov
  -- strict thread matches above, loose matches on Subject: below --
2011-02-25 22:47 Greg KH
2011-02-27 15:36 Greg KH
2011-03-05  4:24 Greg KH
2011-03-11 17:43 ` Matthew Garrett
2011-03-11 19:35   ` Greg KH
2011-03-11 19:44     ` Matthew Garrett
2011-03-11 19:52       ` Greg KH
2011-03-11 19:55         ` Matthew Garrett
2011-03-14 15:18           ` Jörg-Volker Peetz
2011-03-11 17:57 Tommaso Massimi
2011-06-26 15:08 J Witteveen

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=4D7C9668.3000902@gmail.com \
    --to=spbnick@gmail.com \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg@redhat.com \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=rdunlap@xenotime.net \
    /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