On Tue, 2017-06-20 at 04:21 +0800, kbuild test robot wrote: > Hi Vivek, > > [auto build test WARNING on pza/reset/next] > [also build test WARNING on v4.12-rc6 next-20170619] > [cannot apply to balbi-usb/next] > [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/reset-Add-APIs-to-manage-array-of-resets/20170620-021320 > base: git://git.pengutronix.de/git/pza/linux reset/next > config: i386-randconfig-c0-06200218 (attached as .config) > compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4 > reproduce: > # save the attached .config to linux build tree > make ARCH=i386 > > All warnings (new ones prefixed by >>): > > In file included from drivers/gpu//drm/nouveau/include/nvif/os.h:28:0, > from drivers/gpu//drm/nouveau/include/nvkm/core/os.h:3, > from drivers/gpu//drm/nouveau/include/nvkm/core/event.h:3, > from drivers/gpu//drm/nouveau/include/nvkm/core/device.h:3, > from drivers/gpu//drm/nouveau/include/nvkm/core/subdev.h:3, > from drivers/gpu//drm/nouveau/include/nvkm/core/engine.h:4, > from drivers/gpu//drm/nouveau/include/nvkm/engine/fifo.h:3, > from drivers/gpu//drm/nouveau/nvkm/engine/fifo/priv.h:4, > from drivers/gpu//drm/nouveau/nvkm/engine/fifo/chan.h:4, > from drivers/gpu//drm/nouveau/nvkm/engine/fifo/channv50.h:4, > from drivers/gpu//drm/nouveau/nvkm/engine/fifo/chang84.c:24: > >> include/linux/reset.h:110:37: warning: 'struct reset_control_array' declared inside parameter list > void reset_control_array_put(struct reset_control_array *resets) > ^ > >> include/linux/reset.h:110:37: warning: its scope is only this definition or declaration, which is probably not what you want > > vim +110 include/linux/reset.h > > 94 return optional ? NULL : ERR_PTR(-ENOTSUPP); > 95 } > 96 > 97 static inline struct reset_control * > 98 devm_reset_control_array_get(struct device *dev, bool shared, bool optional) > 99 { > 100 return optional ? NULL : ERR_PTR(-ENOTSUPP); > 101 } > 102 > 103 static inline struct reset_control * > 104 of_reset_control_array_get(struct device_node *np, bool shared, bool optional) > 105 { > 106 return optional ? NULL : ERR_PTR(-ENOTSUPP); > 107 } > 108 > 109 static inline > > 110 void reset_control_array_put(struct reset_control_array *resets) > 111 { > 112 } reset_control_array_put is static now, I forgot to remove this stub. regards * Philipp -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
