Hi Philipp, I love your patch! Yet something to improve: [auto build test ERROR on sof-driver-fuweitax/master] [also build test ERROR on v4.20-rc1 next-20181105] [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/Philipp-Zabel/drm-imx-Allow-building-under-COMPILE_TEST/20181106-050753 base: https://github.com/fuweitax/linux master config: ia64-allmodconfig (attached as .config) compiler: ia64-linux-gcc (GCC) 8.1.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=8.1.0 make.cross ARCH=ia64 All error/warnings (new ones prefixed by >>): >> drivers/gpu/drm/imx/imx-tve.c:123:16: error: field 'clk_hw_di' has incomplete type struct clk_hw clk_hw_di; ^~~~~~~~~ In file included from include/linux/err.h:5, from include/linux/clk.h:15, from drivers/gpu/drm/imx/imx-tve.c:16: drivers/gpu/drm/imx/imx-tve.c: In function 'clk_tve_di_recalc_rate': include/linux/kernel.h:962:32: error: dereferencing pointer to incomplete type 'struct clk_hw' BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \ ^~~~~~ include/linux/compiler.h:316:19: note: in definition of macro '__compiletime_assert' bool __cond = !(condition); \ ^~~~~~~~~ include/linux/compiler.h:339:2: note: in expansion of macro '_compiletime_assert' _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__) ^~~~~~~~~~~~~~~~~~~ include/linux/build_bug.h:45:37: note: in expansion of macro 'compiletime_assert' #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) ^~~~~~~~~~~~~~~~~~ include/linux/kernel.h:962:2: note: in expansion of macro 'BUILD_BUG_ON_MSG' BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \ ^~~~~~~~~~~~~~~~ include/linux/kernel.h:962:20: note: in expansion of macro '__same_type' BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \ ^~~~~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:384:24: note: in expansion of macro 'container_of' struct imx_tve *tve = container_of(hw, struct imx_tve, clk_hw_di); ^~~~~~~~~~~~ drivers/gpu/drm/imx/imx-tve.c: At top level: >> drivers/gpu/drm/imx/imx-tve.c:445:15: error: variable 'clk_tve_di_ops' has initializer but incomplete type static struct clk_ops clk_tve_di_ops = { ^~~~~~~ >> drivers/gpu/drm/imx/imx-tve.c:446:3: error: 'struct clk_ops' has no member named 'round_rate' .round_rate = clk_tve_di_round_rate, ^~~~~~~~~~ >> drivers/gpu/drm/imx/imx-tve.c:446:16: warning: excess elements in struct initializer .round_rate = clk_tve_di_round_rate, ^~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:446:16: note: (near initialization for 'clk_tve_di_ops') >> drivers/gpu/drm/imx/imx-tve.c:447:3: error: 'struct clk_ops' has no member named 'set_rate' .set_rate = clk_tve_di_set_rate, ^~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:447:14: warning: excess elements in struct initializer .set_rate = clk_tve_di_set_rate, ^~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:447:14: note: (near initialization for 'clk_tve_di_ops') >> drivers/gpu/drm/imx/imx-tve.c:448:3: error: 'struct clk_ops' has no member named 'recalc_rate' .recalc_rate = clk_tve_di_recalc_rate, ^~~~~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:448:17: warning: excess elements in struct initializer .recalc_rate = clk_tve_di_recalc_rate, ^~~~~~~~~~~~~~~~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:448:17: note: (near initialization for 'clk_tve_di_ops') drivers/gpu/drm/imx/imx-tve.c: In function 'tve_clk_init': >> drivers/gpu/drm/imx/imx-tve.c:454:9: error: variable 'init' has initializer but incomplete type struct clk_init_data init = { ^~~~~~~~~~~~~ >> drivers/gpu/drm/imx/imx-tve.c:455:4: error: 'struct clk_init_data' has no member named 'name' .name = "tve_di", ^~~~ drivers/gpu/drm/imx/imx-tve.c:455:11: warning: excess elements in struct initializer .name = "tve_di", ^~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:455:11: note: (near initialization for 'init') >> drivers/gpu/drm/imx/imx-tve.c:456:4: error: 'struct clk_init_data' has no member named 'ops' .ops = &clk_tve_di_ops, ^~~ drivers/gpu/drm/imx/imx-tve.c:456:10: warning: excess elements in struct initializer .ops = &clk_tve_di_ops, ^ drivers/gpu/drm/imx/imx-tve.c:456:10: note: (near initialization for 'init') >> drivers/gpu/drm/imx/imx-tve.c:457:4: error: 'struct clk_init_data' has no member named 'num_parents' .num_parents = 1, ^~~~~~~~~~~ drivers/gpu/drm/imx/imx-tve.c:457:18: warning: excess elements in struct initializer .num_parents = 1, ^ drivers/gpu/drm/imx/imx-tve.c:457:18: note: (near initialization for 'init') >> drivers/gpu/drm/imx/imx-tve.c:458:4: error: 'struct clk_init_data' has no member named 'flags' .flags = 0, ^~~~~ drivers/gpu/drm/imx/imx-tve.c:458:12: warning: excess elements in struct initializer .flags = 0, ^ drivers/gpu/drm/imx/imx-tve.c:458:12: note: (near initialization for 'init') >> drivers/gpu/drm/imx/imx-tve.c:454:23: error: storage size of 'init' isn't known struct clk_init_data init = { ^~~~ >> drivers/gpu/drm/imx/imx-tve.c:461:21: error: implicit declaration of function '__clk_get_name'; did you mean 'clk_get_rate'? [-Werror=implicit-function-declaration] tve_di_parent[0] = __clk_get_name(tve->clk); ^~~~~~~~~~~~~~ clk_get_rate >> drivers/gpu/drm/imx/imx-tve.c:461:19: warning: assignment to 'const char *' from 'int' makes pointer from integer without a cast [-Wint-conversion] tve_di_parent[0] = __clk_get_name(tve->clk); ^ >> drivers/gpu/drm/imx/imx-tve.c:465:16: error: implicit declaration of function 'clk_register'; did you mean 'bus_register'? [-Werror=implicit-function-declaration] tve->di_clk = clk_register(tve->dev, &tve->clk_hw_di); ^~~~~~~~~~~~ bus_register drivers/gpu/drm/imx/imx-tve.c:454:23: warning: unused variable 'init' [-Wunused-variable] struct clk_init_data init = { ^~~~ drivers/gpu/drm/imx/imx-tve.c: At top level: >> drivers/gpu/drm/imx/imx-tve.c:445:23: error: storage size of 'clk_tve_di_ops' isn't known static struct clk_ops clk_tve_di_ops = { ^~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +/clk_hw_di +123 drivers/gpu/drm/imx/imx-tve.c fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 107 fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 108 struct imx_tve { fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 109 struct drm_connector connector; 49f98bc4 drivers/gpu/drm/imx/imx-tve.c Philipp Zabel 2016-07-06 110 struct drm_encoder encoder; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 111 struct device *dev; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 112 spinlock_t lock; /* register lock */ fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 113 bool enabled; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 114 int mode; 49f98bc4 drivers/gpu/drm/imx/imx-tve.c Philipp Zabel 2016-07-06 115 int di_hsync_pin; 49f98bc4 drivers/gpu/drm/imx/imx-tve.c Philipp Zabel 2016-07-06 116 int di_vsync_pin; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 117 fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 118 struct regmap *regmap; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 119 struct regulator *dac_reg; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 120 struct i2c_adapter *ddc; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 121 struct clk *clk; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 122 struct clk *di_sel_clk; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 @123 struct clk_hw clk_hw_di; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 124 struct clk *di_clk; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 125 }; fcbc51e5 drivers/staging/imx-drm/imx-tve.c Philipp Zabel 2013-04-08 126 :::::: The code at line 123 was first introduced by commit :::::: fcbc51e54d2aa9d402206601f4894251049e5d77 staging: drm/imx: Add support for Television Encoder (TVEv2) :::::: TO: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> :::::: CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
Attachment:
.config.gz
Description: application/gzip
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
