[fujitsu-integration:mailbox-for-next 8/11] drivers/mailbox/arm_mhuv2.c:1127:13: error: incompatible function pointer types initializing 'int amba_device with an expression of type 'void (struct amba_device

[fujitsu-integration:mailbox-for-next 8/11] drivers/mailbox/arm_mhuv2.c:1127:13: error: incompatible function pointer types initializing 'int amba_device with an expression of type 'void (struct amba_device

[kernel test robot]

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

tree:   https://git.linaro.org/landing-teams/working/fujitsu/integration.git mailbox-for-next
head:   6b50df2b8c208a04d44b8df5b7baaf668ceb8fc3
commit: 09d12157b3bf3b42f74b7871930064eee3e080a3 [8/11] mailbox: arm_mhuv2: make remove callback return void
config: arm64-randconfig-r002-20210223 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project f14a14dd2564703db02f80c00db8ae492b594f77)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm64 cross compiling tool for clang build
        # apt-get install binutils-aarch64-linux-gnu
        git remote add fujitsu-integration https://git.linaro.org/landing-teams/working/fujitsu/integration.git
        git fetch --no-tags fujitsu-integration mailbox-for-next
        git checkout 09d12157b3bf3b42f74b7871930064eee3e080a3
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 

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

All errors (new ones prefixed by >>):

>> drivers/mailbox/arm_mhuv2.c:1127:13: error: incompatible function pointer types initializing 'int (*)(struct amba_device *)' with an expression of type 'void (struct amba_device *)' [-Werror,-Wincompatible-function-pointer-types]
           .remove         = mhuv2_remove,
                             ^~~~~~~~~~~~
   1 error generated.


vim +1127 drivers/mailbox/arm_mhuv2.c

5a6338cce9f413 Viresh Kumar 2020-11-17  1120  
5a6338cce9f413 Viresh Kumar 2020-11-17  1121  static struct amba_driver mhuv2_driver = {
5a6338cce9f413 Viresh Kumar 2020-11-17  1122  	.drv = {
5a6338cce9f413 Viresh Kumar 2020-11-17  1123  		.name	= "arm-mhuv2",
5a6338cce9f413 Viresh Kumar 2020-11-17  1124  	},
5a6338cce9f413 Viresh Kumar 2020-11-17  1125  	.id_table	= mhuv2_ids,
5a6338cce9f413 Viresh Kumar 2020-11-17  1126  	.probe		= mhuv2_probe,
5a6338cce9f413 Viresh Kumar 2020-11-17 @1127  	.remove		= mhuv2_remove,
5a6338cce9f413 Viresh Kumar 2020-11-17  1128  };
5a6338cce9f413 Viresh Kumar 2020-11-17  1129  module_amba_driver(mhuv2_driver);
5a6338cce9f413 Viresh Kumar 2020-11-17  1130  

:::::: The code at line 1127 was first introduced by commit
:::::: 5a6338cce9f4133c478d3b10b300f96dd644379a mailbox: arm_mhuv2: Add driver

:::::: TO: Viresh Kumar <viresh.kumar@linaro.org>
:::::: CC: Jassi Brar <jaswinder.singh@linaro.org>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 34734 bytes --]

Re: [fujitsu-integration:mailbox-for-next 8/11] drivers/mailbox/arm_mhuv2.c:1127:13: error: incompatible function pointer types initializing 'int amba_device with an expression of type 'void (struct amba_device

[Uwe Kleine-König]

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

On Wed, Feb 24, 2021 at 03:48:24PM +0800, kernel test robot wrote:
> tree:   https://git.linaro.org/landing-teams/working/fujitsu/integration.git mailbox-for-next
> head:   6b50df2b8c208a04d44b8df5b7baaf668ceb8fc3
> commit: 09d12157b3bf3b42f74b7871930064eee3e080a3 [8/11] mailbox: arm_mhuv2: make remove callback return void

That happens because 09d12157b3bf3b42f74b7871930064eee3e080a3 doesn't
include commit 3fd269e74f2f ("amba: Make the remove callback return
void").

Given that both patches are also in Linus' tree (as
860660fd829e and 3fd269e74f2f) the best fix would be to drop
6b50df2b8c208a04d44b8df5b7baaf668ceb8fc3 from this tree.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]