linux-kernel - Re: [watchdog-next:v5.18-rc1 12/13] drivers/watchdog/sp5100_tco.c:345:6: warning: assignment to 'struct resource *' from 'int' makes pointer from integer without a cast

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20220329143054.13a0057d@endymion.delvare>
Date:   Tue, 29 Mar 2022 14:30:54 +0200
From:   Jean Delvare <jdelvare@...e.de>
To:     kernel test robot <lkp@...el.com>, Wolfram Sang <wsa@...nel.org>
Cc:     Terry Bowman <terry.bowman@....com>, kbuild-all@...ts.01.org,
        linux-kernel@...r.kernel.org,
        Wim Van Sebroeck <wim@...ux-watchdog.org>,
        Robert Richter <rrichter@....com>,
        Guenter Roeck <linux@...ck-us.net>
Subject: Re: [watchdog-next:v5.18-rc1 12/13]
 drivers/watchdog/sp5100_tco.c:345:6: warning: assignment to 'struct
 resource *' from 'int' makes pointer from integer without a cast

Hi all,

On Mon, 28 Mar 2022 04:50:02 +0800, kernel test robot wrote:
> tree:   git://www.linux-watchdog.org/linux-watchdog-next.git v5.18-rc1
> head:   826270373f17fd8ebd10753ca0a5fd2ceb1dc38e
> commit: 0578fff4aae5bce3f09875f58e68e9ffbab8daf5 [12/13] Watchdog: sp5100_tco: Add initialization using EFCH MMIO
> config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20220328/202203280446.sZhhsZYx-lkp@intel.com/config)
> compiler: gcc-9 (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
> reproduce (this is a W=1 build):
>         git remote add watchdog-next git://www.linux-watchdog.org/linux-watchdog-next.git
>         git fetch --no-tags watchdog-next v5.18-rc1
>         git checkout 0578fff4aae5bce3f09875f58e68e9ffbab8daf5
>         # save the config file to linux build tree
>         mkdir build_dir
>         make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/watchdog/
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@...el.com>
> 
> All warnings (new ones prefixed by >>):
> 
>    drivers/watchdog/sp5100_tco.c: In function 'sp5100_tco_setupdevice_mmio':
>    drivers/watchdog/sp5100_tco.c:345:8: error: implicit declaration of function 'request_mem_region_muxed'; did you mean 'request_mem_region'? [-Werror=implicit-function-declaration]
>      345 |  res = request_mem_region_muxed(EFCH_PM_ACPI_MMIO_PM_ADDR,
>          |        ^~~~~~~~~~~~~~~~~~~~~~~~
>          |        request_mem_region
> >> drivers/watchdog/sp5100_tco.c:345:6: warning: assignment to 'struct resource *' from 'int' makes pointer from integer without a cast [-Wint-conversion]  
>      345 |  res = request_mem_region_muxed(EFCH_PM_ACPI_MMIO_PM_ADDR,
>          |      ^
>    cc1: some warnings being treated as errors

This function is added in the i2c tree, that was supposed to be merged
before the watchdog tree for v5.18-rc1. Were the merge requests sent in
the wrong order maybe?

Either way, the problem will solve itself automatically as soon as
Wolfram's pull request for the i2c tree is processed (might have
already happened by now) so this can be ignored.

-- 
Jean Delvare
SUSE L3 Support