[PATCH v18 06/15] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2016-12-08 Thread fu . wei
From: Fu Wei Currently, the counter frequency detection call(arch_timer_detect_rate) combines all the ways to get counter frequency: device-tree property, system coprocessor register, MMIO timer. But in the most of use cases, we don't need all the ways to try: For example, reading device-tree

[PATCH v18 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-12-08 Thread fu . wei
From: Fu Wei Because arch_timer_needs_of_probing is only for booting with device-tree, but arch_timer_common_init is a generic init call which shouldn't include the FW-specific code. It's better to put arch_timer_needs_of_probing into DT init function. But for per-cpu timer

[PATCH v18 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_needs_probing

2016-12-08 Thread fu . wei
From: Fu Wei When system init with device-tree, we don't know which node will be initialized first. And the code in arch_timer_common_init should wait until per-cpu timer and MMIO timer are both initialized. So we need arch_timer_needs_probing to detect the init status of system. But currently

[PATCH v18 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-12-08 Thread fu . wei
From: Fu Wei The patch refactor original memory-mapped timer init code: (1) Refactor "arch_timer_mem_init", make it become a common code for memory-mapped timer init. (2) Add a new function "arch_timer_mem_of_init" for DT init. Signed-off-by: Fu Wei ---

[PATCH v18 03/15] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-12-08 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patch defines pr_fmt(fmt) for all pr_* functions, then the pr_* doesn't need to add "arch_timer:" everytime. According to the suggestion from checkpatch.pl: (1) delete some Blank Spaces in arch_timer_banner; (2) delete a redundant Tab

[PATCH v18 05/15] clocksource/drivers/arm_arch_timer: rework PPI determination

2016-12-08 Thread fu . wei
From: Fu Wei <fu@linaro.org> Currently, the arch timer driver uses ARCH_TIMER_PHYS_SECURE_PPI to mean the driver will use the secure PPI *and* potentialy also use the non-secure PPI. This is somewhat confusing. For arm64, where it never makes sense to use the secure PPI, this means w

[PATCH v18 03/15] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-12-08 Thread fu . wei
From: Fu Wei This patch defines pr_fmt(fmt) for all pr_* functions, then the pr_* doesn't need to add "arch_timer:" everytime. According to the suggestion from checkpatch.pl: (1) delete some Blank Spaces in arch_timer_banner; (2) delete a redundant Tab in a bland line of arch_timer

[PATCH v18 05/15] clocksource/drivers/arm_arch_timer: rework PPI determination

2016-12-08 Thread fu . wei
From: Fu Wei Currently, the arch timer driver uses ARCH_TIMER_PHYS_SECURE_PPI to mean the driver will use the secure PPI *and* potentialy also use the non-secure PPI. This is somewhat confusing. For arm64, where it never makes sense to use the secure PPI, this means we must always request

[PATCH v18 02/15] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-12-08 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patch add a new enum "arch_timer_spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei <fu@linaro.org> Acked-by: Mark Rutland <mark.rutl...@arm.com> --- drivers/cloc

[PATCH v18 02/15] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-12-08 Thread fu . wei
From: Fu Wei This patch add a new enum "arch_timer_spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei Acked-by: Mark Rutland --- drivers/clocksource/arm_arch_timer.c | 4 ++-- include/clocksource/arm_arch_timer.h | 6 +

[PATCH v18 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-12-08 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patchset: (1)Preparation for adding GTDT support in arm_arch_timer: 1. Move some enums and marcos to header file; 2. Add a new enum for spi type; 3. Improve printk relevant code; 4. Rename some enums and defines;

[PATCH v18 01/15] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-12-08 Thread fu . wei
From: Fu Wei <fu@linaro.org> To support the arm_arch_timer via ACPI we need to share defines and enums between the driver and the ACPI parser code. Split out the relevant defines and enums into arm_arch_timer.h, and change "enum ppi_nr" to "enum arch_timer_ppi_nr" t

[PATCH v18 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-12-08 Thread fu . wei
From: Fu Wei This patchset: (1)Preparation for adding GTDT support in arm_arch_timer: 1. Move some enums and marcos to header file; 2. Add a new enum for spi type; 3. Improve printk relevant code; 4. Rename some enums and defines; 5. Rework PPI

[PATCH v18 01/15] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-12-08 Thread fu . wei
From: Fu Wei To support the arm_arch_timer via ACPI we need to share defines and enums between the driver and the ACPI parser code. Split out the relevant defines and enums into arm_arch_timer.h, and change "enum ppi_nr" to "enum arch_timer_ppi_nr" to avoid the potentia

Re: [Linaro-acpi] [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-12-08 Thread Fu Wei
Hi Mark, On 8 December 2016 at 19:04, Mark Rutland <mark.rutl...@arm.com> wrote: > On Thu, Dec 08, 2016 at 11:16:21AM +0800, Fu Wei wrote: >> Hi Timur, >> >> On 8 December 2016 at 01:25, Timur Tabi <ti...@codeaurora.org> wrote: >> > On Fri, Nov 25,

Re: [Linaro-acpi] [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-12-08 Thread Fu Wei
Hi Mark, On 8 December 2016 at 19:04, Mark Rutland wrote: > On Thu, Dec 08, 2016 at 11:16:21AM +0800, Fu Wei wrote: >> Hi Timur, >> >> On 8 December 2016 at 01:25, Timur Tabi wrote: >> > On Fri, Nov 25, 2016 at 9:06 AM, Fu Wei wrote: >> >> >&

Re: [Linaro-acpi] [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-12-07 Thread Fu Wei
Hi Timur, On 8 December 2016 at 01:25, Timur Tabi <ti...@codeaurora.org> wrote: > On Fri, Nov 25, 2016 at 9:06 AM, Fu Wei <fu@linaro.org> wrote: >> >> a "+ int ret;" should be move from [12/15] to here, I have fix the >> problem in my repo, i

Re: [Linaro-acpi] [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-12-07 Thread Fu Wei
Hi Timur, On 8 December 2016 at 01:25, Timur Tabi wrote: > On Fri, Nov 25, 2016 at 9:06 AM, Fu Wei wrote: >> >> a "+ int ret;" should be move from [12/15] to here, I have fix the >> problem in my repo, it would happen in next patchset >> >> https://gi

Re: [PATCH v2] ACPI / APEI: Fix NMI notification handling

2016-12-02 Thread Fu Wei
afael :-) > > Thanks, > Rafael > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v2] ACPI / APEI: Fix NMI notification handling

2016-12-02 Thread Fu Wei
Rafael > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best regards, Fu Wei Software Engineer Red Hat

[PATCH v15] acpi, apei, arm64: APEI initial support for aarch64.

2016-12-01 Thread fu . wei
ribute is using efi_mem_attributes on ARM64. [Fu Wei: improve && upstream] Signed-off-by: Tomasz Nowicki <tomasz.nowi...@linaro.org> Tested-by: Jonathan (Zhixiong) Zhang <zjzh...@codeaurora.org> Signed-off-by: Fu Wei <fu@linaro.org> Acked-by: Hanjun Guo <hanj

[PATCH v15] acpi, apei, arm64: APEI initial support for aarch64.

2016-12-01 Thread fu . wei
efi_mem_attributes on ARM64. [Fu Wei: improve && upstream] Signed-off-by: Tomasz Nowicki Tested-by: Jonathan (Zhixiong) Zhang Signed-off-by: Fu Wei Acked-by: Hanjun Guo Tested-by: Tyler Baicar Acked-by: Will Deacon Reviewed-by: Borislav Petkov --- This patchset has been tested on the following p

Re: [PATCH v14] acpi, apei, arm64: APEI initial support for aarch64.

2016-12-01 Thread Fu Wei
that: > > Reviewed-by: Borislav Petkov <b...@suse.de> Great thanks for your rapidly feedback :-) will send v15 with that fix immediately > > Thanks! > > -- > Regards/Gruss, > Boris. > > SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB > 21284 (AG Nürnberg) > -- -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v14] acpi, apei, arm64: APEI initial support for aarch64.

2016-12-01 Thread Fu Wei
for your rapidly feedback :-) will send v15 with that fix immediately > > Thanks! > > -- > Regards/Gruss, > Boris. > > SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB > 21284 (AG Nürnberg) > -- -- Best regards, Fu Wei Software Engineer Red Hat

Re: [Linaro-acpi] [PATCH v14] acpi, apei, arm64: APEI initial support for aarch64.

2016-11-30 Thread Fu Wei
ng efi_mem_attributes on ARM64. > > [Fu Wei: improve && upstream] > > Signed-off-by: Tomasz Nowicki <tomasz.nowi...@linaro.org> > Tested-by: Jonathan (Zhixiong) Zhang <zjzh...@codeaurora.org> > Signed-off-by: Fu Wei <fu@linaro.org> > Acked-by: Hanjun

Re: [Linaro-acpi] [PATCH v14] acpi, apei, arm64: APEI initial support for aarch64.

2016-11-30 Thread Fu Wei
ific bits for aarch64 > > Meanwhile, > (1)move HEST type (ACPI_HEST_TYPE_IA32_CORRECTED_CHECK) checking to > a generic place. > (2)select HAVE_ACPI_APEI when EFI and ACPI is set on ARM64, > because arch_apei_get_mem_attribute is using efi_mem_attributes on ARM64. > > [Fu Wei: im

Re: [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-11-25 Thread Fu Wei
e, please drop us a note to > help improve the system] > > url: > https://github.com/0day-ci/linux/commits/fu-wei-linaro-org/acpi-clocksource-add-GTDT-driver-and-GTDT-support-in-arm_arch_timer/20161125-17 > base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/lin

Re: [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-11-25 Thread Fu Wei
to > help improve the system] > > url: > https://github.com/0day-ci/linux/commits/fu-wei-linaro-org/acpi-clocksource-add-GTDT-driver-and-GTDT-support-in-arm_arch_timer/20161125-17 > base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git > linux-next

[PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> Because arch_timer_needs_of_probing is only for booting with device-tree, but arch_timer_common_init is a generic init call which shouldn't include the FW-specific code. It's better to put arch_timer_needs_of_probing into DT init function. But for per-cpu

[PATCH v17 04/15] clocksource/drivers/arm_arch_timer: rename some enums and defines.

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> Rename some enums and defines, to unify the format of enums and defines in arm_arch_timer.h, also update all the users of these enums and defines: drivers/clocksource/arm_arch_timer.c virt/kvm/arm/hyp/timer-sr.c No functional change. Signed-off-

[PATCH v17 04/15] clocksource/drivers/arm_arch_timer: rename some enums and defines.

2016-11-25 Thread fu . wei
From: Fu Wei Rename some enums and defines, to unify the format of enums and defines in arm_arch_timer.h, also update all the users of these enums and defines: drivers/clocksource/arm_arch_timer.c virt/kvm/arm/hyp/timer-sr.c No functional change. Signed-off-by: Fu Wei Tested

[PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

2016-11-25 Thread fu . wei
From: Fu Wei Because arch_timer_needs_of_probing is only for booting with device-tree, but arch_timer_common_init is a generic init call which shouldn't include the FW-specific code. It's better to put arch_timer_needs_of_probing into DT init function. But for per-cpu timer

[PATCH v17 01/15] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> To support the arm_arch_timer via ACPI we need to share defines and enums between the driver and the ACPI parser code. Split out the relevant defines and enums into arm_arch_timer.h, and change "enum ppi_nr" to "enum arch_timer_ppi_nr" t

[PATCH v17 01/15] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-11-25 Thread fu . wei
From: Fu Wei To support the arm_arch_timer via ACPI we need to share defines and enums between the driver and the ACPI parser code. Split out the relevant defines and enums into arm_arch_timer.h, and change "enum ppi_nr" to "enum arch_timer_ppi_nr" to avoid the potentia

[PATCH v17 06/15] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> Currently, the counter frequency detection call(arch_timer_detect_rate) combines all the ways to get counter frequency: device-tree property, system coprocessor register, MMIO timer. But in the most of use cases, we don't need all the ways to try: For e

[PATCH v17 06/15] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2016-11-25 Thread fu . wei
From: Fu Wei Currently, the counter frequency detection call(arch_timer_detect_rate) combines all the ways to get counter frequency: device-tree property, system coprocessor register, MMIO timer. But in the most of use cases, we don't need all the ways to try: For example, reading device-tree

[PATCH v17 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> On platforms booting with ACPI, architected memory-mapped timers' configuration data is provided by firmware through the ACPI GTDT static table. The clocksource architected timer kernel driver requires a firmware interface to collect timer configu

[PATCH v17 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patch adds support for parsing arch timer info 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.

[PATCH v17 15/15] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> This driver adds support for parsing SBSA Generic Watchdog timer in GTDT, parse all info in SBSA Generic Watchdog Structure in GTDT, and creating a platform device with that information. This allows the operating system to obtain device data from the re

[PATCH v17 12/15] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch update arm_arch_timer driver to use the function provided by the new GTDT driver of ACPI. By this way, arm_arch_timer.c can be simplified, and separate all the ACPI GTDT knowledge from this timer driver. Signed-off-by: Fu Wei <fu@linaro.or

[PATCH v17 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch refactor original memory-mapped timer init code: (1) Refactor "arch_timer_mem_init", make it become a common code for memory-mapped timer init. (2) Add a new function "arch_timer_mem_of_init" for DT init.

[PATCH v17 09/15] clocksource/drivers/arm_arch_timer: Introduce some new structs to prepare for GTDT

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch introduce two new structs: arch_timer_mem, arch_timer_mem_frame. And also introduce a new define: ARCH_TIMER_MEM_MAX_FRAMES These will be used for refactoring the memory-mapped timer init code to prepare for GTDT Signed-off-by: Fu W

[PATCH v17 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_needs_probing

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> When system init with device-tree, we don't know which node will be initialized first. And the code in arch_timer_common_init should wait until per-cpu timer and MMIO timer are both initialized. So we need arch_timer_needs_probing to detect the init

[PATCH v17 14/15] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch add memory-mapped timer register support by using the information provided by the new GTDT driver of ACPI. Signed-off-by: Fu Wei <fu@linaro.org> --- drivers/clocksource/arm_arch_timer.c | 35 --- 1 file

[PATCH v17 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-25 Thread fu . wei
From: Fu Wei On platforms booting with ACPI, architected memory-mapped timers' configuration data is provided by firmware through the ACPI GTDT static table. The clocksource architected timer kernel driver requires a firmware interface to collect timer configuration and configure its driver

[PATCH v17 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-25 Thread fu . wei
From: Fu Wei This patch adds support for parsing arch timer info 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

[PATCH v17 15/15] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-11-25 Thread fu . wei
From: Fu Wei This driver adds support for parsing SBSA Generic Watchdog timer in GTDT, parse all info in SBSA Generic Watchdog Structure in GTDT, and creating a platform device with that information. This allows the operating system to obtain device data from the resource of platform device

[PATCH v17 12/15] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-11-25 Thread fu . wei
From: Fu Wei The patch update arm_arch_timer driver to use the function provided by the new GTDT driver of ACPI. By this way, arm_arch_timer.c can be simplified, and separate all the ACPI GTDT knowledge from this timer driver. Signed-off-by: Fu Wei Signed-off-by: Hanjun Guo Tested

[PATCH v17 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-11-25 Thread fu . wei
From: Fu Wei The patch refactor original memory-mapped timer init code: (1) Refactor "arch_timer_mem_init", make it become a common code for memory-mapped timer init. (2) Add a new function "arch_timer_mem_of_init" for DT init. Signed-off-by: Fu Wei ---

[PATCH v17 09/15] clocksource/drivers/arm_arch_timer: Introduce some new structs to prepare for GTDT

2016-11-25 Thread fu . wei
From: Fu Wei The patch introduce two new structs: arch_timer_mem, arch_timer_mem_frame. And also introduce a new define: ARCH_TIMER_MEM_MAX_FRAMES These will be used for refactoring the memory-mapped timer init code to prepare for GTDT Signed-off-by: Fu Wei --- include/clocksource

[PATCH v17 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_needs_probing

2016-11-25 Thread fu . wei
From: Fu Wei When system init with device-tree, we don't know which node will be initialized first. And the code in arch_timer_common_init should wait until per-cpu timer and MMIO timer are both initialized. So we need arch_timer_needs_probing to detect the init status of system. But currently

[PATCH v17 14/15] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-11-25 Thread fu . wei
From: Fu Wei The patch add memory-mapped timer register support by using the information provided by the new GTDT driver of ACPI. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 35 --- 1 file changed, 32 insertions(+), 3 deletions(-) diff

[PATCH v17 05/15] clocksource/drivers/arm_arch_timer: rework PPI determination

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> Currently, the arch timer driver uses ARCH_TIMER_PHYS_SECURE_PPI to mean the driver will use the secure PPI *and* potentialy also use the non-secure PPI. This is somewhat confusing. For arm64, where it never makes sense to use the secure PPI, this means w

[PATCH v17 05/15] clocksource/drivers/arm_arch_timer: rework PPI determination

2016-11-25 Thread fu . wei
From: Fu Wei Currently, the arch timer driver uses ARCH_TIMER_PHYS_SECURE_PPI to mean the driver will use the secure PPI *and* potentialy also use the non-secure PPI. This is somewhat confusing. For arm64, where it never makes sense to use the secure PPI, this means we must always request

[PATCH v17 03/15] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patch defines pr_fmt(fmt) for all pr_* functions, then the pr_* doesn't need to add "arch_timer:" everytime. According to the suggestion from checkpatch.pl: (1) delete some Blank Spaces in arch_timer_banner; (2) delete a redundant Tab

[PATCH v17 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patchset: (1)Preparation for adding GTDT support in arm_arch_timer: 1. Move some enums and marcos to header file; 2. Add a new enum for spi type; 3. Improve printk relevant code; 4. Rename some enums and defines;

[PATCH v17 03/15] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-11-25 Thread fu . wei
From: Fu Wei This patch defines pr_fmt(fmt) for all pr_* functions, then the pr_* doesn't need to add "arch_timer:" everytime. According to the suggestion from checkpatch.pl: (1) delete some Blank Spaces in arch_timer_banner; (2) delete a redundant Tab in a bland line of arch_timer

[PATCH v17 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-11-25 Thread fu . wei
From: Fu Wei This patchset: (1)Preparation for adding GTDT support in arm_arch_timer: 1. Move some enums and marcos to header file; 2. Add a new enum for spi type; 3. Improve printk relevant code; 4. Rename some enums and defines; 5. Rework PPI

[PATCH v17 02/15] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-11-25 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patch add a new enum "arch_timer_spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei <fu@linaro.org> Acked-by: Mark Rutland <mark.rutl...@arm.com> --- drivers/cloc

[PATCH v17 02/15] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-11-25 Thread fu . wei
From: Fu Wei This patch add a new enum "arch_timer_spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei Acked-by: Mark Rutland --- drivers/clocksource/arm_arch_timer.c | 4 ++-- include/clocksource/arm_arch_timer.h | 6 +

Re: [PATCH v16 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-23 Thread Fu Wei
Hi Lorenzo, On 23 November 2016 at 19:53, Fu Wei <fu@linaro.org> wrote: > Hi Lorenzo, > > On 18 November 2016 at 22:22, Lorenzo Pieralisi > <lorenzo.pieral...@arm.com> wrote: >> On Wed, Nov 16, 2016 at 09:49:06PM +0800, fu....@linaro.org wrote: >&

Re: [PATCH v16 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-23 Thread Fu Wei
Hi Lorenzo, On 23 November 2016 at 19:53, Fu Wei wrote: > Hi Lorenzo, > > On 18 November 2016 at 22:22, Lorenzo Pieralisi > wrote: >> On Wed, Nov 16, 2016 at 09:49:06PM +0800, fu@linaro.org wrote: >>> From: Fu Wei >>> >>> On platforms bootin

Re: [PATCH v16 14/15] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-11-23 Thread Fu Wei
Hi Mark, On 19 November 2016 at 04:20, Mark Rutland <mark.rutl...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:49:07PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> The patch add memory-mapped timer register support by using the >&g

Re: [PATCH v16 14/15] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-11-23 Thread Fu Wei
Hi Mark, On 19 November 2016 at 04:20, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:49:07PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The patch add memory-mapped timer register support by using the >> information provided by the new GTDT driver of AC

Re: [PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-23 Thread Fu Wei
nb); >> int acpi_reconfig_notifier_unregister(struct notifier_block *nb); >> >> +#ifdef CONFIG_ACPI_GTDT >> +int acpi_gtdt_init(struct acpi_table_header *table); >> +int acpi_gtdt_map_ppi(int type); >> +bool acpi_gtdt_c3stop(int type); >> +void acpi_gtdt_release(void); > > Why do these need to be here? > > What possible value is ther in exporting acpi_gtdt_release() !? because I need to release these after mem timer init, But like your comment in 14/15, if I fill in the entire arch_timer_mem in one go, I don't need to export acpi_gtdt_release. Will try this way in v17 > > Thanks, > Mark. -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-23 Thread Fu Wei
ier_unregister(struct notifier_block *nb); >> >> +#ifdef CONFIG_ACPI_GTDT >> +int acpi_gtdt_init(struct acpi_table_header *table); >> +int acpi_gtdt_map_ppi(int type); >> +bool acpi_gtdt_c3stop(int type); >> +void acpi_gtdt_release(void); > > Why do these need to be here? > > What possible value is ther in exporting acpi_gtdt_release() !? because I need to release these after mem timer init, But like your comment in 14/15, if I fill in the entire arch_timer_mem in one go, I don't need to export acpi_gtdt_release. Will try this way in v17 > > Thanks, > Mark. -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v16 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-23 Thread Fu Wei
Hi Lorenzo, On 18 November 2016 at 22:22, Lorenzo Pieralisi <lorenzo.pieral...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:49:06PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> On platforms booting with ACPI, architected memory-map

Re: [PATCH v16 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-23 Thread Fu Wei
Hi Lorenzo, On 18 November 2016 at 22:22, Lorenzo Pieralisi wrote: > On Wed, Nov 16, 2016 at 09:49:06PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> On platforms booting with ACPI, architected memory-mapped timers' >> configuration data is provided by fir

Re: [PATCH v16 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-11-22 Thread Fu Wei
Hi Mark, On 19 November 2016 at 04:03, Mark Rutland <mark.rutl...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:49:03PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> The patch refactor original memory-mapped timer init code:

Re: [PATCH v16 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-11-22 Thread Fu Wei
Hi Mark, On 19 November 2016 at 04:03, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:49:03PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The patch refactor original memory-mapped timer init code: >> (1) Extract a subfunction for

Re: [PATCH v16 08/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_needs_probing, and call it only if acpi disabled.

2016-11-21 Thread Fu Wei
Hi Mark, On 19 November 2016 at 03:56, Mark Rutland <mark.rutl...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:49:01PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> The patch refactor original arch_timer_needs_probing

Re: [PATCH v16 08/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_needs_probing, and call it only if acpi disabled.

2016-11-21 Thread Fu Wei
Hi Mark, On 19 November 2016 at 03:56, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:49:01PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The patch refactor original arch_timer_needs_probing function: >> (1) Separate out arch_timer_needs_probing

Re: [PATCH v16 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_detect_rate to keep dt code in *_of_init

2016-11-21 Thread Fu Wei
Hi Mark On 19 November 2016 at 03:52, Mark Rutland <mark.rutl...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:49:00PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> The patch refactor original arch_timer_detect_rate function: >>

Re: [PATCH v16 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_detect_rate to keep dt code in *_of_init

2016-11-21 Thread Fu Wei
Hi Mark On 19 November 2016 at 03:52, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:49:00PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The patch refactor original arch_timer_detect_rate function: >> (1) Separate out device-tree code, ke

Re: [PATCH v16 06/15] clocksource/drivers/arm_arch_timer: separate out arch_timer_uses_ppi init code to prepare for GTDT.

2016-11-21 Thread Fu Wei
Hi Mark, On 19 November 2016 at 03:30, Mark Rutland <mark.rutl...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:48:59PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> The patch refactor original arch_timer_uses_ppi init c

Re: [PATCH v16 06/15] clocksource/drivers/arm_arch_timer: separate out arch_timer_uses_ppi init code to prepare for GTDT.

2016-11-21 Thread Fu Wei
Hi Mark, On 19 November 2016 at 03:30, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:48:59PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The patch refactor original arch_timer_uses_ppi init code: >> (1) Extract a subfunction: arch_timer_uses_ppi_init >

Re: [PATCH v16 05/15] clocksource/drivers/arm_arch_timer: fix a bug in arch_timer_register about arch_timer_uses_ppi

2016-11-20 Thread Fu Wei
Hi Mark, On 19 November 2016 at 02:52, Mark Rutland <mark.rutl...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:48:58PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> The patch fix a potential bug about arch_timer_uses_ppi in >> arc

Re: [PATCH v16 05/15] clocksource/drivers/arm_arch_timer: fix a bug in arch_timer_register about arch_timer_uses_ppi

2016-11-20 Thread Fu Wei
Hi Mark, On 19 November 2016 at 02:52, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:48:58PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The patch fix a potential bug about arch_timer_uses_ppi in >> arch_timer_register. >> On ARM64, we don't use AR

Re: [PATCH v16 04/15] clocksource/drivers/arm_arch_timer: rename some enums and defines, and some cleanups.

2016-11-20 Thread Fu Wei
Hi Mark, On 19 November 2016 at 02:49, Mark Rutland <mark.rutl...@arm.com> wrote: > On Wed, Nov 16, 2016 at 09:48:57PM +0800, fu@linaro.org wrote: >> From: Fu Wei <fu@linaro.org> >> >> Rename some enums and defines, to unify the format of enums and def

Re: [PATCH v16 04/15] clocksource/drivers/arm_arch_timer: rename some enums and defines, and some cleanups.

2016-11-20 Thread Fu Wei
Hi Mark, On 19 November 2016 at 02:49, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:48:57PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> Rename some enums and defines, to unify the format of enums and defines >> in arm_arch_timer.h, also update a

Re: [PATCH v16 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-11-17 Thread Fu Wei
hich function well on D05 board. Thanks for your testing, So I will apply your Tested-by in 1~8, 11, 12, 15 in my next patchset. > > On 2016/11/17 11:34, Xiongfeng Wang wrote: >> Tested-by: wangxiongfe...@huawei.com on D05 board. >> >> >> On 2016/11/16 21:48, fu@linar

Re: [PATCH v16 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-11-17 Thread Fu Wei
nks for your testing, So I will apply your Tested-by in 1~8, 11, 12, 15 in my next patchset. > > On 2016/11/17 11:34, Xiongfeng Wang wrote: >> Tested-by: wangxiongfe...@huawei.com on D05 board. >> >> >> On 2016/11/16 21:48, fu@linaro.org wrote: >>&

[PATCH v16 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> On platforms booting with ACPI, architected memory-mapped timers' configuration data is provided by firmware through the ACPI GTDT static table. The clocksource architected timer kernel driver requires a firmware interface to collect timer configu

[PATCH v16 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_detect_rate to keep dt code in *_of_init

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch refactor original arch_timer_detect_rate function: (1) Separate out device-tree code, keep them in device-tree init function: arch_timer_of_init, arch_timer_mem_init; (2) Improve original mechanism, if getting from

[PATCH v16 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-11-16 Thread fu . wei
From: Fu Wei On platforms booting with ACPI, architected memory-mapped timers' configuration data is provided by firmware through the ACPI GTDT static table. The clocksource architected timer kernel driver requires a firmware interface to collect timer configuration and configure its driver

[PATCH v16 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_detect_rate to keep dt code in *_of_init

2016-11-16 Thread fu . wei
From: Fu Wei The patch refactor original arch_timer_detect_rate function: (1) Separate out device-tree code, keep them in device-tree init function: arch_timer_of_init, arch_timer_mem_init; (2) Improve original mechanism, if getting from memory-mapped timer fail

[PATCH v16 14/15] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch add memory-mapped timer register support by using the information provided by the new GTDT driver of ACPI. Signed-off-by: Fu Wei <fu@linaro.org> --- drivers/clocksource/arm_arch_timer.c | 26 +- 1 file changed, 2

[PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-16 Thread fu . wei
From: Fu Wei <fu@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. Sign

[PATCH v16 15/15] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> This driver adds support for parsing SBSA Generic Watchdog timer in GTDT, parse all info in SBSA Generic Watchdog Structure in GTDT, and creating a platform device with that information. This allows the operating system to obtain device data from the re

[PATCH v16 14/15] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-11-16 Thread fu . wei
From: Fu Wei The patch add memory-mapped timer register support by using the information provided by the new GTDT driver of ACPI. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git

[PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-16 Thread fu . wei
From: Fu Wei 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 Signed

[PATCH v16 15/15] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-11-16 Thread fu . wei
From: Fu Wei This driver adds support for parsing SBSA Generic Watchdog timer in GTDT, parse all info in SBSA Generic Watchdog Structure in GTDT, and creating a platform device with that information. This allows the operating system to obtain device data from the resource of platform device

[PATCH v16 12/15] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch update arm_arch_timer driver to use the function provided by the new GTDT driver of ACPI. By this way, arm_arch_timer.c can be simplified, and separate all the ACPI GTDT knowledge from this timer driver. Signed-off-by: Fu Wei <fu@linaro.or

[PATCH v16 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch refactor original memory-mapped timer init code: (1) Extract a subfunction for detecting a bast time frame: is_best_frame. (2) Refactor "arch_timer_mem_init", make it become a common code for memory-mapped timer i

[PATCH v16 12/15] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-11-16 Thread fu . wei
From: Fu Wei The patch update arm_arch_timer driver to use the function provided by the new GTDT driver of ACPI. By this way, arm_arch_timer.c can be simplified, and separate all the ACPI GTDT knowledge from this timer driver. Signed-off-by: Fu Wei Signed-off-by: Hanjun Guo --- drivers

[PATCH v16 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-11-16 Thread fu . wei
From: Fu Wei The patch refactor original memory-mapped timer init code: (1) Extract a subfunction for detecting a bast time frame: is_best_frame. (2) Refactor "arch_timer_mem_init", make it become a common code for memory-mapped timer init. (3) Add a ne

[PATCH v16 09/15] clocksource/drivers/arm_arch_timer: Introduce some new structs to prepare for GTDT

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch introduce two new structs: arch_timer_mem, arch_timer_mem_frame. And also introduce a new define: ARCH_TIMER_MEM_MAX_FRAMES These will be used for refactoring the memory-mapped timer init code to prepare for GTDT Signed-off-by:

[PATCH v16 06/15] clocksource/drivers/arm_arch_timer: separate out arch_timer_uses_ppi init code to prepare for GTDT.

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch refactor original arch_timer_uses_ppi init code: (1) Extract a subfunction: arch_timer_uses_ppi_init (2) Use the new subfunction in arch_timer_of_init and arch_timer_acpi_init Signed-off-by: Fu Wei <fu@linaro.org> --- drivers

[PATCH v16 03/15] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> This patch defines pr_fmt(fmt) for all pr_* functions, then the pr_* doesn't need to add "arch_timer:" everytime. According to the suggestion from checkpatch.pl: (1) delete some Blank Spaces in arch_timer_banner; (2) delete a redundant Tab

[PATCH v16 08/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_needs_probing, and call it only if acpi disabled.

2016-11-16 Thread fu . wei
From: Fu Wei <fu@linaro.org> The patch refactor original arch_timer_needs_probing function: (1) Separate out arch_timer_needs_probing from arch_timer_common_init, and call it only if acpi disabled. (2) Rename arch_timer_needs_probing to arch_timer_needs_of_probing. Sign

<    1   2   3   4   5   6   7   8   9   10   >