linux-kernel - Re: [PATCH v7 2/2] drm/bridge: Add I2C based driver for ps8640 bridge

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 for Android: free password hash cracker in your pocket

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

Date:	Thu, 14 Jan 2016 09:24:27 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Jitao Shi <jitao.shi@...iatek.com>
Cc:	kbuild-all@...org, Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>,
	David Airlie <airlied@...ux.ie>,
	Matthias Brugger <matthias.bgg@...il.com>,
	Jitao Shi <jitao.shi@...iatek.com>,
	Thierry Reding <treding@...dia.com>,
	Ajay Kumar <ajaykumar.rs@...sung.com>,
	Inki Dae <inki.dae@...sung.com>,
	Rahul Sharma <rahul.sharma@...sung.com>,
	Sean Paul <seanpaul@...omium.org>,
	Vincent Palatin <vpalatin@...omium.org>,
	Andy Yan <andy.yan@...k-chips.com>,
	Philipp Zabel <p.zabel@...gutronix.de>,
	Russell King <rmk+kernel@....linux.org.uk>,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	dri-devel@...ts.freedesktop.org,
	linux-arm-kernel@...ts.infradead.org,
	linux-mediatek@...ts.infradead.org, srv_heupstream@...iatek.com,
	Sascha Hauer <kernel@...gutronix.de>,
	yingjoe.chen@...iatek.com, eddie.huang@...iatek.com,
	cawa.cheng@...iatek.com, bibby.hsieh@...iatek.com,
	ck.hu@...iatek.com, stonea168@....com
Subject: Re: [PATCH v7 2/2] drm/bridge: Add I2C based driver for ps8640 bridge

Hi Jitao,

[auto build test WARNING on v4.4-rc8]
[also build test WARNING on next-20160113]
[cannot apply to drm/drm-next]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Jitao-Shi/Documentation-bridge-Add-documentation-for-ps8640-DT-properties/20160113-102004
config: i386-randconfig-sb0-01140840 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/bridge/parade-ps8640.c: In function 'ps8640_update_fw_store':
>> drivers/gpu/drm/bridge/parade-ps8640.c:788:20: warning: 'progress_cnt' may be used uninitialized in this function [-Wmaybe-uninitialized]
      if (progress_cnt != (pos * 100) / fw->size) {
                       ^
   drivers/gpu/drm/bridge/parade-ps8640.c:770:5: note: 'progress_cnt' was declared here
     u8 progress_cnt, rom_page_id[2];
        ^

vim +/progress_cnt +788 drivers/gpu/drm/bridge/parade-ps8640.c

   772	
   773		ps8640_regw_byte(client2, PAGE2_SPI_CFG3, I2C_TO_SPI_RESET);
   774		msleep(100);
   775		ps8640_regw_byte(client2, PAGE2_SPI_CFG3, 0x00);
   776	
   777		while (pos < fw->size) {
   778			rom_page_id[0] = (pos >> 8) & 0xFF;
   779			rom_page_id[1] = (pos >> 16) & 0xFF;
   780			ret = ps8640_regw(client2, PAGE2_ROMADD_BYTE1, rom_page_id, 2);
   781			if (ret)
   782				goto error;
   783			ret = ps8640_regw(client7, 0, fw->data + pos, 256);
   784			if (ret)
   785				goto error;
   786	
   787			pos += 256;
 > 788			if (progress_cnt != (pos * 100) / fw->size) {
   789				progress_cnt = (pos * 100) / fw->size;
   790				dev_info(dev, "fw update progress percent %d\n",
   791					 progress_cnt);
   792			}
   793		}
   794		return 0;
   795	
   796	error:

---
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/octet-stream" (22464 bytes)