Re: [PATCH v4 3/3] drm/tinydrm: Add the dummy versions of drm_of_find_backlight functions

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

 



On Sat, Sep 30, 2017 at 01:18:53PM +0800, kbuild test robot wrote:
> Hi Meghana,
> 
> [auto build test WARNING on drm/drm-next]
> [also build test WARNING on v4.14-rc2 next-20170929]
> [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/Meghana-Madhyastha/drm-tinydrm-drm_of_find_backlight-helper/20170930-122931
> base:   git://people.freedesktop.org/~airlied/linux.git drm-next
> config: x86_64-randconfig-x017-201739 (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=x86_64 
> 
> All warnings (new ones prefixed by >>):
> 
>    In file included from drivers/gpu//drm/tinydrm/mi0283qt.c:15:0:
>    include/drm/drm_of.h:74:1: error: expected identifier or '(' before '{' token
>     {
>     ^
>    include/drm/drm_of.h:80:1: error: expected identifier or '(' before '{' token
>     {
>     ^
>    include/drm/drm_of.h:72:40: warning: 'drm_of_find_backlight' declared 'static' but never defined [-Wunused-function]
>     static inline struct backlight_device *drm_of_find_backlight(
>                                            ^~~~~~~~~~~~~~~~~~~~~
> >> include/drm/drm_of.h:78:40: warning: 'devm_drm_of_find_backlight' used but never defined
>     static inline struct backlight_device *devm_drm_of_find_backlight(

You need to squash the dummy version into the patch that introduces the
first usage, otherwise compilation can fail and break bisecting. And looks
liek this doesn't compile. It takes a bit of playing around until you have
a Kconfig that hits this. Luckily 0day has constructed one for you
already.
-Daniel

>                                            ^~~~~~~~~~~~~~~~~~~~~~~~~~
> 
> vim +/devm_drm_of_find_backlight +78 include/drm/drm_of.h
> 
>     71	
>     72	static inline struct backlight_device *drm_of_find_backlight(
>     73						struct device *dev);
>   > 74	{
>     75		return -EINVAL;
>     76	}
>     77	
>   > 78	static inline struct backlight_device *devm_drm_of_find_backlight(
>     79							struct device *dev);
>     80	{
>     81		return -EINVAL;
>     82	}
>     83	#endif
>     84	
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation



-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux