[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <201806120747.RxEPI6T2%fengguang.wu@intel.com>
Date: Tue, 12 Jun 2018 07:08:32 +0800
From: kbuild test robot <lkp@...el.com>
To: Stefan Agner <stefan@...er.ch>
Cc: kbuild-all@...org, boris.brezillon@...tlin.com,
dwmw2@...radead.org, computersforpeace@...il.com,
marek.vasut@...il.com, robh+dt@...nel.org, mark.rutland@....com,
thierry.reding@...il.com, dev@...xeye.de,
miquel.raynal@...tlin.com, richard@....at, marcel@...wiler.com,
krzk@...nel.org, digetx@...il.com, benjamin.lindqvist@...ian.se,
jonathanh@...dia.com, pdeschrijver@...dia.com, pgaikwad@...dia.com,
mirza.krak@...il.com, gaireg@...reg.de,
linux-mtd@...ts.infradead.org, linux-tegra@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Stefan Agner <stefan@...er.ch>
Subject: Re: [PATCH v4 4/6] mtd: rawnand: add NVIDIA Tegra NAND Flash
controller driver
Hi Stefan,
I love your patch! Perhaps something to improve:
[auto build test WARNING on mtd/nand/next]
[also build test WARNING on v4.17 next-20180608]
[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/Stefan-Agner/mtd-rawnand-add-NVIDIA-Tegra-NAND-flash-support/20180612-045611
base: git://git.infradead.org/linux-mtd.git nand/next
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
drivers/mtd/nand/raw/tegra_nand.c:422:41: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:425:41: sparse: expression using sizeof(void)
>> drivers/mtd/nand/raw/tegra_nand.c:656:56: sparse: Using plain integer as NULL pointer
drivers/mtd/nand/raw/tegra_nand.c:667:56: sparse: Using plain integer as NULL pointer
drivers/mtd/nand/raw/tegra_nand.c:693:56: sparse: Using plain integer as NULL pointer
drivers/mtd/nand/raw/tegra_nand.c:759:48: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:759:48: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:762:24: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:762:24: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:792:56: sparse: Using plain integer as NULL pointer
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:814:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:818:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:823:15: sparse: expression using sizeof(void)
drivers/mtd/nand/raw/tegra_nand.c:823:15: sparse: expression using sizeof(void)
vim +656 drivers/mtd/nand/raw/tegra_nand.c
651
652 static int tegra_nand_read_page_raw(struct mtd_info *mtd,
653 struct nand_chip *chip,
654 uint8_t *buf, int oob_required, int page)
655 {
> 656 void *oob_buf = oob_required ? chip->oob_poi : 0;
657
658 return tegra_nand_page_xfer(mtd, chip, buf, oob_buf,
659 mtd->oobsize, page, true);
660 }
661
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Powered by blists - more mailing lists