mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Scott Matheina <scott@matheina.com>
Cc: kbuild-all@01.org, linux-kernel@vger.kernel.org,
	Scott Matheina <scott@matheina.com>,
	Jonathan Cameron <jic23@kernel.org>,
	Hartmut Knaack <knaack.h@gmx.de>,
	Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
	linux-iio@vger.kernel.org, devel@driverdev.osuosl.org
Subject: Re: [PATCHv4 2/8] Fixed variables not being consistently lower case
Date: Mon, 26 Dec 2016 04:31:23 +0800	[thread overview]
Message-ID: <201612260448.SvN0rjhT%fengguang.wu@intel.com> (raw)
In-Reply-To: <1482696216-8013-3-git-send-email-scott@matheina.com>

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

Hi Scott,

[auto build test ERROR on iio/togreg]
[also build test ERROR on v4.9 next-20161224]
[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/Scott-Matheina/Checkpatch-fixes-to-driver-staging-iio-addac/20161226-041021
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
config: i386-randconfig-x001-201652 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/staging/iio/addac/adt7316.c: In function 'adt7316_event_handler':
>> drivers/staging/iio/addac/adt7316.c:1758:12: error: implicit declaration of function 'iio_unmod_event_code' [-Werror=implicit-function-declaration]
               iio_unmod_event_code(iio_temp, 0,
               ^~~~~~~~~~~~~~~~~~~~
>> drivers/staging/iio/addac/adt7316.c:1758:33: error: 'iio_temp' undeclared (first use in this function)
               iio_unmod_event_code(iio_temp, 0,
                                    ^~~~~~~~
   drivers/staging/iio/addac/adt7316.c:1758:33: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/staging/iio/addac/adt7316.c:1759:12: error: 'iio_ev_type_thresh' undeclared (first use in this function)
               iio_ev_type_thresh,
               ^~~~~~~~~~~~~~~~~~
>> drivers/staging/iio/addac/adt7316.c:1760:12: error: 'iio_ev_dir_rising' undeclared (first use in this function)
               iio_ev_dir_rising),
               ^~~~~~~~~~~~~~~~~
>> drivers/staging/iio/addac/adt7316.c:1766:12: error: 'iio_ev_dir_falling' undeclared (first use in this function)
               iio_ev_dir_falling),
               ^~~~~~~~~~~~~~~~~~
>> drivers/staging/iio/addac/adt7316.c:1782:33: error: 'iio_voltage' undeclared (first use in this function)
               iio_unmod_event_code(iio_voltage, 1,
                                    ^~~~~~~~~~~
>> drivers/staging/iio/addac/adt7316.c:1784:12: error: 'iio_ev_dir_either' undeclared (first use in this function)
               iio_ev_dir_either),
               ^~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/iio_unmod_event_code +1758 drivers/staging/iio/addac/adt7316.c

  1752			if ((chip->id & ID_FAMILY_MASK) != ID_ADT75XX)
  1753				stat1 &= 0x1F;
  1754	
  1755			time = iio_get_time_ns(indio_dev);
  1756			if (stat1 & BIT(0))
  1757				iio_push_event(indio_dev,
> 1758					       iio_unmod_event_code(iio_temp, 0,
> 1759								    iio_ev_type_thresh,
> 1760								    iio_ev_dir_rising),
  1761					       time);
  1762			if (stat1 & BIT(1))
  1763				iio_push_event(indio_dev,
  1764					       iio_unmod_event_code(iio_temp, 0,
  1765								    iio_ev_type_thresh,
> 1766								    iio_ev_dir_falling),
  1767					       time);
  1768			if (stat1 & BIT(2))
  1769				iio_push_event(indio_dev,
  1770					       iio_unmod_event_code(iio_temp, 1,
  1771								    iio_ev_type_thresh,
  1772								    iio_ev_dir_rising),
  1773					       time);
  1774			if (stat1 & BIT(3))
  1775				iio_push_event(indio_dev,
  1776					       iio_unmod_event_code(iio_temp, 1,
  1777								    iio_ev_type_thresh,
  1778								    iio_ev_dir_falling),
  1779					       time);
  1780			if (stat1 & BIT(5))
  1781				iio_push_event(indio_dev,
> 1782					       iio_unmod_event_code(iio_voltage, 1,
  1783								    iio_ev_type_thresh,
> 1784								    iio_ev_dir_either),
  1785					       time);
  1786			if (stat1 & BIT(6))
  1787				iio_push_event(indio_dev,

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

  reply	other threads:[~2016-12-25 20:24 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-25 19:56 [PATCHv4 0/8] Checkpatch fixes to driver/staging/iio/addac Scott Matheina
2016-12-25 19:56 ` [PATCHv4 1/8] fixed long description text exceeding 80 characters Scott Matheina
2016-12-30 18:40   ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 2/8] Fixed variables not being consistently lower case Scott Matheina
2016-12-25 20:31   ` kbuild test robot [this message]
2016-12-25 19:56 ` [PATCHv4 3/8] Fix camel case issues Scott Matheina
2016-12-25 20:54   ` kbuild test robot
2016-12-30 18:45   ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 4/8] Fix braces not present on all arms of if else statement Scott Matheina
2016-12-30 18:46   ` Jonathan Cameron
2016-12-30 18:48     ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 5/8] Remove line after closing braces Scott Matheina
2016-12-25 19:56 ` [PATCHv4 6/8] Fixed code wrap alignment with preceding ( Scott Matheina
2016-12-30 18:51   ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 7/8] Changed code to align with coding style of using octat Scott Matheina
2016-12-25 20:41   ` kbuild test robot
2016-12-30 18:53   ` Jonathan Cameron
2016-12-25 19:56 ` [PATCHv4 8/8] Fixes style issues due to mis-aligned carry over lines Scott Matheina
2016-12-25 21:18   ` kbuild test robot
2016-12-30 18:56   ` Jonathan Cameron
  -- strict thread matches above, loose matches on Subject: below --
2016-12-25 19:41 [PATCHv4 0/8] Checkpatch fixes to driver/staging/iio/addac Scott Matheina
2016-12-25 19:41 ` [PATCHv4 2/8] Fixed variables not being consistently lower case Scott Matheina
2016-12-25 20:14   ` Al Viro
2016-12-25 22:34     ` Jonathan Cameron
2016-12-26  0:13       ` Al Viro

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=201612260448.SvN0rjhT%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=jic23@kernel.org \
    --cc=kbuild-all@01.org \
    --cc=knaack.h@gmx.de \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pmeerw@pmeerw.net \
    --cc=scott@matheina.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®