linux-kernel - Re: [PATCH] gpu: anx7808: fix a missing check of return value

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 21 Dec 2018 07:48:40 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Kangjie Lu <kjlu@....edu>
Cc:     kbuild-all@...org, kjlu@....edu, pakki001@....edu,
        Archit Taneja <architt@...eaurora.org>,
        Andrzej Hajda <a.hajda@...sung.com>,
        Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
        David Airlie <airlied@...ux.ie>,
        Daniel Vetter <daniel@...ll.ch>,
        dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] gpu: anx7808: fix a missing check of return value

Hi Kangjie,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.20-rc7 next-20181220]
[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/Kangjie-Lu/gpu-anx7808-fix-a-missing-check-of-return-value/20181221-054313
config: nds32-allmodconfig (attached as .config)
compiler: nds32le-linux-gcc (GCC) 6.4.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=6.4.0 make.cross ARCH=nds32 

All errors (new ones prefixed by >>):

   drivers/gpu/drm/bridge/analogix-anx78xx.c: In function 'anx78xx_poweron':
>> drivers/gpu/drm/bridge/analogix-anx78xx.c:646:3: error: implicit declaration of function 'anx78xx_poweroff' [-Werror=implicit-function-declaration]
      anx78xx_poweroff(anx78xx);
      ^~~~~~~~~~~~~~~~
   drivers/gpu/drm/bridge/analogix-anx78xx.c: At top level:
   drivers/gpu/drm/bridge/analogix-anx78xx.c:655:13: warning: conflicting types for 'anx78xx_poweroff'
    static void anx78xx_poweroff(struct anx78xx *anx78xx)
                ^~~~~~~~~~~~~~~~
   drivers/gpu/drm/bridge/analogix-anx78xx.c:655:13: error: static declaration of 'anx78xx_poweroff' follows non-static declaration
   drivers/gpu/drm/bridge/analogix-anx78xx.c:646:3: note: previous implicit declaration of 'anx78xx_poweroff' was here
      anx78xx_poweroff(anx78xx);
      ^~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/anx78xx_poweroff +646 drivers/gpu/drm/bridge/analogix-anx78xx.c

   612	
   613	static int anx78xx_poweron(struct anx78xx *anx78xx)
   614	{
   615		struct anx78xx_platform_data *pdata = &anx78xx->pdata;
   616		int err = 0;
   617	
   618		if (WARN_ON(anx78xx->powered))
   619			return err;
   620	
   621		if (pdata->dvdd10) {
   622			err = regulator_enable(pdata->dvdd10);
   623			if (err) {
   624				DRM_ERROR("Failed to enable DVDD10 regulator: %d\n",
   625					  err);
   626				return err;
   627			}
   628	
   629			usleep_range(1000, 2000);
   630		}
   631	
   632		gpiod_set_value_cansleep(pdata->gpiod_reset, 1);
   633		usleep_range(1000, 2000);
   634	
   635		gpiod_set_value_cansleep(pdata->gpiod_pd, 0);
   636		usleep_range(1000, 2000);
   637	
   638		gpiod_set_value_cansleep(pdata->gpiod_reset, 0);
   639	
   640		/* Power on registers module */
   641		err = anx78xx_set_bits(anx78xx->map[I2C_IDX_TX_P2], SP_POWERDOWN_CTRL_REG,
   642				 SP_HDCP_PD | SP_AUDIO_PD | SP_VIDEO_PD | SP_LINK_PD);
   643		err |= anx78xx_clear_bits(anx78xx->map[I2C_IDX_TX_P2], SP_POWERDOWN_CTRL_REG,
   644				   SP_REGISTER_PD | SP_TOTAL_PD);
   645		if (err) { 
 > 646			anx78xx_poweroff(anx78xx);
   647			return err;
   648		}
   649	
   650		anx78xx->powered = true;
   651	
   652		return err;
   653	}
   654	

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

Download attachment ".config.gz" of type "application/gzip" (48511 bytes)