Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 29 Mar 2022 14:30:54 +0200
From:   Jean Delvare <jdelvare@suse.de>
To:     kernel test robot <lkp@intel.com>, Wolfram Sang <wsa@kernel.org>
Cc:     Terry Bowman <terry.bowman@amd.com>, kbuild-all@lists.01.org,
        linux-kernel@vger.kernel.org,
        Wim Van Sebroeck <wim@linux-watchdog.org>,
        Robert Richter <rrichter@amd.com>,
        Guenter Roeck <linux@roeck-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
Message-ID: <20220329143054.13a0057d@endymion.delvare>
In-Reply-To: <202203280446.sZhhsZYx-lkp@intel.com>
References: <202203280446.sZhhsZYx-lkp@intel.com>
Organization: SUSE Linux
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

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@intel.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