Re: [PATCH 5/8] v4l: Switch from V4L2 OF not V4L2 fwnode API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Sakari,

[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on next-20170213]
[cannot apply to v4.10-rc8]
[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/Sakari-Ailus/v4l-flash-led-class-Use-fwnode_handle-instead-of-device_node-in-init/20170213-213642
base:   git://linuxtv.org/media_tree.git master
config: i386-randconfig-x003-201707 (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 error/warnings (new ones prefixed by >>):

   In file included from drivers/media/i2c/tvp514x.c:41:0:
   include/media/v4l2-fwnode.h:67:25: error: field 'base' has incomplete type
     struct fwnode_endpoint base;
                            ^~~~
   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/linux/mod_devicetable.h:11,
                    from include/linux/i2c.h:29,
                    from drivers/media/i2c/tvp514x.c:28:
   drivers/media/i2c/tvp514x.c: In function 'tvp514x_get_pdata':
>> drivers/media/i2c/tvp514x.c:1012:33: error: implicit declaration of function 'of_fwnode_handle' [-Werror=implicit-function-declaration]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> drivers/media/i2c/tvp514x.c:1012:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
>> drivers/media/i2c/tvp514x.c:1012:33: warning: passing argument 1 of 'v4l2_fwnode_endpoint_parse' makes pointer from integer without a cast [-Wint-conversion]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> drivers/media/i2c/tvp514x.c:1012:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
   In file included from drivers/media/i2c/tvp514x.c:41:0:
   include/media/v4l2-fwnode.h:95:5: note: expected 'struct fwnode_handle *' but argument is of type 'int'
    int v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwn,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   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/linux/mod_devicetable.h:11,
                    from include/linux/i2c.h:29,
                    from drivers/media/i2c/tvp514x.c:28:
>> drivers/media/i2c/tvp514x.c:1012:33: warning: passing argument 1 of 'v4l2_fwnode_endpoint_parse' makes pointer from integer without a cast [-Wint-conversion]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:149:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^~~~
>> drivers/media/i2c/tvp514x.c:1012:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
   In file included from drivers/media/i2c/tvp514x.c:41:0:
   include/media/v4l2-fwnode.h:95:5: note: expected 'struct fwnode_handle *' but argument is of type 'int'
    int v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwn,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   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/linux/mod_devicetable.h:11,
                    from include/linux/i2c.h:29,
                    from drivers/media/i2c/tvp514x.c:28:
>> drivers/media/i2c/tvp514x.c:1012:33: warning: passing argument 1 of 'v4l2_fwnode_endpoint_parse' makes pointer from integer without a cast [-Wint-conversion]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:160:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^~~~
>> drivers/media/i2c/tvp514x.c:1012:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
   In file included from drivers/media/i2c/tvp514x.c:41:0:
   include/media/v4l2-fwnode.h:95:5: note: expected 'struct fwnode_handle *' but argument is of type 'int'
    int v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwn,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors
--
   In file included from drivers/media/i2c/tvp7002.c:36:0:
   include/media/v4l2-fwnode.h:67:25: error: field 'base' has incomplete type
     struct fwnode_endpoint base;
                            ^~~~
   In file included from include/linux/linkage.h:4:0,
                    from include/linux/kernel.h:6,
                    from include/linux/delay.h:10,
                    from drivers/media/i2c/tvp7002.c:23:
   drivers/media/i2c/tvp7002.c: In function 'tvp7002_get_pdata':
>> drivers/media/i2c/tvp7002.c:904:33: error: implicit declaration of function 'of_fwnode_handle' [-Werror=implicit-function-declaration]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> drivers/media/i2c/tvp7002.c:904:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
>> drivers/media/i2c/tvp7002.c:904:33: warning: passing argument 1 of 'v4l2_fwnode_endpoint_parse' makes pointer from integer without a cast [-Wint-conversion]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> drivers/media/i2c/tvp7002.c:904:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
   In file included from drivers/media/i2c/tvp7002.c:36:0:
   include/media/v4l2-fwnode.h:95:5: note: expected 'struct fwnode_handle *' but argument is of type 'int'
    int v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwn,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from include/linux/linkage.h:4:0,
                    from include/linux/kernel.h:6,
                    from include/linux/delay.h:10,
                    from drivers/media/i2c/tvp7002.c:23:
>> drivers/media/i2c/tvp7002.c:904:33: warning: passing argument 1 of 'v4l2_fwnode_endpoint_parse' makes pointer from integer without a cast [-Wint-conversion]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:149:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^~~~
>> drivers/media/i2c/tvp7002.c:904:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
   In file included from drivers/media/i2c/tvp7002.c:36:0:
   include/media/v4l2-fwnode.h:95:5: note: expected 'struct fwnode_handle *' but argument is of type 'int'
    int v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwn,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from include/linux/linkage.h:4:0,
                    from include/linux/kernel.h:6,
                    from include/linux/delay.h:10,
                    from drivers/media/i2c/tvp7002.c:23:
>> drivers/media/i2c/tvp7002.c:904:33: warning: passing argument 1 of 'v4l2_fwnode_endpoint_parse' makes pointer from integer without a cast [-Wint-conversion]
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
                                    ^
   include/linux/compiler.h:160:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^~~~
>> drivers/media/i2c/tvp7002.c:904:2: note: in expansion of macro 'if'
     if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
     ^~
   In file included from drivers/media/i2c/tvp7002.c:36:0:
   include/media/v4l2-fwnode.h:95:5: note: expected 'struct fwnode_handle *' but argument is of type 'int'
    int v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwn,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors
--
   In file included from drivers/media/i2c/ov2659.c:45:0:
   include/media/v4l2-fwnode.h:67:25: error: field 'base' has incomplete type
     struct fwnode_endpoint base;
                            ^~~~
   drivers/media/i2c/ov2659.c: In function 'ov2659_get_pdata':
>> drivers/media/i2c/ov2659.c:1359:45: error: implicit declaration of function 'of_fwnode_handle' [-Werror=implicit-function-declaration]
     bus_cfg = v4l2_fwnode_endpoint_alloc_parse(of_fwnode_handle(endpoint));
                                                ^~~~~~~~~~~~~~~~
>> drivers/media/i2c/ov2659.c:1359:45: warning: passing argument 1 of 'v4l2_fwnode_endpoint_alloc_parse' makes pointer from integer without a cast [-Wint-conversion]
   In file included from drivers/media/i2c/ov2659.c:45:0:
   include/media/v4l2-fwnode.h:97:30: note: expected 'struct fwnode_handle *' but argument is of type 'int'
    struct v4l2_fwnode_endpoint *v4l2_fwnode_endpoint_alloc_parse(
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors
--
   In file included from drivers/media/i2c/smiapp/smiapp-core.c:35:0:
   include/media/v4l2-fwnode.h:67:25: error: field 'base' has incomplete type
     struct fwnode_endpoint base;
                            ^~~~
   drivers/media/i2c/smiapp/smiapp-core.c: In function 'smiapp_get_hwconfig':
>> drivers/media/i2c/smiapp/smiapp-core.c:2790:30: error: implicit declaration of function 'device_fwnode_handle' [-Werror=implicit-function-declaration]
     struct fwnode_handle *fwn = device_fwnode_handle(dev);
                                 ^~~~~~~~~~~~~~~~~~~~
>> drivers/media/i2c/smiapp/smiapp-core.c:2790:30: warning: initialization makes pointer from integer without a cast [-Wint-conversion]
>> drivers/media/i2c/smiapp/smiapp-core.c:2797:7: error: implicit declaration of function 'fwnode_graph_get_next_endpoint' [-Werror=implicit-function-declaration]
     ep = fwnode_graph_get_next_endpoint(fwn, NULL);
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/media/i2c/smiapp/smiapp-core.c:2797:5: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
     ep = fwnode_graph_get_next_endpoint(fwn, NULL);
        ^
   cc1: some warnings being treated as errors

vim +/of_fwnode_handle +1012 drivers/media/i2c/tvp514x.c

  1006			return client->dev.platform_data;
  1007	
  1008		endpoint = of_graph_get_next_endpoint(client->dev.of_node, NULL);
  1009		if (!endpoint)
  1010			return NULL;
  1011	
> 1012		if (v4l2_fwnode_endpoint_parse(of_fwnode_handle(endpoint), &bus_cfg))
  1013			goto done;
  1014	
  1015		pdata = devm_kzalloc(&client->dev, sizeof(*pdata), GFP_KERNEL);

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

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux