[Intel-gfx] [PATCH] drm/i915/opregion: Use ASLE response codes defined in 0.1

2010-08-05 Thread Chris Wilson
Within i915_opregion.c there are two blocks of semantically identical
ASLE response codes defined. Only one of those matches the ACPI IGD
OpRegion Specification 0.1, use those.

Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Matthew Garrett mj...@srcf.ucam.org
---
 drivers/gpu/drm/i915/i915_opregion.c |   10 +++---
 1 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_opregion.c 
b/drivers/gpu/drm/i915/i915_opregion.c
index 8fcc75c..ce402b2 100644
--- a/drivers/gpu/drm/i915/i915_opregion.c
+++ b/drivers/gpu/drm/i915/i915_opregion.c
@@ -114,10 +114,6 @@ struct opregion_asle {
 #define ASLE_REQ_MSK   0xf
 
 /* response bits of ASLE irq request */
-#define ASLE_ALS_ILLUM_FAIL(210)
-#define ASLE_BACKLIGHT_FAIL(212)
-#define ASLE_PFIT_FAIL (214)
-#define ASLE_PWM_FREQ_FAIL (216)
 #define ASLE_ALS_ILLUM_FAILED  (110)
 #define ASLE_BACKLIGHT_FAILED  (112)
 #define ASLE_PFIT_FAILED   (114)
@@ -155,11 +151,11 @@ static u32 asle_set_backlight(struct drm_device *dev, u32 
bclp)
u32 max_backlight, level, shift;
 
if (!(bclp  ASLE_BCLP_VALID))
-   return ASLE_BACKLIGHT_FAIL;
+   return ASLE_BACKLIGHT_FAILED;
 
bclp = ASLE_BCLP_MSK;
if (bclp  0 || bclp  255)
-   return ASLE_BACKLIGHT_FAIL;
+   return ASLE_BACKLIGHT_FAILED;
 
blc_pwm_ctl = I915_READ(BLC_PWM_CTL);
blc_pwm_ctl2 = I915_READ(BLC_PWM_CTL2);
@@ -211,7 +207,7 @@ static u32 asle_set_pfit(struct drm_device *dev, u32 pfit)
/* Panel fitting is currently controlled by the X code, so this is a
   noop until modesetting support works fully */
if (!(pfit  ASLE_PFIT_VALID))
-   return ASLE_PFIT_FAIL;
+   return ASLE_PFIT_FAILED;
return 0;
 }
 
-- 
1.7.1

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx


Re: [Intel-gfx] [PATCH] drm/i915/opregion: Use ASLE response codes defined in 0.1

2010-08-05 Thread Matthew Garrett
On Thu, Aug 05, 2010 at 12:54:49PM +0100, Chris Wilson wrote:
 Within i915_opregion.c there are two blocks of semantically identical
 ASLE response codes defined. Only one of those matches the ACPI IGD
 OpRegion Specification 0.1, use those.
 
 Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
 Cc: Matthew Garrett mj...@srcf.ucam.org

Acked-by: Matthew Garrett m...@redhat.com

-- 
Matthew Garrett | mj...@srcf.ucam.org
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx