drm/i915: add support for SBI ops

Dan Carpenter dan.carpenter at oracle.com
Fri Jun 8 07:30:00 PDT 2012


Hello Eugeni Dodonov,

The patch a416edefbbaf: "drm/i915: add support for SBI ops" from May
9, 2012, leads to the following warning:
drivers/gpu/drm/i915/intel_display.c:1347 intel_sbi_write()
	 warn: bitwise AND condition is false here

SBI_READY is defined like this:

#define  SBI_READY                              (0x0<<0)

  1346          spin_lock_irqsave(&dev_priv->dpio_lock, flags);
  1347          if (wait_for((I915_READ(SBI_CTL_STAT) & SBI_READY) == 0,

	if ((x & 0) == 0) { ...

This is always true, so this condition will never timeout.

  1348                                  100)) {
  1349                  DRM_ERROR("timeout waiting for SBI to become ready\n");
  1350                  goto out_unlock;
  1351          }

All the places that use SBI_READY are like this:

drivers/gpu/drm/i915/intel_display.c:1347 intel_sbi_write() warn: bitwise AND condition is false here
drivers/gpu/drm/i915/intel_display.c:1361 intel_sbi_write() warn: bitwise AND condition is false here
drivers/gpu/drm/i915/intel_display.c:1378 intel_sbi_read() warn: bitwise AND condition is false here
drivers/gpu/drm/i915/intel_display.c:1390 intel_sbi_read() warn: bitwise AND condition is false here

regards,
dan carpenter



More information about the dri-devel mailing list