Re: [PATCH 1/2] drm/bridge: add Silicon Image SiI9234 driver

[kbuild test robot <lkp-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 8562 bytes --]

Hi Maciej,

[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.13-rc4 next-20170811]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Maciej-Purski/add-Silicon-Image-SiI9234-driver/20170803-200255
base:   git://people.freedesktop.org/~airlied/linux.git drm-next
config: i386-randconfig-h0-08130402 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/gpu//drm/bridge/sii9234.c: In function 'sii9234_mode_valid':
   drivers/gpu//drm/bridge/sii9234.c:909:18: warning: unused variable 'ctx' [-Wunused-variable]
     struct sii9234 *ctx = bridge_to_sii9234(bridge);
                     ^~~
   drivers/gpu//drm/bridge/sii9234.c: In function 'sii9234_mhl_tx_i2c_probe':
>> drivers/gpu//drm/bridge/sii9234.c:971:13: error: 'struct drm_bridge' has no member named 'of_node'
     ctx->bridge.of_node = dev->of_node;
                ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/drm/bridge/mhl.h:18,
                    from drivers/gpu//drm/bridge/sii9234.c:26:
   drivers/gpu//drm/bridge/sii9234.c: At top level:
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'strcpy' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:390:2: note: in expansion of macro 'if'
     if (p_size == (size_t)-1 && q_size == (size_t)-1)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:380:2: note: in expansion of macro 'if'
     if (p_size < size)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:378:2: note: in expansion of macro 'if'
     if (__builtin_constant_p(size) && p_size < size)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:369:2: note: in expansion of macro 'if'
     if (p_size < size)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:367:2: note: in expansion of macro 'if'
     if (__builtin_constant_p(size) && p_size < size)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:358:2: note: in expansion of macro 'if'
     if (p_size < size)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:356:2: note: in expansion of macro 'if'
     if (__builtin_constant_p(size) && p_size < size)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:348:2: note: in expansion of macro 'if'
     if (p_size < size || q_size < size)
     ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:345:3: note: in expansion of macro 'if'
      if (q_size < size)
      ^~
   include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static
       ______f = {     \
       ^
   include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^~~~~~~~~~
   include/linux/string.h:343:3: note: in expansion of macro 'if'
      if (p_size < size)
      ^~

vim +971 drivers/gpu//drm/bridge/sii9234.c

   905	
   906	static enum drm_mode_status sii9234_mode_valid(struct drm_bridge *bridge,
   907						const struct drm_display_mode *mode)
   908	{
 > 909		struct sii9234 *ctx = bridge_to_sii9234(bridge);
   910	
   911		if (mode->clock > MHL1_MAX_CLK)
   912			return MODE_CLOCK_HIGH;
   913	
   914		return MODE_OK;
   915	}
   916	
   917	static const struct drm_bridge_funcs sii9234_bridge_funcs = {
   918		.mode_valid = sii9234_mode_valid,
   919	};
   920	
   921	static int sii9234_mhl_tx_i2c_probe(struct i2c_client *client,
   922						      const struct i2c_device_id *id)
   923	{
   924		struct i2c_adapter *adapter = to_i2c_adapter(client->dev.parent);
   925		struct sii9234 *ctx;
   926		struct device *dev = &client->dev;
   927		int ret;
   928	
   929		ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
   930		if (!ctx)
   931			return -ENOMEM;
   932	
   933		ctx->dev = dev;
   934	
   935		if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA)) {
   936			dev_err(dev, "I2C adapter lacks SMBUS feature\n");
   937			return -EIO;
   938		}
   939	
   940		mutex_init(&ctx->lock);
   941	
   942		ret = sii9234_init_resources(ctx, client);
   943		if (ret < 0) {
   944			dev_err(&client->dev, "failed to initialize sii9234 resources\n");
   945			return ret;
   946		}
   947		ret = sii9234_hw_on(ctx);
   948		if (ret) {
   949			dev_err(&client->dev, "failed to enable power\n");
   950			goto err_resource;
   951		}
   952		sii9234_hw_reset(ctx);
   953	
   954		if (!client->irq) {
   955			dev_err(dev, "no irq provided\n");
   956			return -EINVAL;
   957		}
   958		irq_set_status_flags(client->irq, IRQ_NOAUTOEN);
   959		ret = devm_request_threaded_irq(dev, client->irq, NULL,
   960						sii9234_irq_thread,
   961						IRQF_TRIGGER_HIGH | IRQF_ONESHOT,
   962						"sii9234", ctx);
   963		if (ret < 0) {
   964			dev_err(dev, "failed to install IRQ handler\n");
   965			return ret;
   966		}
   967	
   968		i2c_set_clientdata(client, ctx);
   969	
   970		ctx->bridge.funcs = &sii9234_bridge_funcs;
 > 971		ctx->bridge.of_node = dev->of_node;
   972		drm_bridge_add(&ctx->bridge);
   973	
   974		sii9234_cable_in(ctx);
   975	
   976		return 0;
   977	
   978	err_resource:
   979		sii9234_deinit_resources(ctx);
   980	
   981		return ret;
   982	}
   983	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 25180 bytes --]