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

["Andrew Jeffery" <andrew@xxxxxxxx>]

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@xxxxxxxxx>
> 
> 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@xxxxxxxx>
> :::::: CC: Linus Walleij <linus.walleij@xxxxxxxxxx>
> 

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