Re: [PATCH v8 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller

From: kbuild test robot <lkp@intel.com>
To: Jianxin Pan <jianxin.pan@amlogic.com>
Cc: Victor Wan <victor.wan@amlogic.com>,
	Neil Armstrong <narmstrong@baylibre.com>,
	Jian Hu <jian.hu@amlogic.com>,
	linux-mtd@lists.infradead.org,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	Jerome Brunet <jbrunet@baylibre.com>,
	Rob Herring <robh@kernel.org>,
	Richard Weinberger <richard@nod.at>,
	Marek Vasut <marek.vasut@gmail.com>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
	Yixun Lan <yixun.lan@amlogic.com>,
	Boris Brezillon <boris.brezillon@bootlin.com>,
	linux-amlogic@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	Hanjie Lin <hanjie.lin@amlogic.com>,
	Jianxin Pan <jianxin.pan@amlogic.com>,
	linux-kernel@vger.kernel.org, Liang Yang <liang.yang@amlogic.com>,
	kbuild-all@01.org, Kevin Hilman <khilman@baylibre.com>,
	Carlo Caione <carlo@caione.org>,
	Brian Norris <computersforpeace@gmail.com>,
	David Woodhouse <dwmw2@infradead.org>
Subject: Re: [PATCH v8 2/2] mtd: rawnand: meson: add support for Amlogic NAND flash controller
Date: Tue, 18 Dec 2018 23:57:53 +0800	[thread overview]
Message-ID: <201812182347.rUfXa3YR%fengguang.wu@intel.com> (raw)
In-Reply-To: <1544973463-59523-3-git-send-email-jianxin.pan@amlogic.com>

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

Hi Liang,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on mtd/nand/next]
[also build test ERROR on v4.20-rc7 next-20181218]
[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/Jianxin-Pan/dt-bindings-nand-meson-add-Amlogic-NAND-controller-driver/20181217-070347
base:   git://git.infradead.org/linux-mtd.git nand/next
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-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
        GCC_VERSION=7.2.0 make.cross ARCH=arm 

All errors (new ones prefixed by >>):

   drivers/mtd/nand/raw/meson_nand.c: In function 'meson_nfc_is_buffer_dma_safe':
>> drivers/mtd/nand/raw/meson_nand.c:833:35: error: implicit declaration of function 'object_is_on_stack'; did you mean 'do_softirq_own_stack'? [-Werror=implicit-function-declaration]
     if (virt_addr_valid(buffer) && (!object_is_on_stack(buffer)))
                                      ^~~~~~~~~~~~~~~~~~
                                      do_softirq_own_stack
   cc1: some warnings being treated as errors

vim +833 drivers/mtd/nand/raw/meson_nand.c

   830	
   831	bool meson_nfc_is_buffer_dma_safe(const void *buffer)
   832	{
 > 833		if (virt_addr_valid(buffer) && (!object_is_on_stack(buffer)))
   834			return true;
   835		return false;
   836	}
   837	

---
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: 67803 bytes --]

[-- Attachment #3: Type: text/plain, Size: 167 bytes --]

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic