Re: [linux-next:master 11126/11648] surface_charger.c:(.text+0x60): multiple definition of `__raw_ssam_bat_get_sta'; drivers/power/supply/surface_battery.o:surface_battery.c:(.text+0x440): first defined here

[Maximilian Luz <luzmaximilian@gmail.com>]

On 4/10/21 12:13 PM, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   e99d8a8495175df8cb8b739f8cf9b0fc9d0cd3b5
> commit: e61ffb344591fca443be4fc633290cbf15ee3298 [11126/11648] power: supply: Add AC driver for Surface Aggregator Module
> config: arm64-allyesconfig (attached as .config)
> compiler: aarch64-linux-gcc (GCC) 9.3.0
> 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
>          # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=e61ffb344591fca443be4fc633290cbf15ee3298
>          git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
>          git fetch --no-tags linux-next master
>          git checkout e61ffb344591fca443be4fc633290cbf15ee3298
>          # save the attached .config to linux build tree
>          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm64
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> Note: the linux-next/master HEAD e99d8a8495175df8cb8b739f8cf9b0fc9d0cd3b5 builds fine.
>        It may have been fixed somewhere.
> 
> All errors (new ones prefixed by >>):
> 
>     aarch64-linux-ld: drivers/power/supply/surface_charger.o: in function `__raw_ssam_bat_get_sta':
>>> surface_charger.c:(.text+0x60): multiple definition of `__raw_ssam_bat_get_sta'; drivers/power/supply/surface_battery.o:surface_battery.c:(.text+0x440): first defined here
>     aarch64-linux-ld: drivers/power/supply/surface_charger.o: in function `ssam_bat_get_sta':
>>> surface_charger.c:(.text+0x660): multiple definition of `ssam_bat_get_sta'; drivers/power/supply/surface_battery.o:surface_battery.c:(.text+0x740): first defined here

This should be fixed by patch

   platform/surface: aggregator: Make SSAM_DEFINE_SYNC_REQUEST_x define static functions

which is queued as commit 03ee31839170 ("platform/surface: aggregator:
Make SSAM_DEFINE_SYNC_REQUEST_x define static functions") in the
for-next branch of the platform-drivers-x86 tree.

Patchwork: https://lore.kernel.org/patchwork/patch/1389859/
PDx86 commit: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/commit/?h=for-next&id=03ee318391707e822eb915f4f30fe42c78b9d89b

Regards,
Max