Merge branches 'acpica-fixes', 'acpi-cppc-fixes' and 'acpi-tools-fixes'

* acpica-fixes:
  Revert "ACPICA: FADT support cleanup"

* acpi-cppc-fixes:
  mailbox: PCC: Fix lockdep warning when request PCC channel

* acpi-tools-fixes:
  tools/power/acpi: Remove direct kernel source include reference
This commit is contained in:
Rafael J. Wysocki 2016-11-18 21:34:42 +01:00
commit aab0b243b9
7 changed files with 63 additions and 40 deletions

View File

@ -65,6 +65,7 @@
#include <linux/mailbox_controller.h> #include <linux/mailbox_controller.h>
#include <linux/mailbox_client.h> #include <linux/mailbox_client.h>
#include <linux/io-64-nonatomic-lo-hi.h> #include <linux/io-64-nonatomic-lo-hi.h>
#include <acpi/pcc.h>
#include "mailbox.h" #include "mailbox.h"
@ -267,6 +268,8 @@ struct mbox_chan *pcc_mbox_request_channel(struct mbox_client *cl,
if (chan->txdone_method == TXDONE_BY_POLL && cl->knows_txdone) if (chan->txdone_method == TXDONE_BY_POLL && cl->knows_txdone)
chan->txdone_method |= TXDONE_BY_ACK; chan->txdone_method |= TXDONE_BY_ACK;
spin_unlock_irqrestore(&chan->lock, flags);
if (pcc_doorbell_irq[subspace_id] > 0) { if (pcc_doorbell_irq[subspace_id] > 0) {
int rc; int rc;
@ -275,12 +278,11 @@ struct mbox_chan *pcc_mbox_request_channel(struct mbox_client *cl,
if (unlikely(rc)) { if (unlikely(rc)) {
dev_err(dev, "failed to register PCC interrupt %d\n", dev_err(dev, "failed to register PCC interrupt %d\n",
pcc_doorbell_irq[subspace_id]); pcc_doorbell_irq[subspace_id]);
pcc_mbox_free_channel(chan);
chan = ERR_PTR(rc); chan = ERR_PTR(rc);
} }
} }
spin_unlock_irqrestore(&chan->lock, flags);
return chan; return chan;
} }
EXPORT_SYMBOL_GPL(pcc_mbox_request_channel); EXPORT_SYMBOL_GPL(pcc_mbox_request_channel);
@ -304,20 +306,19 @@ void pcc_mbox_free_channel(struct mbox_chan *chan)
return; return;
} }
if (pcc_doorbell_irq[id] > 0)
devm_free_irq(chan->mbox->dev, pcc_doorbell_irq[id], chan);
spin_lock_irqsave(&chan->lock, flags); spin_lock_irqsave(&chan->lock, flags);
chan->cl = NULL; chan->cl = NULL;
chan->active_req = NULL; chan->active_req = NULL;
if (chan->txdone_method == (TXDONE_BY_POLL | TXDONE_BY_ACK)) if (chan->txdone_method == (TXDONE_BY_POLL | TXDONE_BY_ACK))
chan->txdone_method = TXDONE_BY_POLL; chan->txdone_method = TXDONE_BY_POLL;
if (pcc_doorbell_irq[id] > 0)
devm_free_irq(chan->mbox->dev, pcc_doorbell_irq[id], chan);
spin_unlock_irqrestore(&chan->lock, flags); spin_unlock_irqrestore(&chan->lock, flags);
} }
EXPORT_SYMBOL_GPL(pcc_mbox_free_channel); EXPORT_SYMBOL_GPL(pcc_mbox_free_channel);
/** /**
* pcc_send_data - Called from Mailbox Controller code. Used * pcc_send_data - Called from Mailbox Controller code. Used
* here only to ring the channel doorbell. The PCC client * here only to ring the channel doorbell. The PCC client

View File

@ -191,6 +191,9 @@
#ifndef __init #ifndef __init
#define __init #define __init
#endif #endif
#ifndef __iomem
#define __iomem
#endif
/* Host-dependent types and defines for user-space ACPICA */ /* Host-dependent types and defines for user-space ACPICA */

View File

@ -8,18 +8,19 @@
# as published by the Free Software Foundation; version 2 # as published by the Free Software Foundation; version 2
# of the License. # of the License.
include ../../../../scripts/Makefile.include ifeq ($(srctree),)
srctree := $(patsubst %/,%,$(dir $(shell pwd)))
srctree := $(patsubst %/,%,$(dir $(srctree)))
#$(info Determined 'srctree' to be $(srctree))
endif
OUTPUT=./ include $(srctree)/../../scripts/Makefile.include
OUTPUT=$(srctree)/
ifeq ("$(origin O)", "command line") ifeq ("$(origin O)", "command line")
OUTPUT := $(O)/ OUTPUT := $(O)/power/acpi/
endif
ifneq ($(OUTPUT),)
# check that the output directory actually exists
OUTDIR := $(shell cd $(OUTPUT) && /bin/pwd)
$(if $(OUTDIR),, $(error output directory "$(OUTPUT)" does not exist))
endif endif
#$(info Determined 'OUTPUT' to be $(OUTPUT))
# --- CONFIGURATION BEGIN --- # --- CONFIGURATION BEGIN ---
@ -70,8 +71,8 @@ WARNINGS := -Wall
WARNINGS += $(call cc-supports,-Wstrict-prototypes) WARNINGS += $(call cc-supports,-Wstrict-prototypes)
WARNINGS += $(call cc-supports,-Wdeclaration-after-statement) WARNINGS += $(call cc-supports,-Wdeclaration-after-statement)
KERNEL_INCLUDE := ../../../include KERNEL_INCLUDE := $(OUTPUT)include
ACPICA_INCLUDE := ../../../drivers/acpi/acpica ACPICA_INCLUDE := $(srctree)/../../../drivers/acpi/acpica
CFLAGS += -D_LINUX -I$(KERNEL_INCLUDE) -I$(ACPICA_INCLUDE) CFLAGS += -D_LINUX -I$(KERNEL_INCLUDE) -I$(ACPICA_INCLUDE)
CFLAGS += $(WARNINGS) CFLAGS += $(WARNINGS)

View File

@ -8,28 +8,42 @@
# as published by the Free Software Foundation; version 2 # as published by the Free Software Foundation; version 2
# of the License. # of the License.
$(OUTPUT)$(TOOL): $(TOOL_OBJS) FORCE objdir := $(OUTPUT)tools/$(TOOL)/
$(ECHO) " LD " $@ toolobjs := $(addprefix $(objdir),$(TOOL_OBJS))
$(QUIET) $(LD) $(CFLAGS) $(LDFLAGS) $(TOOL_OBJS) -L$(OUTPUT) -o $@ $(OUTPUT)$(TOOL): $(toolobjs) FORCE
$(ECHO) " LD " $(subst $(OUTPUT),,$@)
$(QUIET) $(LD) $(CFLAGS) $(LDFLAGS) $(toolobjs) -L$(OUTPUT) -o $@
$(ECHO) " STRIP " $(subst $(OUTPUT),,$@)
$(QUIET) $(STRIPCMD) $@ $(QUIET) $(STRIPCMD) $@
$(OUTPUT)%.o: %.c $(KERNEL_INCLUDE):
$(ECHO) " CC " $@ $(ECHO) " MKDIR " $(subst $(OUTPUT),,$@)
$(QUIET) mkdir -p $(KERNEL_INCLUDE)
$(ECHO) " CP " $(subst $(OUTPUT),,$@)
$(QUIET) cp -rf $(srctree)/../../../include/acpi $(KERNEL_INCLUDE)/
$(objdir)%.o: %.c $(KERNEL_INCLUDE)
$(ECHO) " CC " $(subst $(OUTPUT),,$@)
$(QUIET) $(CC) -c $(CFLAGS) -o $@ $< $(QUIET) $(CC) -c $(CFLAGS) -o $@ $<
all: $(OUTPUT)$(TOOL) all: $(OUTPUT)$(TOOL)
clean: clean:
-find $(OUTPUT) \( -not -type d \) \ $(ECHO) " RMOBJ " $(subst $(OUTPUT),,$(objdir))
-and \( -name '*~' -o -name '*.[oas]' \) \ $(QUIET) find $(objdir) \( -not -type d \)\
-type f -print \ -and \( -name '*~' -o -name '*.[oas]' \)\
| xargs rm -f -type f -print | xargs rm -f
-rm -f $(OUTPUT)$(TOOL) $(ECHO) " RM " $(TOOL)
$(QUIET) rm -f $(OUTPUT)$(TOOL)
$(ECHO) " RMINC " $(subst $(OUTPUT),,$(KERNEL_INCLUDE))
$(QUIET) rm -rf $(KERNEL_INCLUDE)
install-tools: install-tools:
$(INSTALL) -d $(DESTDIR)${sbindir} $(ECHO) " INST " $(TOOL)
$(INSTALL_PROGRAM) $(OUTPUT)$(TOOL) $(DESTDIR)${sbindir} $(QUIET) $(INSTALL) -d $(DESTDIR)$(sbindir)
$(QUIET) $(INSTALL_PROGRAM) $(OUTPUT)$(TOOL) $(DESTDIR)$(sbindir)
uninstall-tools: uninstall-tools:
- rm -f $(DESTDIR)${sbindir}/$(TOOL) $(ECHO) " UNINST " $(TOOL)
$(QUIET) rm -f $(DESTDIR)$(sbindir)/$(TOOL)
install: all install-tools $(EXTRA_INSTALL) install: all install-tools $(EXTRA_INSTALL)
uninstall: uninstall-tools $(EXTRA_UNINSTALL) uninstall: uninstall-tools $(EXTRA_UNINSTALL)

View File

@ -17,9 +17,7 @@ vpath %.c \
../../os_specific/service_layers\ ../../os_specific/service_layers\
. .
CFLAGS += -DACPI_APPLICATION -DACPI_SINGLE_THREAD -DACPI_DEBUGGER\ CFLAGS += -DACPI_APPLICATION -DACPI_SINGLE_THREAD -DACPI_DEBUGGER\
-I.\ -I.
-I../../../../../drivers/acpi/acpica\
-I../../../../../include
LDFLAGS += -lpthread LDFLAGS += -lpthread
TOOL_OBJS = \ TOOL_OBJS = \
acpidbg.o acpidbg.o

View File

@ -12,10 +12,16 @@
#include <acpi/acpi.h> #include <acpi/acpi.h>
/* Headers not included by include/acpi/platform/aclinux.h */ /* Headers not included by include/acpi/platform/aclinux.h */
#include <unistd.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <error.h>
#include <stdbool.h> #include <stdbool.h>
#include <fcntl.h> #include <fcntl.h>
#include <assert.h> #include <assert.h>
#include <linux/circ_buf.h> #include <sys/select.h>
#include "../../../../../include/linux/circ_buf.h"
#define ACPI_AML_FILE "/sys/kernel/debug/acpi/acpidbg" #define ACPI_AML_FILE "/sys/kernel/debug/acpi/acpidbg"
#define ACPI_AML_SEC_TICK 1 #define ACPI_AML_SEC_TICK 1

View File

@ -19,9 +19,7 @@ vpath %.c \
./\ ./\
../../common\ ../../common\
../../os_specific/service_layers ../../os_specific/service_layers
CFLAGS += -DACPI_DUMP_APP -I.\ CFLAGS += -DACPI_DUMP_APP -I.
-I../../../../../drivers/acpi/acpica\
-I../../../../../include
TOOL_OBJS = \ TOOL_OBJS = \
apdump.o\ apdump.o\
apfiles.o\ apfiles.o\
@ -49,7 +47,9 @@ TOOL_OBJS = \
include ../../Makefile.rules include ../../Makefile.rules
install-man: ../../man/acpidump.8 install-man: $(srctree)/man/acpidump.8
$(INSTALL_DATA) -D $< $(DESTDIR)${mandir}/man8/acpidump.8 $(ECHO) " INST " acpidump.8
$(QUIET) $(INSTALL_DATA) -D $< $(DESTDIR)$(mandir)/man8/acpidump.8
uninstall-man: uninstall-man:
- rm -f $(DESTDIR)${mandir}/man8/acpidump.8 $(ECHO) " UNINST " acpidump.8
$(QUIET) rm -f $(DESTDIR)$(mandir)/man8/acpidump.8