From: kbuild test robot <lkp@intel.com>
To: Aditya Prayoga <aditya@kobol.io>
Cc: kbuild-all@01.org, linux-ide@vger.kernel.org,
linux-leds@vger.kernel.org, Jason Cooper <jason@lakedaemon.net>,
Andrew Lunn <andrew@lunn.ch>,
Gregory Clement <gregory.clement@bootlin.com>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
Russell King <linux@armlinux.org.uk>,
Jens Axboe <axboe@kernel.dk>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
Jacek Anaszewski <jacek.anaszewski@gmail.com>,
Pavel Machek <pavel@ucw.cz>, Daniel Golle <daniel@makrotopia.org>,
Aditya Prayoga <aditya@kobol.io>
Subject: Re: [PATCH 1/2] libata: add ledtrig support
Date: Thu, 20 Sep 2018 02:36:19 +0800 [thread overview]
Message-ID: <201809200252.JlgK7IS7%fengguang.wu@intel.com> (raw)
In-Reply-To: <1537328730-9156-2-git-send-email-aditya@kobol.io>
[-- Attachment #1: Type: text/plain, Size: 2905 bytes --]
Hi Daniel,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on block/for-next]
[also build test ERROR on v4.19-rc4 next-20180919]
[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/Aditya-Prayoga/Add-support-per-ATA-port-ledtrigger-on-armada-38x/20180920-002037
base: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-next
config: i386-randconfig-s1-201837 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
In file included from drivers/scsi/mvsas/mv_94xx.c:27:0:
>> drivers/scsi/mvsas/mv_94xx.h:294:2: error: redeclaration of enumerator 'LED_OFF'
LED_OFF = 0,
^~~~~~~
In file included from include/linux/libata.h:41:0,
from include/scsi/libsas.h:33,
from drivers/scsi/mvsas/mv_sas.h:43,
from drivers/scsi/mvsas/mv_94xx.c:26:
include/linux/leds.h:30:2: note: previous definition of 'LED_OFF' was here
LED_OFF = 0,
^~~~~~~
In file included from drivers/scsi/mvsas/mv_94xx.c:27:0:
>> drivers/scsi/mvsas/mv_94xx.h:295:2: error: redeclaration of enumerator 'LED_ON'
LED_ON = 1,
^~~~~~
In file included from include/linux/libata.h:41:0,
from include/scsi/libsas.h:33,
from drivers/scsi/mvsas/mv_sas.h:43,
from drivers/scsi/mvsas/mv_94xx.c:26:
include/linux/leds.h:31:2: note: previous definition of 'LED_ON' was here
LED_ON = 1,
^~~~~~
vim +/LED_OFF +294 drivers/scsi/mvsas/mv_94xx.h
c56f5f1d Wilfried Weissmann 2015-12-27 292
c56f5f1d Wilfried Weissmann 2015-12-27 293 enum sgpio_led_status {
c56f5f1d Wilfried Weissmann 2015-12-27 @294 LED_OFF = 0,
c56f5f1d Wilfried Weissmann 2015-12-27 @295 LED_ON = 1,
c56f5f1d Wilfried Weissmann 2015-12-27 296 LED_BLINKA = 2,
c56f5f1d Wilfried Weissmann 2015-12-27 297 LED_BLINKA_INV = 3,
c56f5f1d Wilfried Weissmann 2015-12-27 298 LED_BLINKA_SOF = 4,
c56f5f1d Wilfried Weissmann 2015-12-27 299 LED_BLINKA_EOF = 5,
c56f5f1d Wilfried Weissmann 2015-12-27 300 LED_BLINKB = 6,
c56f5f1d Wilfried Weissmann 2015-12-27 301 LED_BLINKB_INV = 7,
c56f5f1d Wilfried Weissmann 2015-12-27 302 };
c56f5f1d Wilfried Weissmann 2015-12-27 303
:::::: The code at line 294 was first introduced by commit
:::::: c56f5f1de3a6ab8ec985edbc358e1fd8d4e36a65 mvsas: Add SGPIO support to Marvell 94xx
:::::: TO: Wilfried Weissmann <Wilfried.Weissmann@gmx.at>
:::::: CC: Martin K. Petersen <martin.petersen@oracle.com>
---
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: 29628 bytes --]
next prev parent reply other threads:[~2018-09-19 18:36 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-19 3:45 [PATCH 0/2] Add support per ATA port ledtrigger on armada 38x Aditya Prayoga
2018-09-19 3:45 ` [PATCH 1/2] libata: add ledtrig support Aditya Prayoga
2018-09-19 13:01 ` Andrew Lunn
2018-09-20 9:53 ` Aditya Prayoga
2018-09-19 18:36 ` kbuild test robot [this message]
2018-09-19 21:49 ` kbuild test robot
2018-09-19 21:49 ` [PATCH] libata: fix semicolon.cocci warnings kbuild test robot
2018-09-20 7:23 ` [PATCH 1/2] libata: add ledtrig support Pavel Machek
2018-09-20 8:24 ` Daniel Golle
2018-09-20 22:04 ` Pavel Machek
2018-09-21 6:31 ` Daniel Golle
2018-09-19 3:45 ` [PATCH 2/2] ARM: mvebu: Enable ARCH_WANT_LIBATA_LEDS in Armada 38x Aditya Prayoga
2018-09-20 7:26 ` Pavel Machek
2018-09-26 6:05 ` Aditya Prayoga
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=201809200252.JlgK7IS7%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=aditya@kobol.io \
--cc=andrew@lunn.ch \
--cc=axboe@kernel.dk \
--cc=daniel@makrotopia.org \
--cc=gregory.clement@bootlin.com \
--cc=jacek.anaszewski@gmail.com \
--cc=jason@lakedaemon.net \
--cc=kbuild-all@01.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=pavel@ucw.cz \
--cc=sebastian.hesselbarth@gmail.com \
/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
all inboxes | Powered by JetHome®