All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Zheng, Lv" <lv.zheng@intel.com>
To: "Rafael J . Wysocki" <rjw@rjwysocki.net>
Cc: "kbuild-all@01.org" <kbuild-all@01.org>,
	"Wysocki, Rafael J" <rafael.j.wysocki@intel.com>,
	"Brown, Len" <len.brown@intel.com>, Lv Zheng <zetalog@gmail.com>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	lkp <lkp@intel.com>
Subject: RE: [PATCH 2/2] ACPI: ACPICA: Cleanup drivers/acpi/acpica Makefile
Date: Tue, 11 Apr 2017 07:19:09 +0000	[thread overview]
Message-ID: <1AE640813FDE7649BE1B193DEA596E886CE8DA28@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <201704021640.4xwSssrU%fengguang.wu@intel.com>

Hi, Rafael

According to the test, PATCH 02 is not possible unless ACPICA debugging function stubs are correctly prepared for ACPI_DEBUG_OUTPUT=n.
So please help to drop PATCH 02 and only PATCH 01 is useful for now.

Thanks and best regards
Lv


> From: linux-acpi-owner@vger.kernel.org [mailto:linux-acpi-owner@vger.kernel.org] On Behalf Of kbuild
> test robot
> Subject: Re: [PATCH 2/2] ACPI: ACPICA: Cleanup drivers/acpi/acpica Makefile
> 
> Hi Lv,
> 
> [auto build test WARNING on pm/linux-next]
> [also build test WARNING on v4.11-rc4 next-20170331]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
> 
> url:    https://github.com/0day-ci/linux/commits/Lv-Zheng/ACPI-ACPICA-Add-non-linux-host-build-
> support/20170402-125838
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next
> reproduce:
>         # apt-get install sparse
>         make ARCH=x86_64 allmodconfig
>         make C=1 CF=-D__CHECK_ENDIAN__
> 
> 
> sparse warnings: (new ones prefixed by >>)
> 
> >> drivers/platform/x86/panasonic-laptop.c:266:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:282:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:289:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:295:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:306:25: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:462:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:478:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 3 given
>    drivers/platform/x86/panasonic-laptop.c:514:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:521:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:560:9: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 3 given
>    drivers/platform/x86/panasonic-laptop.c:579:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:585:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:605:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    drivers/platform/x86/panasonic-laptop.c:611:17: sparse: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>    include/linux/compiler.h:264:8: sparse: attribute 'no_sanitize_address': unknown attribute
>    drivers/platform/x86/panasonic-laptop.c:266:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:579:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:585:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:605:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:611:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:560:9: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:282:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:289:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:295:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:306:25: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:462:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:478:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:514:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c:521:17: sparse: undefined identifier 'ACPI_ACTUAL_DEBUG'
>    drivers/platform/x86/panasonic-laptop.c: In function 'acpi_pcc_get_sqty':
>    drivers/platform/x86/panasonic-laptop.c:267:38: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>           "evaluation error HKEY.SQTY\n"));
>                                          ^
>    In file included from include/acpi/acpi.h:61:0,
>                     from include/linux/acpi.h:33,
>                     from drivers/platform/x86/panasonic-laptop.c:128:
>    include/acpi/acoutput.h:338:41: error: 'ACPI_ACTUAL_DEBUG' undeclared (first use in this function)
>     #define ACPI_DEBUG_PRINT(plist)         ACPI_ACTUAL_DEBUG plist
>                                             ^
>    drivers/platform/x86/panasonic-laptop.c:266:3: note: in expansion of macro 'ACPI_DEBUG_PRINT'
>       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
>       ^~~~~~~~~~~~~~~~
>    include/acpi/acoutput.h:338:41: note: each undeclared identifier is reported only once for each
> function it appears in
>     #define ACPI_DEBUG_PRINT(plist)         ACPI_ACTUAL_DEBUG plist
>                                             ^
>    drivers/platform/x86/panasonic-laptop.c:266:3: note: in expansion of macro 'ACPI_DEBUG_PRINT'
>       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
>       ^~~~~~~~~~~~~~~~
>    drivers/platform/x86/panasonic-laptop.c: In function 'acpi_pcc_retrieve_biosdata':
>    drivers/platform/x86/panasonic-laptop.c:283:38: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>           "evaluation error HKEY.SINF\n"));
>                                          ^
>    In file included from include/acpi/acpi.h:61:0,
>                     from include/linux/acpi.h:33,
>                     from drivers/platform/x86/panasonic-laptop.c:128:
>    include/acpi/acoutput.h:338:41: error: 'ACPI_ACTUAL_DEBUG' undeclared (first use in this function)
>     #define ACPI_DEBUG_PRINT(plist)         ACPI_ACTUAL_DEBUG plist
>                                             ^
>    drivers/platform/x86/panasonic-laptop.c:282:3: note: in expansion of macro 'ACPI_DEBUG_PRINT'
>       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
>       ^~~~~~~~~~~~~~~~
>    drivers/platform/x86/panasonic-laptop.c:289:58: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>       ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "Invalid HKEY.SINF\n"));
>                                                              ^
>    drivers/platform/x86/panasonic-laptop.c:296:39: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>          "SQTY reports bad SINF length\n"));
>                                           ^
>    drivers/platform/x86/panasonic-laptop.c:307:34: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>           "Invalid HKEY.SINF data\n"));
>                                      ^
>    drivers/platform/x86/panasonic-laptop.c: In function 'acpi_pcc_generate_keyinput':
>    drivers/platform/x86/panasonic-laptop.c:463:38: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>          "error getting hotkey status\n"));
>                                          ^
>    In file included from include/acpi/acpi.h:61:0,
>                     from include/linux/acpi.h:33,
>                     from drivers/platform/x86/panasonic-laptop.c:128:
>    include/acpi/acoutput.h:338:41: error: 'ACPI_ACTUAL_DEBUG' undeclared (first use in this function)
>     #define ACPI_DEBUG_PRINT(plist)         ACPI_ACTUAL_DEBUG plist
>                                             ^
>    drivers/platform/x86/panasonic-laptop.c:462:3: note: in expansion of macro 'ACPI_DEBUG_PRINT'
>       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
>       ^~~~~~~~~~~~~~~~
>    drivers/platform/x86/panasonic-laptop.c:479:44: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 3 given
>           "Unknown hotkey event: %d\n", result));
>                                                ^
>    drivers/platform/x86/panasonic-laptop.c: In function 'acpi_pcc_init_input':
>    drivers/platform/x86/panasonic-laptop.c:515:47: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>           "Unable to setup input device keymap\n"));
>                                                   ^
>    In file included from include/acpi/acpi.h:61:0,
>                     from include/linux/acpi.h:33,
>                     from drivers/platform/x86/panasonic-laptop.c:128:
>    include/acpi/acoutput.h:338:41: error: 'ACPI_ACTUAL_DEBUG' undeclared (first use in this function)
>     #define ACPI_DEBUG_PRINT(plist)         ACPI_ACTUAL_DEBUG plist
>                                             ^
>    drivers/platform/x86/panasonic-laptop.c:514:3: note: in expansion of macro 'ACPI_DEBUG_PRINT'
>       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
>       ^~~~~~~~~~~~~~~~
>    drivers/platform/x86/panasonic-laptop.c:522:43: error: macro "ACPI_ACTUAL_DEBUG" requires 6
> arguments, but only 2 given
>           "Unable to register input device\n"));
> 
> vim +/ACPI_ACTUAL_DEBUG +266 drivers/platform/x86/panasonic-laptop.c
> 
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  250  	status =
> acpi_evaluate_object(pcc->handle, METHOD_HKEY_SSET,
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  251
> &params, NULL);
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  252
> 9fab10cd drivers/platform/x86/panasonic-laptop.c Axel Lin     2010-07-07  253  	return (status ==
> AE_OK) ? 0 : -EIO;
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  254  }
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  255
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  256  static inline int
> acpi_pcc_get_sqty(struct acpi_device *device)
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  257  {
> 3e2dab9a drivers/misc/panasonic-laptop.c         Len Brown    2008-10-11  258  	unsigned long long
> s;
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  259  	acpi_status status;
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  260
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  261  	status =
> acpi_evaluate_integer(device->handle, METHOD_HKEY_SQTY,
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  262
> NULL, &s);
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  263  	if
> (ACPI_SUCCESS(status))
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  264  		return s;
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  265  	else {
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23 @266
> 	ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  267
> "evaluation error HKEY.SQTY\n"));
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  268  		return -
> EINVAL;
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  269  	}
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  270  }
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  271
> e253fb94 drivers/platform/x86/panasonic-laptop.c Jean Delvare 2010-10-21  272  static int
> acpi_pcc_retrieve_biosdata(struct pcc_acpi *pcc)
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  273  {
> 709ee531 drivers/misc/panasonic-laptop.c         Harald Welte 2008-09-23  274  	acpi_status status;
> 
> :::::: The code at line 266 was first introduced by commit
> :::::: 709ee531c153038d81b30649b9eeed3c44a4d5cc panasonic-laptop: add Panasonic Let's Note laptop
> extras driver v0.94
> 
> :::::: TO: Harald Welte <laforge@gnumonks.org>
> :::::: CC: Len Brown <len.brown@intel.com>
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2017-04-11  7:31 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-31  4:53 [PATCH 1/2] ACPI: ACPICA: Add non-linux host build support Lv Zheng
2017-03-31  4:53 ` [PATCH 2/2] ACPI: ACPICA: Cleanup drivers/acpi/acpica Makefile Lv Zheng
2017-04-02  6:29   ` kbuild test robot
2017-04-02  8:16   ` kbuild test robot
2017-04-11  7:19     ` Zheng, Lv [this message]
2017-04-11 13:53       ` Rafael J. Wysocki
2017-04-12  2:11         ` Zheng, Lv
2017-04-03 21:47 ` [PATCH 1/2] ACPI: ACPICA: Add non-linux host build support Al Stone
2017-04-11  7:22   ` Zheng, Lv
2017-04-11  7:48 ` [PATCH v2] " Lv Zheng

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=1AE640813FDE7649BE1B193DEA596E886CE8DA28@SHSMSX101.ccr.corp.intel.com \
    --to=lv.zheng@intel.com \
    --cc=kbuild-all@01.org \
    --cc=len.brown@intel.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=rafael.j.wysocki@intel.com \
    --cc=rjw@rjwysocki.net \
    --cc=zetalog@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.