From: kernel test robot <lkp@intel.com>
To: Denis Pauk <pauk.denis@gmail.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
eugene.shalygin@gmail.com, andy.shevchenko@gmail.com,
pauk.denis@gmail.com, platform-driver-x86@vger.kernel.org,
Tor Vic <torvic9@mailbox.org>, Jean Delvare <jdelvare@suse.com>,
Guenter Roeck <linux@roeck-us.net>,
Jonathan Corbet <corbet@lwn.net>,
linux-hwmon@vger.kernel.org, linux-doc@vger.kernel.org
Subject: Re: [PATCH v4 1/2] hwmon: (asus_wmi_ec_sensors) Support B550 Asus WMI.
Date: Mon, 11 Oct 2021 11:05:41 +0800 [thread overview]
Message-ID: <202110111159.pfJSa3ml-lkp@intel.com> (raw)
In-Reply-To: <20211010210513.343925-2-pauk.denis@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2912 bytes --]
Hi Denis,
I love your patch! Yet something to improve:
[auto build test ERROR on 39b483aa38995329326988cbc4077422bebc175a]
url: https://github.com/0day-ci/linux/commits/Denis-Pauk/Update-ASUS-WMI-supported-boards/20211011-050740
base: 39b483aa38995329326988cbc4077422bebc175a
config: x86_64-randconfig-r033-20211011 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 903b30fea21f99d8f48fde4defcc838970e30ee1)
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://github.com/0day-ci/linux/commit/4ac420ec20a4c088ebd178ab3d4a345de53b91a2
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Denis-Pauk/Update-ASUS-WMI-supported-boards/20211011-050740
git checkout 4ac420ec20a4c088ebd178ab3d4a345de53b91a2
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=x86_64
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/hwmon/asus_wmi_ec_sensors.c:356:11: error: implicit declaration of function 'wmi_evaluate_method' [-Werror,-Wimplicit-function-declaration]
status = wmi_evaluate_method(ASUSWMI_MONITORING_GUID, 0, method_id, &input,
^
drivers/hwmon/asus_wmi_ec_sensors.c:356:11: note: did you mean 'wmidev_evaluate_method'?
include/linux/wmi.h:24:20: note: 'wmidev_evaluate_method' declared here
extern acpi_status wmidev_evaluate_method(struct wmi_device *wdev,
^
1 error generated.
vim +/wmi_evaluate_method +356 drivers/hwmon/asus_wmi_ec_sensors.c
344
345 static int asus_wmi_ec_block_read(u32 method_id, char *query, u8 *out)
346 {
347 struct acpi_buffer output = { ACPI_ALLOCATE_BUFFER,
348 NULL };
349 struct acpi_buffer input;
350 union acpi_object *obj;
351 acpi_status status;
352
353 /* the first byte of the BRxx() argument string has to be the string size */
354 input.length = (acpi_size)query[0] + 2;
355 input.pointer = query;
> 356 status = wmi_evaluate_method(ASUSWMI_MONITORING_GUID, 0, method_id, &input,
357 &output);
358 if (ACPI_FAILURE(status))
359 return -EIO;
360
361 obj = output.pointer;
362 if (!obj || obj->type != ACPI_TYPE_BUFFER) {
363 if (!obj)
364 acpi_os_free(output.pointer);
365
366 return -EIO;
367 }
368 asus_wmi_ec_decode_reply_buffer(obj->buffer.pointer, out);
369 acpi_os_free(output.pointer);
370 return 0;
371 }
372
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 38700 bytes --]
next prev parent reply other threads:[~2021-10-11 3:06 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-10 21:05 [PATCH v4 0/2] Update ASUS WMI supported boards Denis Pauk
2021-10-10 21:05 ` [PATCH v4 1/2] hwmon: (asus_wmi_ec_sensors) Support B550 Asus WMI Denis Pauk
2021-10-10 22:09 ` Eugene Shalygin
2021-10-10 23:51 ` Guenter Roeck
2021-10-11 2:04 ` kernel test robot
2021-10-11 3:05 ` kernel test robot [this message]
2021-10-10 21:05 ` [PATCH v4 2/2] hwmon: (asus_wmi_sensors) Support X370 " Denis Pauk
2021-10-10 23:32 ` kernel test robot
2021-10-11 2:53 ` kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202110111159.pfJSa3ml-lkp@intel.com \
--to=lkp@intel.com \
--cc=andy.shevchenko@gmail.com \
--cc=corbet@lwn.net \
--cc=eugene.shalygin@gmail.com \
--cc=jdelvare@suse.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=llvm@lists.linux.dev \
--cc=pauk.denis@gmail.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=torvic9@mailbox.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).