linux-kernel - Re: [PATCH] drm/bridge/sii8620: fix display modes validation

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] [day] [month] [year] [list]

Date:   Fri, 26 Jan 2018 07:27:52 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Maciej Purski <m.purski@...sung.com>
Cc:     kbuild-all@...org, dri-devel@...ts.freedesktop.org,
        Archit Taneja <architt@...eaurora.org>,
        Andrzej Hajda <a.hajda@...sung.com>,
        Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
        David Airlie <airlied@...ux.ie>,
        Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
        Marek Szyprowski <m.szyprowski@...sung.com>,
        linux-kernel@...r.kernel.org, Maciej Purski <m.purski@...sung.com>
Subject: Re: [PATCH] drm/bridge/sii8620: fix display modes validation

Hi Maciej,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm/drm-next]
[also build test WARNING on next-20180119]
[cannot apply to v4.15-rc9]
[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/Maciej-Purski/drm-bridge-sii8620-fix-display-modes-validation/20180125-174703
base:   git://people.freedesktop.org/~airlied/linux.git drm-next
config: i386-randconfig-sb0-01260635 (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 

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/sil-sii8620.c: In function 'sii8620_mode_valid':
>> drivers/gpu/drm/bridge/sil-sii8620.c:2242:2: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
     return ret;
     ^

vim +/ret +2242 drivers/gpu/drm/bridge/sil-sii8620.c

  2220	
  2221	static enum drm_mode_status sii8620_mode_valid(struct drm_bridge *bridge,
  2222						 const struct drm_display_mode *mode)
  2223	{
  2224		enum drm_mode_status ret;
  2225		struct sii8620 *ctx = bridge_to_sii8620(bridge);
  2226		int pack_required = sii8620_is_packing_required(ctx, mode);
  2227		bool can_pack = ctx->devcap[MHL_DCAP_VID_LINK_MODE] &
  2228				MHL_DCAP_VID_LINK_PPIXEL;
  2229	
  2230		switch (pack_required) {
  2231		case -1:
  2232			ret = MODE_CLOCK_HIGH;
  2233			break;
  2234		case 1:
  2235			ret = (can_pack) ? MODE_OK : MODE_CLOCK_HIGH;
  2236			break;
  2237		case 0:
  2238			ret = MODE_OK;
  2239			break;
  2240		}
  2241	
> 2242		return ret;
  2243	}
  2244	

---
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" (32004 bytes)