[pinctrl:for-next 54/63] drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: error: initialization from incompatible pointer type

[pinctrl:for-next 54/63] drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: error: initialization from incompatible pointer type

[kbuild test robot]

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

tree:   https://kernel.googlesource.com/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git for-next
head:   2295f64645cbc4f9ca3002f12d1b36cff3d04ac5
commit: 21b2920fb587e570b43973300a11b921c3a61d3e [54/63] pinctrl: aspeed: g6: Remove const specifier from aspeed_g6_sig_expr_set's ctx parameter
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 21b2920fb587e570b43973300a11b921c3a61d3e
        # save the attached .config to linux build tree
        GCC_VERSION=7.4.0 make.cross ARCH=arm 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
     .set = aspeed_g6_sig_expr_set,
            ^~~~~~~~~~~~~~~~~~~~~~
   drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: note: (near initialization for 'aspeed_g5_ops.set')
   cc1: some warnings being treated as errors

vim +2325 drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c

2eda1cdec49f8a Andrew Jeffery 2019-07-11  2323  
2eda1cdec49f8a Andrew Jeffery 2019-07-11  2324  static const struct aspeed_pinmux_ops aspeed_g5_ops = {
2eda1cdec49f8a Andrew Jeffery 2019-07-11 @2325  	.set = aspeed_g6_sig_expr_set,
2eda1cdec49f8a Andrew Jeffery 2019-07-11  2326  };
2eda1cdec49f8a Andrew Jeffery 2019-07-11  2327  

:::::: The code at line 2325 was first introduced by commit
:::::: 2eda1cdec49f8ae7878e60d1b06bd8157a95424f pinctrl: aspeed: Add AST2600 pinmux support

:::::: TO: Andrew Jeffery <andrew@aj.id.au>
:::::: 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: 71394 bytes --]

Re: [pinctrl:for-next 54/63] drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: error: initialization from incompatible pointer type

[Andrew Jeffery]

On Wed, 21 Aug 2019, at 10:19, kbuild test robot wrote:
> tree:   
> https://kernel.googlesource.com/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git for-next
> head:   2295f64645cbc4f9ca3002f12d1b36cff3d04ac5
> commit: 21b2920fb587e570b43973300a11b921c3a61d3e [54/63] pinctrl: 
> aspeed: g6: Remove const specifier from aspeed_g6_sig_expr_set's ctx 
> parameter
> config: arm-allmodconfig (attached as .config)
> compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0
> reproduce:
>         wget 
> https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 21b2920fb587e570b43973300a11b921c3a61d3e
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.4.0 make.cross ARCH=arm 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
> >> drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: error: 
> initialization from incompatible pointer type 
> [-Werror=incompatible-pointer-types]
>      .set = aspeed_g6_sig_expr_set,
>             ^~~~~~~~~~~~~~~~~~~~~~
>    drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: note: (near 
> initialization for 'aspeed_g5_ops.set')
>    cc1: some warnings being treated as errors
> 
> vim +2325 drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
> 
> 2eda1cdec49f8a Andrew Jeffery 2019-07-11  2323  
> 2eda1cdec49f8a Andrew Jeffery 2019-07-11  2324  static const struct 
> aspeed_pinmux_ops aspeed_g5_ops = {
> 2eda1cdec49f8a Andrew Jeffery 2019-07-11 @2325  	.set = 
> aspeed_g6_sig_expr_set,
> 2eda1cdec49f8a Andrew Jeffery 2019-07-11  2326  };
> 2eda1cdec49f8a Andrew Jeffery 2019-07-11  2327  
> 
> :::::: The code at line 2325 was first introduced by commit
> :::::: 2eda1cdec49f8ae7878e60d1b06bd8157a95424f pinctrl: aspeed: Add 
> AST2600 pinmux support
> 
> :::::: TO: Andrew Jeffery <andrew@aj.id.au>
> :::::: CC: Linus Walleij <linus.walleij@linaro.org>
> 

This is resolved by back-merging the pinctrl-v5.3-2 into pinctrl/devel
or pinctrl/for-next as mentioned in the thread on Nathan's patch.

Alternatively it will be fixed if pinctrl/devel is merged as is into 5.4 which
contains the change that changes the interface - patch 674fa8daa8c9
("pinctrl: aspeed-g5: Delay acquisition of regmaps").

Andrew

Re: [pinctrl:for-next 54/63] drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: error: initialization from incompatible pointer type

[Linus Walleij]

On Wed, Aug 21, 2019 at 3:09 AM Andrew Jeffery <andrew@aj.id.au> wrote:

> This is resolved by back-merging the pinctrl-v5.3-2 into pinctrl/devel
> or pinctrl/for-next as mentioned in the thread on Nathan's patch.

OK I merged v5.3-rc5 into my devel branch, that should cut it.

Yours,
Linus Walleij

Re: [pinctrl:for-next 54/63] drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c:2325:9: error: initialization from incompatible pointer type

[Andrew Jeffery]

On Wed, 21 Aug 2019, at 21:06, Linus Walleij wrote:
> On Wed, Aug 21, 2019 at 3:09 AM Andrew Jeffery <andrew@aj.id.au> wrote:
> 
> > This is resolved by back-merging the pinctrl-v5.3-2 into pinctrl/devel
> > or pinctrl/for-next as mentioned in the thread on Nathan's patch.
> 
> OK I merged v5.3-rc5 into my devel branch, that should cut it.

Yep, I just did a successful build of pinctrl/devel with multi_v5_defconfig and
CONFIG_COMPILE_TEST=y (the AST2600 arch support only just hit the lists),
looks good.

Thanks,

Andrew