drm/i915: Remove unnecessary () used with WARN()
authorDamien Lespiau <damien.lespiau@intel.com>
Thu, 4 Jun 2015 15:42:15 +0000 (16:42 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 15 Jun 2015 12:30:18 +0000 (14:30 +0200)
In Linux, macros are usually well done and protect their arguments
properly, even avoiding multiple evaluations of the parameters. Extra ()
are really not needed.

Cc: Suketu Shah <suketu.j.shah@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_csr.c

index 5cb8cc18994a37528bef8e1b180b23356dd11509..aae065280d1b40df4280e07a4d87c017f780dc3d 100644 (file)
@@ -459,7 +459,8 @@ void intel_csr_ucode_fini(struct drm_device *dev)
 
 void assert_csr_loaded(struct drm_i915_private *dev_priv)
 {
-       WARN((intel_csr_load_status_get(dev_priv) != FW_LOADED), "CSR is not loaded.\n");
+       WARN(intel_csr_load_status_get(dev_priv) != FW_LOADED,
+            "CSR is not loaded.\n");
        WARN(!I915_READ(CSR_PROGRAM_BASE),
                                "CSR program storage start is NULL\n");
        WARN(!I915_READ(CSR_SSP_BASE), "CSR SSP Base Not fine\n");