drm/i915: Make i2c lock ops static
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 1 Sep 2017 14:31:22 +0000 (17:31 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 1 Sep 2017 16:46:33 +0000 (19:46 +0300)
Make gmbus_lock_ops and proxy_lock_ops static to appease sparse
intel_i2c.c:652:34: warning: symbol 'gmbus_lock_ops' was not declared. Should it be static?
intel_sdvo.c:2981:34: warning: symbol 'proxy_lock_ops' was not declared. Should it be static?

Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Fixes: a85066840d29 ("drm/i915: Rework sdvo proxy i2c locking")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170901143123.7590-2-ville.syrjala@linux.intel.com
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/gpu/drm/i915/intel_i2c.c
drivers/gpu/drm/i915/intel_sdvo.c

index 6698826954e15d98d55bb2b1e35e7520527c4f00..eb5827110d8ffca08cf6b46e6671265a43f40307 100644 (file)
@@ -649,7 +649,7 @@ static void gmbus_unlock_bus(struct i2c_adapter *adapter,
        mutex_unlock(&dev_priv->gmbus_mutex);
 }
 
-const struct i2c_lock_operations gmbus_lock_ops = {
+static const struct i2c_lock_operations gmbus_lock_ops = {
        .lock_bus =    gmbus_lock_bus,
        .trylock_bus = gmbus_trylock_bus,
        .unlock_bus =  gmbus_unlock_bus,
index 9509a18eb00d84f286d525ac7802f7a2da3bd049..7437944b388f3fbcba03d5b3962fa1b1cf13b120 100644 (file)
@@ -2978,7 +2978,7 @@ static void proxy_unlock_bus(struct i2c_adapter *adapter,
        sdvo->i2c->lock_ops->unlock_bus(sdvo->i2c, flags);
 }
 
-const struct i2c_lock_operations proxy_lock_ops = {
+static const struct i2c_lock_operations proxy_lock_ops = {
        .lock_bus =    proxy_lock_bus,
        .trylock_bus = proxy_trylock_bus,
        .unlock_bus =  proxy_unlock_bus,