diff options
| author | Alan Cox <alan@linux.intel.com> | 2012-07-25 08:51:20 -0400 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-07-26 03:50:35 -0400 |
| commit | 9978cf50426bc8d386a0753f74ab895e4d6da554 (patch) | |
| tree | 4c0fa9cf10cc44ca5a2175e2eed8bc8d460d61cb | |
| parent | 0274df3e43be962eee0499ee97c514fdb1c8f3ae (diff) | |
i915: Remove silly test
drv_priv->gmbus is an array. Comparing it with NULL is somewhat less useful
than a chocolate teapot.
Possibly we should be testing bus != NULL each iteration of the loop
instead ?
gcc could help by warning too!
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
| -rw-r--r-- | drivers/gpu/drm/i915/intel_i2c.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/intel_i2c.c b/drivers/gpu/drm/i915/intel_i2c.c index 1991a4408cf9..d79500bc1ce5 100644 --- a/drivers/gpu/drm/i915/intel_i2c.c +++ b/drivers/gpu/drm/i915/intel_i2c.c | |||
| @@ -540,9 +540,6 @@ void intel_teardown_gmbus(struct drm_device *dev) | |||
| 540 | struct drm_i915_private *dev_priv = dev->dev_private; | 540 | struct drm_i915_private *dev_priv = dev->dev_private; |
| 541 | int i; | 541 | int i; |
| 542 | 542 | ||
| 543 | if (dev_priv->gmbus == NULL) | ||
| 544 | return; | ||
| 545 | |||
| 546 | for (i = 0; i < GMBUS_NUM_PORTS; i++) { | 543 | for (i = 0; i < GMBUS_NUM_PORTS; i++) { |
| 547 | struct intel_gmbus *bus = &dev_priv->gmbus[i]; | 544 | struct intel_gmbus *bus = &dev_priv->gmbus[i]; |
| 548 | i2c_del_adapter(&bus->adapter); | 545 | i2c_del_adapter(&bus->adapter); |
