Re: [PATCH 3/5] arch: sh: kfr2r09: Use new renesas-ceu camera driver

From: kbuild test robot <lkp@intel.com>
To: Jacopo Mondi <jacopo+renesas@jmondi.org>
Cc: kbuild-all@01.org, hverkuil@xs4all.nl,
	laurent.pinchart@ideasonboard.com, sakari.ailus@iki.fi,
	mchehab@kernel.org, ysato@users.sourceforge.jp, dalias@libc.org,
	Jacopo Mondi <jacopo+renesas@jmondi.org>,
	linux-renesas-soc@vger.kernel.org, linux-media@vger.kernel.org,
	linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 3/5] arch: sh: kfr2r09: Use new renesas-ceu camera driver
Date: Thu, 31 May 2018 10:48:33 +0800	[thread overview]
Message-ID: <201805310808.IW00DJ10%fengguang.wu@intel.com> (raw)
In-Reply-To: <1527525431-22852-4-git-send-email-jacopo+renesas@jmondi.org>

[-- Attachment #1: Type: text/plain, Size: 2083 bytes --]

Hi Jacopo,

I love your patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v4.17-rc7 next-20180530]
[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/Jacopo-Mondi/Remove-sh_mobile_ceu_camera-from-arch-sh/20180530-060153
config: sh-kfr2r09_defconfig (attached as .config)
compiler: sh4-linux-gnu-gcc (Debian 7.2.0-11) 7.2.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
        make.cross ARCH=sh 

All warnings (new ones prefixed by >>):

   In file included from arch/sh/boards/mach-kfr2r09/setup.c:28:0:
>> include/linux/mtd/onenand.h:225:12: warning: 'struct mtd_oob_ops' declared inside parameter list will not be visible outside of this definition or declaration
        struct mtd_oob_ops *ops);
               ^~~~~~~~~~~

vim +225 include/linux/mtd/onenand.h

cd5f6346 Kyungmin Park     2005-07-11  223  
607d1cb1 Adrian Bunk       2008-04-14  224  int onenand_bbt_read_oob(struct mtd_info *mtd, loff_t from,
607d1cb1 Adrian Bunk       2008-04-14 @225  			 struct mtd_oob_ops *ops);
5988af23 Rohit Hagargundgi 2009-05-12  226  unsigned onenand_block(struct onenand_chip *this, loff_t addr);
5988af23 Rohit Hagargundgi 2009-05-12  227  loff_t onenand_addr(struct onenand_chip *this, int block);
5988af23 Rohit Hagargundgi 2009-05-12  228  int flexonenand_region(struct mtd_info *mtd, loff_t addr);
607d1cb1 Adrian Bunk       2008-04-14  229  

:::::: The code at line 225 was first introduced by commit
:::::: 607d1cb1042657177bf72247eeb85c0d8416bd51 [MTD] [OneNAND] proper onenand_bbt_read_oob() prototype

:::::: TO: Adrian Bunk <bunk@kernel.org>
:::::: CC: David Woodhouse <dwmw2@infradead.org>

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 11216 bytes --]