* [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725
@ 2013-04-09 12:38 Josh Boyer
2013-04-09 14:12 ` Jani Nikula
0 siblings, 1 reply; 6+ messages in thread
From: Josh Boyer @ 2013-04-09 12:38 UTC (permalink / raw)
To: gregkh, Daniel Vetter, Jani Nikula; +Cc: stable, linux-kernel
Upstream commit 01e3a8feb40e54b962a20fa7eb595c5efef5e109
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=31522#c35
[Note: There are more than one broken setups in the bug. This fixes one.]
Reported-by: Martins <andrissr@inbox.lv>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
---
drivers/gpu/drm/i915/intel_display.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index e6e4df7..6292677 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -8899,6 +8899,12 @@ static struct intel_quirk intel_quirks[] = {
/* Acer Aspire 5734Z must invert backlight brightness */
{ 0x2a42, 0x1025, 0x0459, quirk_invert_brightness },
+ /* Acer/eMachines G725 */
+ { 0x2a42, 0x1025, 0x0210, quirk_invert_brightness },
+
+ /* Acer/eMachines e725 */
+ { 0x2a42, 0x1025, 0x0212, quirk_invert_brightness },
+
/* Acer Aspire 4736Z */
{ 0x2a42, 0x1025, 0x0260, quirk_invert_brightness },
};
--
1.8.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725
2013-04-09 12:38 [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725 Josh Boyer
@ 2013-04-09 14:12 ` Jani Nikula
2013-04-09 14:22 ` Josh Boyer
0 siblings, 1 reply; 6+ messages in thread
From: Jani Nikula @ 2013-04-09 14:12 UTC (permalink / raw)
To: Josh Boyer, gregkh, Daniel Vetter; +Cc: stable, linux-kernel
On Tue, 09 Apr 2013, Josh Boyer <jwboyer@redhat.com> wrote:
> Upstream commit 01e3a8feb40e54b962a20fa7eb595c5efef5e109
This patch seems to be the above commit and
commit 1ffff60320879830e469e26062c18f75236822ba
Author: Jani Nikula <jani.nikula@intel.com>
Date: Tue Jan 22 12:50:34 2013 +0200
drm/i915: add quirk to invert brightness on eMachines G725
squashed together. There's a separate bug for that too:
https://bugs.freedesktop.org/show_bug.cgi?id=59628
I think both are okay for stable, but by the stable rules you should
probably split this up, with the appropriate upstream references in
both. Or do whatever the stable team tells you to do. ;)
BR,
Jani.
>
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=31522#c35
> [Note: There are more than one broken setups in the bug. This fixes one.]
> Reported-by: Martins <andrissr@inbox.lv>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> ---
> drivers/gpu/drm/i915/intel_display.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index e6e4df7..6292677 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -8899,6 +8899,12 @@ static struct intel_quirk intel_quirks[] = {
> /* Acer Aspire 5734Z must invert backlight brightness */
> { 0x2a42, 0x1025, 0x0459, quirk_invert_brightness },
>
> + /* Acer/eMachines G725 */
> + { 0x2a42, 0x1025, 0x0210, quirk_invert_brightness },
> +
> + /* Acer/eMachines e725 */
> + { 0x2a42, 0x1025, 0x0212, quirk_invert_brightness },
> +
> /* Acer Aspire 4736Z */
> { 0x2a42, 0x1025, 0x0260, quirk_invert_brightness },
> };
> --
> 1.8.1.4
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725
2013-04-09 14:12 ` Jani Nikula
@ 2013-04-09 14:22 ` Josh Boyer
2013-04-10 2:34 ` Ben Hutchings
0 siblings, 1 reply; 6+ messages in thread
From: Josh Boyer @ 2013-04-09 14:22 UTC (permalink / raw)
To: Jani Nikula; +Cc: gregkh, Daniel Vetter, stable, linux-kernel
On Tue, Apr 09, 2013 at 05:12:18PM +0300, Jani Nikula wrote:
> On Tue, 09 Apr 2013, Josh Boyer <jwboyer@redhat.com> wrote:
> > Upstream commit 01e3a8feb40e54b962a20fa7eb595c5efef5e109
>
> This patch seems to be the above commit and
>
> commit 1ffff60320879830e469e26062c18f75236822ba
> Author: Jani Nikula <jani.nikula@intel.com>
> Date: Tue Jan 22 12:50:34 2013 +0200
>
> drm/i915: add quirk to invert brightness on eMachines G725
>
> squashed together. There's a separate bug for that too:
> https://bugs.freedesktop.org/show_bug.cgi?id=59628
Oh... that actually does seem to be the case. I did a cherry-pick
of just the commit I mentioned on top of 3.8.6 and it seems I resolved
the conflict by including in changes for both. The conflict came up as:
<<<<<<< HEAD
/* Acer Aspire 4736Z */
{ 0x2a42, 0x1025, 0x0260, quirk_invert_brightness },
=======
/* Acer/eMachines G725 */
{ 0x2a42, 0x1025, 0x0210, quirk_invert_brightness },
/* Acer/eMachines e725 */
{ 0x2a42, 0x1025, 0x0212, quirk_invert_brightness },
>>>>>>> 01e3a8f... drm/i915: add quirk to invert brightness on eMachines e725
Thanks for catching that.
> I think both are okay for stable, but by the stable rules you should
> probably split this up, with the appropriate upstream references in
> both. Or do whatever the stable team tells you to do. ;)
Yeah. I'm happy to split them up and send them out separately. Or if
Greg wants to just list both upstream commit IDs with a single backport,
that would also be fine.
josh
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725
2013-04-09 14:22 ` Josh Boyer
@ 2013-04-10 2:34 ` Ben Hutchings
2013-04-10 9:37 ` Jani Nikula
0 siblings, 1 reply; 6+ messages in thread
From: Ben Hutchings @ 2013-04-10 2:34 UTC (permalink / raw)
To: Josh Boyer, Greg Kroah-Hartman, Daniel Vetter
Cc: Jani Nikula, stable, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 2452 bytes --]
On Tue, 2013-04-09 at 10:22 -0400, Josh Boyer wrote:
> On Tue, Apr 09, 2013 at 05:12:18PM +0300, Jani Nikula wrote:
> > On Tue, 09 Apr 2013, Josh Boyer <jwboyer@redhat.com> wrote:
> > > Upstream commit 01e3a8feb40e54b962a20fa7eb595c5efef5e109
> >
> > This patch seems to be the above commit and
> >
> > commit 1ffff60320879830e469e26062c18f75236822ba
> > Author: Jani Nikula <jani.nikula@intel.com>
> > Date: Tue Jan 22 12:50:34 2013 +0200
> >
> > drm/i915: add quirk to invert brightness on eMachines G725
> >
> > squashed together. There's a separate bug for that too:
> > https://bugs.freedesktop.org/show_bug.cgi?id=59628
>
> Oh... that actually does seem to be the case. I did a cherry-pick
> of just the commit I mentioned on top of 3.8.6 and it seems I resolved
> the conflict by including in changes for both. The conflict came up as:
>
> <<<<<<< HEAD
> /* Acer Aspire 4736Z */
> { 0x2a42, 0x1025, 0x0260, quirk_invert_brightness },
> =======
> /* Acer/eMachines G725 */
> { 0x2a42, 0x1025, 0x0210, quirk_invert_brightness },
>
> /* Acer/eMachines e725 */
> { 0x2a42, 0x1025, 0x0212, quirk_invert_brightness },
> >>>>>>> 01e3a8f... drm/i915: add quirk to invert brightness on eMachines e725
>
> Thanks for catching that.
>
> > I think both are okay for stable, but by the stable rules you should
> > probably split this up, with the appropriate upstream references in
> > both. Or do whatever the stable team tells you to do. ;)
>
> Yeah. I'm happy to split them up and send them out separately. Or if
> Greg wants to just list both upstream commit IDs with a single backport,
> that would also be fine.
I cherry-picked the following series for Debian:
[pre-3.8]
7bd90909bbf9 drm/i915: panel: invert brightness via parameter
4dca20efb1a9 drm/i915: panel: invert brightness via quirk
5a15ab5b93e4 drm/i915: panel: invert brightness acer aspire 5734z
5f85f176c2f1 DRM/i915: Add QUIRK_INVERT_BRIGHTNESS for NCR machines.
[post-3.8]
1ffff6032087 drm/i915: add quirk to invert brightness on eMachines G725
01e3a8feb40e drm/i915: add quirk to invert brightness on eMachines e725
5559ecadad5a drm/i915: add quirk to invert brightness on Packard Bell NCL20
Should these all be applied to the various 3.x.y stable branches?
Ben.
--
Ben Hutchings
The program is absolutely right; therefore, the computer must be wrong.
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725
2013-04-10 2:34 ` Ben Hutchings
@ 2013-04-10 9:37 ` Jani Nikula
2013-04-22 1:03 ` Ben Hutchings
0 siblings, 1 reply; 6+ messages in thread
From: Jani Nikula @ 2013-04-10 9:37 UTC (permalink / raw)
To: Ben Hutchings, Josh Boyer, Greg Kroah-Hartman, Daniel Vetter
Cc: stable, linux-kernel
On Wed, 10 Apr 2013, Ben Hutchings <ben@decadent.org.uk> wrote:
> I cherry-picked the following series for Debian:
>
> [pre-3.8]
> 7bd90909bbf9 drm/i915: panel: invert brightness via parameter
> 4dca20efb1a9 drm/i915: panel: invert brightness via quirk
> 5a15ab5b93e4 drm/i915: panel: invert brightness acer aspire 5734z
> 5f85f176c2f1 DRM/i915: Add QUIRK_INVERT_BRIGHTNESS for NCR machines.
> [post-3.8]
> 1ffff6032087 drm/i915: add quirk to invert brightness on eMachines G725
> 01e3a8feb40e drm/i915: add quirk to invert brightness on eMachines e725
> 5559ecadad5a drm/i915: add quirk to invert brightness on Packard Bell NCL20
>
> Should these all be applied to the various 3.x.y stable branches?
Acked-by: Jani Nikula <jani.nikula@intel.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725
2013-04-10 9:37 ` Jani Nikula
@ 2013-04-22 1:03 ` Ben Hutchings
0 siblings, 0 replies; 6+ messages in thread
From: Ben Hutchings @ 2013-04-22 1:03 UTC (permalink / raw)
To: Jani Nikula
Cc: Josh Boyer, Greg Kroah-Hartman, Daniel Vetter, stable,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1041 bytes --]
On Wed, 2013-04-10 at 12:37 +0300, Jani Nikula wrote:
> On Wed, 10 Apr 2013, Ben Hutchings <ben@decadent.org.uk> wrote:
> > I cherry-picked the following series for Debian:
> >
> > [pre-3.8]
> > 7bd90909bbf9 drm/i915: panel: invert brightness via parameter
> > 4dca20efb1a9 drm/i915: panel: invert brightness via quirk
> > 5a15ab5b93e4 drm/i915: panel: invert brightness acer aspire 5734z
> > 5f85f176c2f1 DRM/i915: Add QUIRK_INVERT_BRIGHTNESS for NCR machines.
> > [post-3.8]
> > 1ffff6032087 drm/i915: add quirk to invert brightness on eMachines G725
> > 01e3a8feb40e drm/i915: add quirk to invert brightness on eMachines e725
> > 5559ecadad5a drm/i915: add quirk to invert brightness on Packard Bell NCL20
> >
> > Should these all be applied to the various 3.x.y stable branches?
>
> Acked-by: Jani Nikula <jani.nikula@intel.com>
Queued up for 3.2, thanks.
Ben.
--
Ben Hutchings
Klipstein's 4th Law of Prototyping and Production:
A fail-safe circuit will destroy others.
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-04-22 1:03 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-09 12:38 [PATCH 3.8.y] drm/i915: add quirk to invert brightness on eMachines e725 Josh Boyer
2013-04-09 14:12 ` Jani Nikula
2013-04-09 14:22 ` Josh Boyer
2013-04-10 2:34 ` Ben Hutchings
2013-04-10 9:37 ` Jani Nikula
2013-04-22 1:03 ` Ben Hutchings
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox