| Message ID | 1398432017-8506-3-git-send-email-hanjun.guo@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, 25 Apr 2014 21:20:08 +0800, Hanjun Guo <hanjun.guo@linaro.org> wrote: > The _PDC (Processor Driver Capabilities) object provides OSPM a > mechanism to convey to the platform the capabilities supported > by OSPM for processor power management. > > OSPM evaluates _PDC prior to evaluating any other processor > power management objects returning configuration information. > > This patch introduces the skeleton of _PDC related file to make > ACPI core can be compiled on ARM64, and should be revisited when > implementing CPU power control. > > Signed-off-by: Al Stone <al.stone@linaro.org> > Signed-off-by: Graeme Gregory <graeme.gregory@linaro.org> > Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> > --- > arch/arm64/include/asm/acpi.h | 37 +++++++++++++++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 arch/arm64/include/asm/acpi.h > > diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h > new file mode 100644 > index 0000000..e3e990e > --- /dev/null > +++ b/arch/arm64/include/asm/acpi.h > @@ -0,0 +1,37 @@ > +/* > + * Copyright (C) 2013~2014, Al Stone <al.stone@linaro.org> Not really a correct copyright assignement. The copyright is held by Linaro, but the author is Al. It should instead read: * Copyright (C0 2013-2014, Linaro Ltd. * Author: Al Stone <al.stone@linaro.org> Acked-by: Grant Likely <grant.likely@linaro.org> > + * > + * ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > + */ > + > +#ifndef _ASM_ACPI_H > +#define _ASM_ACPI_H > + > +static inline bool arch_has_acpi_pdc(void) > +{ > + /* > + * Always false for now, should be revisited when > + * implementing CPU power management > + */ > + return false; > +} > + > +static inline void arch_acpi_set_pdc_bits(u32 *buf) > +{ > + /* Should be revisited when implementing CPU power management */ > + return; > +} > + > +#endif /*_ASM_ACPI_H*/ > -- > 1.7.9.5 >
On 2014-4-29 17:39, Grant Likely wrote: > On Fri, 25 Apr 2014 21:20:08 +0800, Hanjun Guo <hanjun.guo@linaro.org> wrote: >> The _PDC (Processor Driver Capabilities) object provides OSPM a >> mechanism to convey to the platform the capabilities supported >> by OSPM for processor power management. >> >> OSPM evaluates _PDC prior to evaluating any other processor >> power management objects returning configuration information. >> >> This patch introduces the skeleton of _PDC related file to make >> ACPI core can be compiled on ARM64, and should be revisited when >> implementing CPU power control. >> >> Signed-off-by: Al Stone <al.stone@linaro.org> >> Signed-off-by: Graeme Gregory <graeme.gregory@linaro.org> >> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> >> --- >> arch/arm64/include/asm/acpi.h | 37 +++++++++++++++++++++++++++++++++++++ >> 1 file changed, 37 insertions(+) >> create mode 100644 arch/arm64/include/asm/acpi.h >> >> diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h >> new file mode 100644 >> index 0000000..e3e990e >> --- /dev/null >> +++ b/arch/arm64/include/asm/acpi.h >> @@ -0,0 +1,37 @@ >> +/* >> + * Copyright (C) 2013~2014, Al Stone <al.stone@linaro.org> > > Not really a correct copyright assignement. The copyright is held by > Linaro, but the author is Al. It should instead read: > > * Copyright (C0 2013-2014, Linaro Ltd. > * Author: Al Stone <al.stone@linaro.org> > > Acked-by: Grant Likely <grant.likely@linaro.org> We will update this patch accordingly. Thanks Hanjun
diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h new file mode 100644 index 0000000..e3e990e --- /dev/null +++ b/arch/arm64/include/asm/acpi.h @@ -0,0 +1,37 @@ +/* + * Copyright (C) 2013~2014, Al Stone <al.stone@linaro.org> + * + * ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + */ + +#ifndef _ASM_ACPI_H +#define _ASM_ACPI_H + +static inline bool arch_has_acpi_pdc(void) +{ + /* + * Always false for now, should be revisited when + * implementing CPU power management + */ + return false; +} + +static inline void arch_acpi_set_pdc_bits(u32 *buf) +{ + /* Should be revisited when implementing CPU power management */ + return; +} + +#endif /*_ASM_ACPI_H*/