linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, Linus Walleij <linus.walleij@linaro.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Carlo Caione <carlo@caione.org>,
	Jerome Brunet <jbrunet@baylibre.com>,
	linux-gpio@vger.kernel.org, linux-amlogic@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] pinctrl: meson: separate soc drivers
Date: Thu, 12 Oct 2017 14:09:29 +0800	[thread overview]
Message-ID: <201710121401.GSjGPWSz%fengguang.wu@intel.com> (raw)
In-Reply-To: <20171009101747.16940-2-jbrunet@baylibre.com>

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

Hi Jerome,

[auto build test ERROR on pinctrl/devel]
[also build test ERROR on v4.14-rc4 next-20171009]
[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/Jerome-Brunet/pinctrl-meson-prepare-for-new-SoC/20171012-094645
base:   https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git devel
config: i386-allmodconfig (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 error/warnings (new ones prefixed by >>):

   In file included from arch/x86/include/asm/bitops.h:513:0,
                    from include/linux/bitops.h:37,
                    from include/linux/kernel.h:10,
                    from include/linux/list.h:8,
                    from include/linux/kobject.h:20,
                    from include/linux/device.h:17,
                    from drivers/pinctrl/meson/pinctrl-meson.c:47:
>> arch/x86/include/asm/arch_hweight.h:17:17: error: expected identifier before string constant
    #define REG_OUT "a"
                    ^
>> drivers/pinctrl/meson/pinctrl-meson.h:74:2: note: in expansion of macro 'REG_OUT'
     REG_OUT,
     ^~~~~~~
   In file included from drivers/pinctrl/meson/pinctrl-meson.c:63:0:
>> drivers/pinctrl/meson/pinctrl-meson.h:99:29: error: 'NUM_REG' undeclared here (not in a function)
     struct meson_reg_desc regs[NUM_REG];
                                ^~~~~~~
   In file included from arch/x86/include/asm/bitops.h:513:0,
                    from include/linux/bitops.h:37,
                    from include/linux/kernel.h:10,
                    from include/linux/list.h:8,
                    from include/linux/kobject.h:20,
                    from include/linux/device.h:17,
                    from drivers/pinctrl/meson/pinctrl-meson.c:47:
   drivers/pinctrl/meson/pinctrl-meson.c: In function 'meson_gpio_direction_output':
>> arch/x86/include/asm/arch_hweight.h:17:17: error: incompatible type for argument 3 of 'meson_calc_reg_and_bit'
    #define REG_OUT "a"
                    ^
>> drivers/pinctrl/meson/pinctrl-meson.c:446:37: note: in expansion of macro 'REG_OUT'
     meson_calc_reg_and_bit(bank, gpio, REG_OUT, &reg, &bit);
                                        ^~~~~~~
   drivers/pinctrl/meson/pinctrl-meson.c:99:13: note: expected 'enum meson_reg_type' but argument is of type 'char *'
    static void meson_calc_reg_and_bit(struct meson_bank *bank, unsigned int pin,
                ^~~~~~~~~~~~~~~~~~~~~~
   In file included from arch/x86/include/asm/bitops.h:513:0,
                    from include/linux/bitops.h:37,
                    from include/linux/kernel.h:10,
                    from include/linux/list.h:8,
                    from include/linux/kobject.h:20,
                    from include/linux/device.h:17,
                    from drivers/pinctrl/meson/pinctrl-meson.c:47:
   drivers/pinctrl/meson/pinctrl-meson.c: In function 'meson_gpio_set':
>> arch/x86/include/asm/arch_hweight.h:17:17: error: incompatible type for argument 3 of 'meson_calc_reg_and_bit'
    #define REG_OUT "a"
                    ^
   drivers/pinctrl/meson/pinctrl-meson.c:462:37: note: in expansion of macro 'REG_OUT'
     meson_calc_reg_and_bit(bank, gpio, REG_OUT, &reg, &bit);
                                        ^~~~~~~
   drivers/pinctrl/meson/pinctrl-meson.c:99:13: note: expected 'enum meson_reg_type' but argument is of type 'char *'
    static void meson_calc_reg_and_bit(struct meson_bank *bank, unsigned int pin,
                ^~~~~~~~~~~~~~~~~~~~~~
   In file included from arch/x86/include/asm/bitops.h:513:0,
                    from include/linux/bitops.h:37,
                    from include/linux/kernel.h:10,
                    from include/linux/list.h:8,
                    from include/linux/kobject.h:20,
                    from include/linux/device.h:17,
                    from drivers/pinctrl/meson/pinctrl-meson.c:47:
   drivers/pinctrl/meson/pinctrl-meson.c: In function 'meson_gpio_get':
   arch/x86/include/asm/arch_hweight.h:16:16: error: incompatible type for argument 3 of 'meson_calc_reg_and_bit'
    #define REG_IN "a"
                   ^
>> drivers/pinctrl/meson/pinctrl-meson.c:478:37: note: in expansion of macro 'REG_IN'
     meson_calc_reg_and_bit(bank, gpio, REG_IN, &reg, &bit);
                                        ^~~~~~
   drivers/pinctrl/meson/pinctrl-meson.c:99:13: note: expected 'enum meson_reg_type' but argument is of type 'char *'
    static void meson_calc_reg_and_bit(struct meson_bank *bank, unsigned int pin,
                ^~~~~~~~~~~~~~~~~~~~~~

vim +/NUM_REG +99 drivers/pinctrl/meson/pinctrl-meson.h

6ac73095 Beniamino Galvani 2015-01-17   66  
6ac73095 Beniamino Galvani 2015-01-17   67  /**
6ac73095 Beniamino Galvani 2015-01-17   68   * enum meson_reg_type - type of registers encoded in @meson_reg_desc
6ac73095 Beniamino Galvani 2015-01-17   69   */
6ac73095 Beniamino Galvani 2015-01-17   70  enum meson_reg_type {
6ac73095 Beniamino Galvani 2015-01-17   71  	REG_PULLEN,
6ac73095 Beniamino Galvani 2015-01-17   72  	REG_PULL,
6ac73095 Beniamino Galvani 2015-01-17   73  	REG_DIR,
6ac73095 Beniamino Galvani 2015-01-17  @74  	REG_OUT,
6ac73095 Beniamino Galvani 2015-01-17   75  	REG_IN,
6ac73095 Beniamino Galvani 2015-01-17   76  	NUM_REG,
6ac73095 Beniamino Galvani 2015-01-17   77  };
6ac73095 Beniamino Galvani 2015-01-17   78  
6ac73095 Beniamino Galvani 2015-01-17   79  /**
6ac73095 Beniamino Galvani 2015-01-17   80   * struct meson bank
6ac73095 Beniamino Galvani 2015-01-17   81   *
6ac73095 Beniamino Galvani 2015-01-17   82   * @name:	bank name
6ac73095 Beniamino Galvani 2015-01-17   83   * @first:	first pin of the bank
6ac73095 Beniamino Galvani 2015-01-17   84   * @last:	last pin of the bank
6c9dc843 Jerome Brunet     2017-06-08   85   * @irq:	hwirq base number of the bank
6ac73095 Beniamino Galvani 2015-01-17   86   * @regs:	array of register descriptors
6ac73095 Beniamino Galvani 2015-01-17   87   *
6ac73095 Beniamino Galvani 2015-01-17   88   * A bank represents a set of pins controlled by a contiguous set of
6ac73095 Beniamino Galvani 2015-01-17   89   * bits in the domain registers. The structure specifies which bits in
6ac73095 Beniamino Galvani 2015-01-17   90   * the regmap control the different functionalities. Each member of
6ac73095 Beniamino Galvani 2015-01-17   91   * the @regs array refers to the first pin of the bank.
6ac73095 Beniamino Galvani 2015-01-17   92   */
6ac73095 Beniamino Galvani 2015-01-17   93  struct meson_bank {
6ac73095 Beniamino Galvani 2015-01-17   94  	const char *name;
6ac73095 Beniamino Galvani 2015-01-17   95  	unsigned int first;
6ac73095 Beniamino Galvani 2015-01-17   96  	unsigned int last;
6c9dc843 Jerome Brunet     2017-06-08   97  	int irq_first;
6c9dc843 Jerome Brunet     2017-06-08   98  	int irq_last;
6ac73095 Beniamino Galvani 2015-01-17  @99  	struct meson_reg_desc regs[NUM_REG];
6ac73095 Beniamino Galvani 2015-01-17  100  };
6ac73095 Beniamino Galvani 2015-01-17  101  

:::::: The code at line 99 was first introduced by commit
:::::: 6ac730951104a437bf828683bcf9ba66336c4fa7 pinctrl: add driver for Amlogic Meson SoCs

:::::: TO: Beniamino Galvani <b.galvani@gmail.com>
:::::: CC: Linus Walleij <linus.walleij@linaro.org>

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

  parent reply	other threads:[~2017-10-12  6:09 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-09 10:17 [PATCH 0/2] pinctrl: meson: prepare for new SoC Jerome Brunet
2017-10-09 10:17 ` [PATCH 1/2] pinctrl: meson: separate soc drivers Jerome Brunet
2017-10-09 11:35   ` Neil Armstrong
2017-10-09 13:11     ` Jerome Brunet
2017-10-11  2:55     ` Yixun Lan
2017-10-12  6:09   ` kbuild test robot [this message]
2017-10-12 11:12   ` kbuild test robot
2017-10-09 10:17 ` [PATCH 2/2] pinctrl: meson: rework pinmux ops Jerome Brunet
2017-10-09 11:42   ` Neil Armstrong
2017-10-09 13:25     ` Jerome Brunet

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=201710121401.GSjGPWSz%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=carlo@caione.org \
    --cc=jbrunet@baylibre.com \
    --cc=kbuild-all@01.org \
    --cc=khilman@baylibre.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /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;
as well as URLs for NNTP newsgroup(s).