Hi Andy,
I love your patch! Yet something to improve:
[auto build test ERROR on pinctrl/devel]
[also build test ERROR on v5.8-rc1 next-20200615]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see
https://stackoverflow.com/a/37406982]
url:
https://github.com/0day-ci/linux/commits/Andy-Shevchenko/pinctrl-intel-Re...
base:
https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git devel
config: x86_64-rhel-7.6 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-13) 9.3.0
reproduce (this is a W=1 build):
# save the attached .config to linux build tree
make W=1 ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
drivers/pinctrl/intel/pinctrl-intel.c: In function 'intel_restore_hostown':
> drivers/pinctrl/intel/pinctrl-intel.c:1601:2: error: implicit
declaration of function 'for_each_requested_gpio_in_range'
[-Werror=implicit-function-declaration]
1601 |
for_each_requested_gpio_in_range(&pctrl->chip, i, padgrp->gpio_base,
padgrp->size, dummy)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/pinctrl/intel/pinctrl-intel.c:1601:91: error: expected
';' before 'requested'
1601 |
for_each_requested_gpio_in_range(&pctrl->chip, i, padgrp->gpio_base,
padgrp->size, dummy)
|
^
|
;
1602 | requested |= BIT(i);
| ~~~~~~~~~
cc1: some warnings being treated as errors
vim +/for_each_requested_gpio_in_range +1601 drivers/pinctrl/intel/pinctrl-intel.c
1587
1588 static void intel_restore_hostown(struct intel_pinctrl *pctrl, unsigned int c,
1589 void __iomem *base, unsigned int gpp, u32 saved)
1590 {
1591 const struct intel_community *community = &pctrl->communities[c];
1592 const struct intel_padgroup *padgrp = &community->gpps[gpp];
1593 struct device *dev = pctrl->dev;
1594 const char *dummy;
1595 u32 requested = 0;
1596 unsigned int i;
1597
1598 if (padgrp->gpio_base == INTEL_GPIO_BASE_NOMAP)
1599 return;
1600
1601 for_each_requested_gpio_in_range(&pctrl->chip, i,
padgrp->gpio_base, padgrp->size, dummy)
1602 requested |= BIT(i);
1603
1604 if (!intel_gpio_update_reg(base + gpp * 4, requested, saved))
1605 return;
1606
1607 dev_dbg(dev, "restored hostown %u/%u %#08x\n", c, gpp, readl(base + gpp
* 4));
1608 }
1609
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org