From: "Pali Rohár" <pali.rohar@gmail.com>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Darren Hart <dvhart@infradead.org>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
Ben Skeggs <bskeggs@redhat.com>,
Azael Avalos <coproscefalo@gmail.com>,
Corentin Chary <corentin.chary@gmail.com>,
Lee Chun-Yi <jlee@suse.com>,
Cezary Jackiewicz <cezary.jackiewicz@gmail.com>,
Matthew Garrett <mjg59@srcf.ucam.org>,
Ike Panhc <ike.pan@canonical.com>,
Anisse Astier <anisse@astier.eu>,
Mattia Dongili <malattia@linux.it>,
Henrique de Moraes Holschuh <ibm-acpi@hmh.eng.br>,
platform-driver-x86@vger.kernel.org,
ibm-acpi-devel@lists.sourceforge.net,
acpi4asus-user@lists.sourceforge.net,
dri-devel@lists.freedesktop.org, Aaron Lu <aaron.lu@intel.com>,
linux-acpi@vger.kernel.org
Subject: Re: [PATCH 07/32] acpi-video-detect: Rewrite backlight interface selection logic
Date: Thu, 11 Jun 2015 11:19:22 +0200 [thread overview]
Message-ID: <20150611091922.GB19820@pali> (raw)
In-Reply-To: <1433941292-21530-8-git-send-email-hdegoede@redhat.com>
On Wednesday 10 June 2015 15:01:07 Hans de Goede wrote:
> Currently we have 2 kernel commandline options + dmi-quirks in 3 places all
> interacting (in interesting ways) to select which which backlight interface
> to use. On the commandline we've acpi_backlight=[video|vendor] and
> video.use_native_backlight=[0|1]. DMI quirks we have in
> acpi/video-detect.c, acpi/video.c and drivers/platform/x86/*.c .
>
> This commit is the first step to cleaning this up, replacing the 2 cmdline
> options with just acpi_video.backlight=[video|vendor|native|none], and
> adding a new API to video_detect.c to reflect this.
>
> Follow up commits will also move other related code, like unregistering the
> acpi_video backlight interface if it was registered before other drivers
> which take priority over it are loaded, to video_detect.c where this
> logic really belongs.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Hello,
lot of people are using acpi_backlight parameter in kernel cmdline to
fix some of problems. I would like to see this parameter still working
and to not break existing configuration. E.g acpi_backlight=vendor to
use dell-laptop.ko or thinkpad_acpi.ko backlight.
It is still nightmare to get laptop panel backlight working on different
(broken) laptops and people learnt to try to use acpi_backlight
parameter for quit/hot fixing these problems. Upgrading kernel (if you
remove acpi_backlight parameter) just break it again.
--
Pali Rohár
pali.rohar@gmail.com
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-06-11 9:19 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-10 13:01 [PATCH 00/32] acpi-video: Rewrite backlight interface selection logic Hans de Goede
2015-06-10 13:01 ` [PATCH 01/32] apple-gmux: Stop using acpi_video_dmi_demote_vendor() Hans de Goede
2015-06-10 13:01 ` [PATCH 02/32] acpi-video-detect: Remove the unused acpi_video_dmi_demote_vendor() function Hans de Goede
2015-06-10 13:01 ` [PATCH 03/32] acpi-video-detect: Make acpi_video_get_capabilities a private function Hans de Goede
2015-06-10 13:01 ` [PATCH 04/32] acpi-video-detect: Move acpi_is_video_device() to acpi/scan.c Hans de Goede
2015-06-10 13:01 ` [PATCH 05/32] acpi-video-detect: Move acpi_osi_is_win8 to osl.c Hans de Goede
2015-06-10 13:01 ` [PATCH 06/32] acpi-video-detect: video: Make video_detect code part of the video module Hans de Goede
2015-06-10 13:01 ` [PATCH 07/32] acpi-video-detect: Rewrite backlight interface selection logic Hans de Goede
2015-06-11 9:00 ` Aaron Lu
2015-06-11 9:19 ` Hans de Goede
2015-06-11 9:29 ` Aaron Lu
2015-06-11 9:19 ` Pali Rohár [this message]
2015-06-11 9:29 ` Hans de Goede
2015-06-11 12:01 ` Pali Rohár
2015-06-11 12:28 ` Jani Nikula
[not found] ` <87bngm5szf.fsf-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-06-11 12:32 ` Hans de Goede
2015-06-10 13:01 ` [PATCH 08/32] acpi-video-detect: Unregister acpi_video backlight when dmi quirks are added Hans de Goede
2015-06-10 13:01 ` [PATCH 09/32] drm: i915: Port to new backlight interface selection API Hans de Goede
2015-06-10 13:01 ` [PATCH 10/32] acpi-video: " Hans de Goede
2015-06-10 13:01 ` [PATCH 11/32] acpi-video: Move backlight notifier to video_detect.c Hans de Goede
2015-06-10 13:01 ` [PATCH 12/32] acpi-video: Move dmi_check_system from module_init to acpi_video_register Hans de Goede
2015-06-10 13:01 ` [PATCH 13/32] acpi-video: Fix acpi_video _register vs _unregister_backlight race Hans de Goede
2015-06-10 13:01 ` [PATCH 14/32] acer-wmi: Port to new backlight interface selection API Hans de Goede
2015-06-11 3:01 ` joeyli
2015-06-10 13:01 ` [PATCH 15/32] apple-gmux: " Hans de Goede
2015-06-10 13:01 ` [PATCH 16/32] asus-laptop: " Hans de Goede
2015-06-10 13:01 ` [PATCH 17/32] asus-wmi: " Hans de Goede
2015-06-10 13:01 ` [PATCH 18/32] compal-laptop: " Hans de Goede
2015-06-10 13:01 ` [PATCH 19/32] dell-laptop: " Hans de Goede
2015-06-11 11:47 ` Pali Rohár
2015-06-11 12:29 ` Hans de Goede
2015-06-11 13:06 ` Pali Rohár
2015-06-10 13:01 ` [PATCH 20/32] dell-wmi: " Hans de Goede
2015-06-11 11:43 ` Pali Rohár
2015-06-11 12:19 ` Hans de Goede
2015-06-11 12:59 ` Pali Rohár
2015-06-10 13:01 ` [PATCH 21/32] eeepc-laptop: " Hans de Goede
2015-06-10 13:01 ` [PATCH 22/32] fujitsu-laptop: " Hans de Goede
2015-06-10 23:04 ` Jonathan Woithe
2015-06-10 13:01 ` [PATCH 23/32] ideapad-laptop: " Hans de Goede
2015-06-10 13:01 ` [PATCH 24/32] intel-oaktrail: " Hans de Goede
2015-06-10 13:01 ` [PATCH 25/32] msi-laptop: " Hans de Goede
2015-06-10 13:01 ` [PATCH 26/32] msi-wmi: " Hans de Goede
2015-06-10 13:01 ` [PATCH 27/32] samsung-laptop: " Hans de Goede
2015-06-10 13:01 ` [PATCH 28/32] sony-laptop: " Hans de Goede
2015-06-10 21:30 ` Mattia Dongili
2015-06-12 8:18 ` Hans de Goede
2015-06-10 13:01 ` [PATCH 29/32] thinkpad-acpi: " Hans de Goede
2015-06-10 13:59 ` Henrique de Moraes Holschuh
2015-06-10 13:01 ` [PATCH 30/32] toshiba-acpi: " Hans de Goede
2015-06-10 13:01 ` [PATCH 31/32] acpi-video-detect: Remove old API Hans de Goede
2015-06-10 13:01 ` [PATCH 32/32] acpi-video: Make acpi_video_unregister_backlight() private Hans de Goede
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=20150611091922.GB19820@pali \
--to=pali.rohar@gmail.com \
--cc=aaron.lu@intel.com \
--cc=acpi4asus-user@lists.sourceforge.net \
--cc=anisse@astier.eu \
--cc=bskeggs@redhat.com \
--cc=cezary.jackiewicz@gmail.com \
--cc=coproscefalo@gmail.com \
--cc=corentin.chary@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=dvhart@infradead.org \
--cc=hdegoede@redhat.com \
--cc=ibm-acpi-devel@lists.sourceforge.net \
--cc=ibm-acpi@hmh.eng.br \
--cc=ike.pan@canonical.com \
--cc=jlee@suse.com \
--cc=linux-acpi@vger.kernel.org \
--cc=malattia@linux.it \
--cc=mjg59@srcf.ucam.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rjw@rjwysocki.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