Re: [PATCH] Platform: samsung-laptop: add DMI information for SamsungN150 Plus

[David Herrmann]

On Sat, Dec 24, 2011 at 3:20 PM, Matthew Garrett <mjg59@xxxxxxxxxxxxx> wrote:
> On Sat, Dec 24, 2011 at 02:12:57PM +0000, Andrzej Prochyra wrote:
>> 'acpi_video0' not functional.  Brightness can be controlled through
>> intel_backlight interface.  There are 21 brightness levels and the
>> backlight can be turned off.
>
> If acpi_video0 doesn't work then we either need to work out why it's not
> working or make it vanish completely, so this patch isn't the correct
> approach.

Just to make clear, intel_backlight isn't working properly either. If
my laptop boots with backlight at 50% then intel_backlight will only
work in the range 0%-50% on my machine. If I use the samsung module to
set it to 100% then  intel_backlight works correctly in the range
0-100% until next reboot/suspend.

So I'd recommend staying with the samsung module and disabling the
acpi_backlight entirely. The samsung module is the only backlight
control which works properly.

Regards
David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/