Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver

From	Paul Gortmaker <paul.gortmaker@windriver.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver
Date	2016-07-14 22:40 +0200
Message-ID	<rUVAl-5VW-1@gated-at.bofh.it> (permalink)
References	<rUwBX-6kl-3@gated-at.bofh.it> <rUwBX-6kl-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, Jul 13, 2016 at 1:53 PM,  <fu.wei@linaro.org> wrote:
> From: Fu Wei <fu.wei@linaro.org>
>
> This patch adds support for parsing arch timer in GTDT,
> provides some kernel APIs to parse all the PPIs and
> always-on info in GTDT and export them.
>
> By this driver, we can simplify arm_arch_timer drivers, and
> separate the ACPI GTDT knowledge from it.
>
> Signed-off-by: Fu Wei <fu.wei@linaro.org>
> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org>
> ---
>  drivers/acpi/Kconfig           |   5 ++
>  drivers/acpi/Makefile          |   1 +
>  drivers/acpi/arm64/Kconfig     |  15 ++++
>  drivers/acpi/arm64/Makefile    |   1 +
>  drivers/acpi/arm64/acpi_gtdt.c | 170 +++++++++++++++++++++++++++++++++++++++++
>  include/linux/acpi.h           |   6 ++
>  6 files changed, 198 insertions(+)
>
> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
> index b7e2e77..1cdc7d2 100644
> --- a/drivers/acpi/Kconfig
> +++ b/drivers/acpi/Kconfig
> @@ -521,4 +521,9 @@ config XPOWER_PMIC_OPREGION
>
>  endif
>
> +if ARM64
> +source "drivers/acpi/arm64/Kconfig"
> +
> +endif
> +
>  endif  # ACPI
> diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile
> index 251ce85..1a94ff7 100644
> --- a/drivers/acpi/Makefile
> +++ b/drivers/acpi/Makefile
> @@ -99,5 +99,6 @@ obj-$(CONFIG_ACPI_EXTLOG)     += acpi_extlog.o
>  obj-$(CONFIG_PMIC_OPREGION)    += pmic/intel_pmic.o
>  obj-$(CONFIG_CRC_PMIC_OPREGION) += pmic/intel_pmic_crc.o
>  obj-$(CONFIG_XPOWER_PMIC_OPREGION) += pmic/intel_pmic_xpower.o
> +obj-$(CONFIG_ARM64)    += arm64/
>
>  video-objs                     += acpi_video.o video_detect.o
> diff --git a/drivers/acpi/arm64/Kconfig b/drivers/acpi/arm64/Kconfig
> new file mode 100644
> index 0000000..ff5c253
> --- /dev/null
> +++ b/drivers/acpi/arm64/Kconfig
> @@ -0,0 +1,15 @@
> +#
> +# ACPI Configuration for ARM64
> +#
> +
> +menu "The ARM64-specific ACPI Support"
> +
> +config ACPI_GTDT
> +       bool "ACPI GTDT table Support"
> +       help
> +         GTDT (Generic Timer Description Table) provides information
> +         for per-processor timers and Platform (memory-mapped) timers
> +         for ARM platforms. Select this option to provide information
> +         needed for the timers init.
> +
> +endmenu
> diff --git a/drivers/acpi/arm64/Makefile b/drivers/acpi/arm64/Makefile
> new file mode 100644
> index 0000000..466de6b
> --- /dev/null
> +++ b/drivers/acpi/arm64/Makefile
> @@ -0,0 +1 @@
> +obj-$(CONFIG_ACPI_GTDT)                += acpi_gtdt.o
> diff --git a/drivers/acpi/arm64/acpi_gtdt.c b/drivers/acpi/arm64/acpi_gtdt.c
> new file mode 100644
> index 0000000..9ee977d
> --- /dev/null
> +++ b/drivers/acpi/arm64/acpi_gtdt.c
> @@ -0,0 +1,170 @@
> +/*
> + * ARM Specific GTDT table Support
> + *
> + * Copyright (C) 2016, Linaro Ltd.
> + * Author: Daniel Lezcano <daniel.lezcano@linaro.org>
> + *         Fu Wei <fu.wei@linaro.org>
> + *         Hanjun Guo <hanjun.guo@linaro.org>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/acpi.h>
> +#include <linux/init.h>
> +#include <linux/kernel.h>
> +#include <linux/module.h>

Please do not use module.h in drivers that are using a
bool Kconfig setting.

Thanks,
Paul.

Thread

[PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver fu.wei@linaro.org - 2016-07-13 20:00 +0200
  Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-13 22:40 +0200
    Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-13 22:40 +0200
      Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-15 09:40 +0200
    Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Guenter Roeck <linux@roeck-us.net> - 2016-07-13 23:10 +0200
      Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-13 23:50 +0200
        Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-15 09:50 +0200
          Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-15 14:10 +0200
            Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-15 18:20 +0200
    Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-15 09:40 +0200
      Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-15 14:20 +0200
        Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-15 15:10 +0200
          Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-15 18:40 +0200
            Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-15 23:20 +0200
              Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-16 04:30 +0200
                Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-16 14:40 +0200
                Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-19 20:30 +0200
  Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-07-14 22:40 +0200
    Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver Fu Wei <fu.wei@linaro.org> - 2016-07-15 09:50 +0200

csiph-web