public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: arvindY <arvind.yadav.cs@gmail.com>
To: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, nsekhar@ti.com, khilman@kernel.org,
	linux@armlinux.org.uk, kaloz@openwrt.org, khalasa@piap.pl,
	aaro.koskinen@iki.fi, tony@atomide.com, jason@lakedaemon.net,
	andrew@lunn.ch, sebastian.hesselbarth@gmail.com,
	gregory.clement@free-electrons.com, daniel@zonque.org,
	haojian.zhuang@gmail.com, robert.jarzmik@free.fr,
	marek.vasut@gmail.com, slapin@ossfans.org, jic23@cam.ac.uk,
	kgene@kernel.org, krzk@kernel.org, ralf@linux-mips.org,
	ysato@users.sourceforge.jp, dalias@libc.org, tglx@linutronix.de,
	mingo@redhat.com, hpa@zytor.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 11/11] ARM: pxa: constify gpio_led
Date: Mon, 25 Dec 2017 18:34:24 +0530	[thread overview]
Message-ID: <5A40F758.6060101@gmail.com> (raw)
In-Reply-To: <201712250154.QftmBzUp%fengguang.wu@intel.com>

Hi,

Sorry for trouble, I will send updated patch.

On Sunday 24 December 2017 11:24 PM, kbuild test robot wrote:
> Hi Arvind,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on arm-soc/for-next]
> [also build test ERROR on v4.15-rc5 next-20171222]
> [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/Arvind-Yadav/constify-gpio_led/20171223-142117
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc.git for-next
> config: arm-pxa_defconfig (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
>          make.cross ARCH=arm
>
> All error/warnings (new ones prefixed by >>):
>
>>> arch/arm/mach-pxa/balloon3.c:390:8: error: expected '{' before 'const'
>      struct const gpio_led balloon3_gpio_leds[] __initconst = {
>             ^~~~~
>>> arch/arm/mach-pxa/balloon3.c:405:11: error: 'balloon3_gpio_leds' undeclared here (not in a function); did you mean 'balloon3_gpio_led_info'?
>       .leds  = balloon3_gpio_leds,
>                ^~~~~~~~~~~~~~~~~~
>                balloon3_gpio_led_info
>     In file included from include/linux/kernel.h:15:0,
>                      from include/linux/list.h:9,
>                      from include/linux/kobject.h:20,
>                      from include/linux/device.h:17,
>                      from include/linux/platform_device.h:14,
>                      from arch/arm/mach-pxa/balloon3.c:18:
>>> include/linux/build_bug.h:30:45: error: bit-field '<anonymous>' width not an integer constant
>      #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
>                                                  ^
>     include/linux/compiler-gcc.h:65:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
>      #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
>                                 ^~~~~~~~~~~~~~~~~
>     include/linux/kernel.h:71:59: note: in expansion of macro '__must_be_array'
>      #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
>                                                                ^~~~~~~~~~~~~~~
>>> arch/arm/mach-pxa/balloon3.c:406:14: note: in expansion of macro 'ARRAY_SIZE'
>       .num_leds = ARRAY_SIZE(balloon3_gpio_leds),
>                   ^~~~~~~~~~
> --
>>> arch/arm/mach-pxa/palmld.c:249:8: error: expected '{' before 'const'
>      struct const gpio_led gpio_leds[] __initconst = {
>             ^~~~~
>>> arch/arm/mach-pxa/palmld.c:262:11: error: 'gpio_leds' undeclared here (not in a function); did you mean 'gpio_led'?
>       .leds  = gpio_leds,
>                ^~~~~~~~~
>                gpio_led
>     In file included from include/linux/kernel.h:15:0,
>                      from include/linux/list.h:9,
>                      from include/linux/kobject.h:20,
>                      from include/linux/device.h:17,
>                      from include/linux/platform_device.h:14,
>                      from arch/arm/mach-pxa/palmld.c:17:
>>> include/linux/build_bug.h:30:45: error: bit-field '<anonymous>' width not an integer constant
>      #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
>                                                  ^
>     include/linux/compiler-gcc.h:65:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
>      #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
>                                 ^~~~~~~~~~~~~~~~~
>     include/linux/kernel.h:71:59: note: in expansion of macro '__must_be_array'
>      #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
>                                                                ^~~~~~~~~~~~~~~
>>> arch/arm/mach-pxa/palmld.c:263:14: note: in expansion of macro 'ARRAY_SIZE'
>       .num_leds = ARRAY_SIZE(gpio_leds),
>                   ^~~~~~~~~~
>
> vim +390 arch/arm/mach-pxa/balloon3.c
>
>     389	
>   > 390	struct const gpio_led balloon3_gpio_leds[] __initconst = {
>     391		{
>     392			.name			= "balloon3:green:idle",
>     393			.default_trigger	= "heartbeat",
>     394			.gpio			= BALLOON3_GPIO_LED_IDLE,
>     395			.active_low		= 1,
>     396		}, {
>     397			.name			= "balloon3:green:nand",
>     398			.default_trigger	= "nand-disk",
>     399			.gpio			= BALLOON3_GPIO_LED_NAND,
>     400			.active_low		= 1,
>     401		},
>     402	};
>     403	
>     404	static struct gpio_led_platform_data balloon3_gpio_led_info = {
>   > 405		.leds		= balloon3_gpio_leds,
>   > 406		.num_leds	= ARRAY_SIZE(balloon3_gpio_leds),
>     407	};
>     408	
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

  reply	other threads:[~2017-12-25 13:04 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-20  8:47 [PATCH 00/11] constify gpio_led Arvind Yadav
2017-12-20  8:47 ` [PATCH 01/11] MIPS: Alchemy: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 02/11] MIPS: AR7: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 03/11] MIPS: TXX9: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 04/11] x86: geode: " Arvind Yadav
2017-12-23  7:50   ` kbuild test robot
2017-12-20  8:47 ` [PATCH 05/11] sh: mach-rsk: rsk7203: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 06/11] ARM: davinci: " Arvind Yadav
2017-12-23 10:01   ` Sekhar Nori
2017-12-20  8:47 ` [PATCH 07/11] ARM: ixp4xx: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 08/11] ARM: OMAP1: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 09/11] ARM: orion5x: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 10/11] ARM: s3c24xx: " Arvind Yadav
2017-12-20  8:47 ` [PATCH 11/11] ARM: pxa: " Arvind Yadav
2017-12-24 17:54   ` kbuild test robot
2017-12-25 13:04     ` arvindY [this message]
2017-12-23 10:13 ` [PATCH 00/11] " Russell King - ARM Linux
2017-12-25 13:07   ` arvindY

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5A40F758.6060101@gmail.com \
    --to=arvind.yadav.cs@gmail.com \
    --cc=aaro.koskinen@iki.fi \
    --cc=andrew@lunn.ch \
    --cc=dalias@libc.org \
    --cc=daniel@zonque.org \
    --cc=gregory.clement@free-electrons.com \
    --cc=haojian.zhuang@gmail.com \
    --cc=hpa@zytor.com \
    --cc=jason@lakedaemon.net \
    --cc=jic23@cam.ac.uk \
    --cc=kaloz@openwrt.org \
    --cc=kbuild-all@01.org \
    --cc=kgene@kernel.org \
    --cc=khalasa@piap.pl \
    --cc=khilman@kernel.org \
    --cc=krzk@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=lkp@intel.com \
    --cc=marek.vasut@gmail.com \
    --cc=mingo@redhat.com \
    --cc=nsekhar@ti.com \
    --cc=ralf@linux-mips.org \
    --cc=robert.jarzmik@free.fr \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=slapin@ossfans.org \
    --cc=tglx@linutronix.de \
    --cc=tony@atomide.com \
    --cc=ysato@users.sourceforge.jp \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox