On Wed, 9 Jun 2010 13:45:19 +0800, Zhenyu Wang zhenyuw@linux.intel.com wrote:

On 2010.06.08 21:18:06 +0200, Nicolas Kaiser wrote:

...

checkpatch complains about this define:

WARNING: space prohibited between function name and open parenthesis '(' #377: FILE: gpu/drm/i915/i915_reg.h:377: +#define GEN6_RENDER TIMEOUT_COUNTER_EXPIRED (1 << 6)

Signed-off-by: Nicolas Kaiser nikai@nikai.net

--- a/drivers/gpu/drm/i915/i915_reg.h 2010-06-07 10:30:16.346785884 +0200 +++ b/drivers/gpu/drm/i915/i915_reg.h 2010-06-08 21:05:25.314967445 +0200 @@ -374,7 +374,7 @@ #define GEN6_RENDER_IMR 0x20a8 #define GEN6_RENDER_CONTEXT_SWITCH_INTERRUPT (1 << 8) #define GEN6_RENDER_PPGTT_PAGE_FAULT (1 << 7) -#define GEN6_RENDER TIMEOUT_COUNTER_EXPIRED (1 << 6) +#define GEN6_RENDER_TIMEOUT_COUNTER_EXPIRED (1 << 6) #define GEN6_RENDER_L3_PARITY_ERROR (1 << 5) #define GEN6_RENDER_PIPE_CONTROL_NOTIFY_INTERRUPT (1 << 4) #define GEN6_RENDER_COMMAND_PARSER_MASTER_ERROR (1 << 3)

Thanks for catching this!

Acked-by: Zhenyu Wang zhenyuw@linux.intel.com

Applied to -next.