Re: [PATCH v3 1/2] rtc-cmos: Clear ACPI-driven alarms upon resume

From: kbuild test robot
Date: Wed Sep 14 2016 - 18:50:42 EST


Hi Gabriele,

[auto build test WARNING on abelloni/rtc-next]
[also build test WARNING on v4.8-rc6 next-20160914]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
[Suggest to use git(>=2.9.0) format-patch --base=<commit> (or --base=auto for convenience) to record what (public, well-known) commit your patch series was built on]
[Check https://git-scm.com/docs/git-format-patch for more information]

url: https://github.com/0day-ci/linux/commits/Gabriele-Mazzotta/rtc-cmos-Clear-ACPI-driven-alarms-upon-resume/20160915-031802
base: https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: i386-randconfig-x0-09150359 (attached as .config)
compiler: gcc-6 (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All warnings (new ones prefixed by >>):

In file included from include/linux/linkage.h:4:0,
from include/linux/kernel.h:6,
from drivers/rtc/rtc-cmos.c:34:
drivers/rtc/rtc-cmos.c: In function 'cmos_resume':
drivers/rtc/rtc-cmos.c:941:9: error: 'acpi_gbl_FADT' undeclared (first use in this function)
!(acpi_gbl_FADT.flags & ACPI_FADT_FIXED_RTC)) {
^
include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^~~~
>> drivers/rtc/rtc-cmos.c:940:3: note: in expansion of macro 'if'
if ((tmp & RTC_AIE) &&
^~
drivers/rtc/rtc-cmos.c:941:9: note: each undeclared identifier is reported only once for each function it appears in
!(acpi_gbl_FADT.flags & ACPI_FADT_FIXED_RTC)) {
^
include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^~~~
>> drivers/rtc/rtc-cmos.c:940:3: note: in expansion of macro 'if'
if ((tmp & RTC_AIE) &&
^~
drivers/rtc/rtc-cmos.c:941:31: error: 'ACPI_FADT_FIXED_RTC' undeclared (first use in this function)
!(acpi_gbl_FADT.flags & ACPI_FADT_FIXED_RTC)) {
^
include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^~~~
>> drivers/rtc/rtc-cmos.c:940:3: note: in expansion of macro 'if'
if ((tmp & RTC_AIE) &&
^~
drivers/rtc/rtc-cmos.c:942:4: error: unknown type name 'acpi_status'
acpi_status status;
^~~~~~~~~~~
drivers/rtc/rtc-cmos.c:943:4: error: unknown type name 'acpi_event_status'
acpi_event_status rtc_status;
^~~~~~~~~~~~~~~~~
drivers/rtc/rtc-cmos.c:945:13: error: implicit declaration of function 'acpi_get_event_status' [-Werror=implicit-function-declaration]
status = acpi_get_event_status(ACPI_EVENT_RTC,
^~~~~~~~~~~~~~~~~~~~~
drivers/rtc/rtc-cmos.c:945:35: error: 'ACPI_EVENT_RTC' undeclared (first use in this function)
status = acpi_get_event_status(ACPI_EVENT_RTC,
^~~~~~~~~~~~~~
In file included from include/linux/linkage.h:4:0,
from include/linux/kernel.h:6,
from drivers/rtc/rtc-cmos.c:34:
drivers/rtc/rtc-cmos.c:947:8: error: implicit declaration of function 'ACPI_FAILURE' [-Werror=implicit-function-declaration]
if (ACPI_FAILURE(status)) {
^
include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^~~~
drivers/rtc/rtc-cmos.c:947:4: note: in expansion of macro 'if'
if (ACPI_FAILURE(status)) {
^~
drivers/rtc/rtc-cmos.c:949:28: error: 'ACPI_EVENT_FLAG_SET' undeclared (first use in this function)
} else if (rtc_status & ACPI_EVENT_FLAG_SET) {
^
include/linux/compiler.h:149:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^~~~
drivers/rtc/rtc-cmos.c:949:11: note: in expansion of macro 'if'
} else if (rtc_status & ACPI_EVENT_FLAG_SET) {
^~
In file included from include/acpi/acpi.h:64:0,
from include/linux/acpi.h:33,
from drivers/rtc/rtc-cmos.c:987:
include/acpi/acpixf.h: At top level:
include/acpi/acpixf.h:705:5: error: conflicting types for 'acpi_get_event_status'
acpi_get_event_status(u32 event,
^
include/acpi/acpixf.h:95:2: note: in definition of macro 'ACPI_EXTERNAL_RETURN_STATUS'
prototype;
^~~~~~~~~
include/acpi/acpixf.h:704:1: note: in expansion of macro 'ACPI_HW_DEPENDENT_RETURN_STATUS'
ACPI_HW_DEPENDENT_RETURN_STATUS(acpi_status
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/rtc/rtc-cmos.c:945:13: note: previous implicit declaration of 'acpi_get_event_status' was here
status = acpi_get_event_status(ACPI_EVENT_RTC,
^~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors

vim +/if +940 drivers/rtc/rtc-cmos.c

924 CMOS_WRITE(tmp, RTC_CONTROL);
925 hpet_set_rtc_irq_bit(tmp & RTC_IRQMASK);
926
927 mask = CMOS_READ(RTC_INTR_FLAGS);
928 mask &= (tmp & RTC_IRQMASK) | RTC_IRQF;
929 if (!is_hpet_enabled() || !is_intr(mask))
930 break;
931
932 /* force one-shot behavior if HPET blocked
933 * the wake alarm's irq
934 */
935 rtc_update_irq(cmos->rtc, 1, mask);
936 tmp &= ~RTC_AIE;
937 hpet_mask_rtc_irq_bit(RTC_AIE);
938 } while (mask & RTC_AIE);
939
> 940 if ((tmp & RTC_AIE) &&
941 !(acpi_gbl_FADT.flags & ACPI_FADT_FIXED_RTC)) {
942 acpi_status status;
943 acpi_event_status rtc_status;
944
945 status = acpi_get_event_status(ACPI_EVENT_RTC,
946 &rtc_status);
947 if (ACPI_FAILURE(status)) {
948 dev_err(dev, "Could not get RTC status\n");

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: Binary data