Re: [RFC PATCH 3/3] mwifiex: pcie: add reset_wsid quirk for Surface 3

[kernel test robot <lkp@intel.com>]

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

Hi "Jonas,

[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on wireless-drivers-next/master]
[also build test ERROR on wireless-drivers/master v5.13-rc2 next-20210521]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Jonas-Dre-ler/mwifiex-Add-quirks-for-MS-Surface-devices/20210522-222428
base:   https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
config: riscv-randconfig-r004-20210523 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project e84a9b9bb3051c35dea993cdad7b3d2575638f85)
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 riscv cross compiling tool for clang build
        # apt-get install binutils-riscv64-linux-gnu
        # https://github.com/0day-ci/linux/commit/664dc875005de6def700fef775192ea019ebc8ab
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Jonas-Dre-ler/mwifiex-Add-quirks-for-MS-Surface-devices/20210522-222428
        git checkout 664dc875005de6def700fef775192ea019ebc8ab
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv 

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/net/wireless/marvell/mwifiex/pcie_quirks.c:202:7: error: implicit declaration of function 'acpi_has_method' [-Werror,-Wimplicit-function-declaration]
           if (!acpi_has_method(handle, "_DSM")) {
                ^
   drivers/net/wireless/marvell/mwifiex/pcie_quirks.c:202:7: note: did you mean 'acpi_has_watchdog'?
   include/linux/acpi.h:1288:20: note: 'acpi_has_watchdog' declared here
   static inline bool acpi_has_watchdog(void) { return false; }
                      ^
>> drivers/net/wireless/marvell/mwifiex/pcie_quirks.c:207:7: error: implicit declaration of function 'acpi_check_dsm' [-Werror,-Wimplicit-function-declaration]
           if (!acpi_check_dsm(handle, &wsid_dsm_guid,
                ^
   2 errors generated.


vim +/acpi_has_method +202 drivers/net/wireless/marvell/mwifiex/pcie_quirks.c

   186	
   187	int mwifiex_pcie_reset_wsid_quirk(struct pci_dev *pdev)
   188	{
   189		acpi_handle handle;
   190		union acpi_object *obj;
   191		acpi_status status;
   192	
   193		dev_info(&pdev->dev, "Using reset_wsid quirk to perform FW reset\n");
   194	
   195		status = acpi_get_handle(NULL, ACPI_WSID_PATH, &handle);
   196		if (ACPI_FAILURE(status)) {
   197			dev_err(&pdev->dev, "No ACPI handle for path %s\n",
   198				ACPI_WSID_PATH);
   199			return -ENODEV;
   200		}
   201	
 > 202		if (!acpi_has_method(handle, "_DSM")) {
   203			dev_err(&pdev->dev, "_DSM method not found\n");
   204			return -ENODEV;
   205		}
   206	
 > 207		if (!acpi_check_dsm(handle, &wsid_dsm_guid,

---
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: 34104 bytes --]