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

2016-11-16 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_

[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 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. Signed-off-by: Fu Wei

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

2016-11-16 Thread fu . wei
From: Fu Wei The patch fix a potential bug about arch_timer_uses_ppi in arch_timer_register. On ARM64, we don't use ARCH_TIMER_PHYS_SECURE_PPI in Linux, so we will just igorne it in init code. If arch_timer_uses_ppi is ARCH_TIMER_PHYS_NONSECURE_PPI, the orignal code of arch_timer_uses_ppi m

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

2016-11-16 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 v16 04/15] clocksource/drivers/arm_arch_timer: rename some enums and defines, and some cleanups.

2016-11-16 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 And do some cleanups, according to the suggestion from

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

2016-11-16 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 v16 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-11-16 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, and some cleanups. 5

Re: [PATCH v15 01/13] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-11-15 Thread Fu Wei
t; > url: > https://github.com/0day-ci/linux/commits/fu-wei-linaro-org/acpi-clocksource-add-GTDT-driver-and-GTDT-support-in-arm_arch_timer/20161116-00 > config: arm-hisi_defconfig (attached as .config) > compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705 >

[PATCH v15 12/13] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-11-15 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 a

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

2016-11-15 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 d

[PATCH v15 13/13] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-11-15 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 v15 06/13] clocksource/drivers/arm_arch_timer: separate out arch_timer_uses_ppi init code to prepare for GTDT.

2016-11-15 Thread fu . wei
From: Fu Wei 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 --- drivers/clocksource/arm_arch_timer.c | 73

[PATCH v15 10/13] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-11-15 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 v15 08/13] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-11-15 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 v15 09/13] acpi/arm64: Add GTDT table parse driver

2016-11-15 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 v15 07/13] clocksource/drivers/arm_arch_timer: Introduce some new structs to prepare for GTDT

2016-11-15 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

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

2016-11-15 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 potent

[PATCH v15 04/13] clocksource/drivers/arm_arch_timer: rename some enums and defines, and some cleanups.

2016-11-15 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 And do some cleanups, according to the suggestion from

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

2016-11-15 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, and some cleanups. 5

[PATCH v15 03/13] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-11-15 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_

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

2016-11-15 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 v15 05/13] clocksource/drivers/arm_arch_timer: fix a bug in arch_timer_register about arch_timer_uses_ppi

2016-11-15 Thread fu . wei
From: Fu Wei The patch fix a potential bug about arch_timer_uses_ppi in arch_timer_register. On ARM64, we don't use ARCH_TIMER_PHYS_SECURE_PPI in Linux, so we will just igorne it in init code. If arch_timer_uses_ppi is ARCH_TIMER_PHYS_NONSECURE_PPI, the orignal code of arch_timer_uses_ppi m

Re: [PATCH v14 7/9] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-10-26 Thread Fu Wei
Hi Mark, On 26 October 2016 at 23:46, Mark Rutland wrote: > On Wed, Oct 26, 2016 at 11:24:32PM +0800, Fu Wei wrote: >> On 21 October 2016 at 19:32, Mark Rutland wrote: >> > On Thu, Sep 29, 2016 at 02:17:15AM +0800, fu@linaro.org wrote: >> >> +static int __init

Re: [PATCH v14 7/9] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-10-26 Thread Fu Wei
Hi Mark, On 21 October 2016 at 19:32, Mark Rutland wrote: > On Thu, Sep 29, 2016 at 02:17:15AM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The patch refactor original memory-mapped timer init code: >> (1) extract some subfunction for reusing some common

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

2016-10-26 Thread Fu Wei
Hi Marc, On 26 October 2016 at 20:11, Marc Zyngier wrote: > On 26/10/16 12:10, Fu Wei wrote: >> Hi Mark, >> >> On 21 October 2016 at 00:37, Mark Rutland wrote: >>> Hi, >>> >>> As a heads-up, on v4.9-rc1 I see conflicts at least against >>&g

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

2016-10-26 Thread Fu Wei
return -EINVAL; >> + } >> + acpi_gtdt_desc.platform_timer_start = start; >> + >> + return gtdt->platform_timer_count; >> +} > > This is never used as anything other than a status code. > > Just return zero; we haven't parsed the platform timers themselves at > this point anyway. Sorry, in my driver, I use this return value to inform driver negative number : error 0 : we don't have platform timer in GTDT table. positive number: the number of platform timer we have in GTDT table. please correct me, if I am doing it in wrong way. Thanks :-) > > Thanks, > Mark. -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v14 1/9] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-10-26 Thread Fu Wei
Hi Mark On 26 October 2016 at 18:51, Mark Rutland wrote: > On Wed, Oct 26, 2016 at 04:31:55PM +0800, Fu Wei wrote: >> On 20 October 2016 at 22:45, Mark Rutland wrote: >> > On Thu, Sep 29, 2016 at 02:17:09AM +0800, fu@linaro.org wrote: >> >> #include >>

Re: [PATCH v14 5/9] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-10-26 Thread Fu Wei
t; Please ignore this point; sorry for the noise. > > Actually, I misread the spec this time around; the flag *can* differ per > interrupt for the sysreg/cp15 timer, but not for the MMIO timers where > the flag is in a common field. > > So please *do* consider the above. yes , you are right , will do Thanks :-) > > Thanks, > Mark. -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v14 1/9] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-10-26 Thread Fu Wei
d add a patch at the end of this patchset to fix it, OK ? Thanks for ACK :-) > > Thanks, > Mark. > >> + >> #define ARCH_TIMER_PHYS_ACCESS 0 >> #define ARCH_TIMER_VIRT_ACCESS 1 >> #define ARCH_TIMER_MEM_PHYS_ACCESS 2 >> -- >> 2.7.4 >> -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v14 3/9] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-10-26 Thread Fu Wei
: "", >> + type & ARCH_MEM_TIMER ? >> + arch_timer_mem_use_virtual ? "virt" : "phys" : >> + ""); > > ... and these two. > > No matter what checkpatch says, I prefer the code that way so as to keep > the ternary clear. yes, NP, I have fixed it :-) > > [...] > >> @@ -768,7 +769,7 @@ static int __init arch_timer_init(void) >> return ret; >> >> arch_timer_kvm_info.virtual_irq = arch_timer_ppi[VIRT_PPI]; >> - >> + > > Please mention the whitespace fixup in the commit message. It's > surprising otherwise. OK, added this message. > > With all that: > > Acked-by: Mark Rutland Great thanks :-) > > Thanks, > Mark. -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v14 2/9] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-10-26 Thread Fu Wei
gt; Please rename this to arch_timer_spi_nr (as with patch 1 for > s/ppi_nr/arch_timer_ppi_nr/). With that: > > Acked-by: Mark Rutland OK, NP, done :-) Great thanks! > > Thanks, > Mark. -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v14 0/9] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-10-26 Thread Fu Wei
Hi Mark On 20 October 2016 at 23:17, Mark Rutland wrote: > On Thu, Oct 20, 2016 at 03:57:52PM +0100, Lorenzo Pieralisi wrote: >> On Thu, Oct 20, 2016 at 03:31:01PM +0100, Mark Rutland wrote: >> > Hi, >> > >> > On Thu, Sep 29, 2016 at 02:17:08AM +0800, fu....@l

Re: [PATCH v14 0/9] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-10-05 Thread Fu Wei
Hi Xiongfeng Wang, On 30 September 2016 at 08:40, Xiongfeng Wang wrote: > for sbsa watchdog part, Tested-by: wangxiongfe...@huawei.com on D05 board. Got it, thanks again for your help > > On 2016/9/29 2:17, fu@linaro.org wrote: >> From: Fu Wei >> >> This patc

Re: [PATCH v12 0/8] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-09-29 Thread Fu Wei
Hi Wang Xiongfeng On 29 September 2016 at 16:33, wangxiongfeng (C) wrote: > for sbsa watchdog part, Tested-by: wangxiongfe...@huawei.com on D05 board. Great thanks for your testing. :-) > > On 2016/9/13 18:38, fu@linaro.org wrote: >> From: Fu Wei >> >&g

[PATCH v14 7/9] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT

2016-09-28 Thread fu . wei
From: Fu Wei The patch refactor original memory-mapped timer init code: (1) extract some subfunction for reusing some common code a. get_cnttidr b. is_best_frame (2) move base address and irq code for arch_timer_mem to arch_timer_mem_register Signed-off-by: Fu Wei --- drivers

[PATCH v14 3/9] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-09-28 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. Also delete some Blank Spaces in arch_timer_banner, according to the suggestion from checkpatch.pl. No functional change. Signed-off-by: Fu We

[PATCH v14 9/9] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-09-28 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 v14 8/9] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-09-28 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 | 92 +--- 1 file changed, 85 insertions(+), 7 deletions(-) diff

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

2016-09-28 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 v14 1/9] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-09-28 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. No functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 11

[PATCH v14 5/9] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-28 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 v14 6/9] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-09-28 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 d

[PATCH v14 0/9] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-09-28 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. (2)Introduce ACPI GTDT parser: drivers/acpi/arm64/acpi_gtdt.c

[PATCH v14 2/9] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-09-28 Thread fu . wei
From: Fu Wei This patch add a new enum "spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 4 ++-- include/clocksource/arm_arch_timer.h | 6 ++ 2 files changed, 8 insertions(

Re: [PATCH v13 7/8] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-09-15 Thread Fu Wei
HI On 15 September 2016 at 04:09, wrote: > From: Fu Wei > > The patch add memory-mapped timer register support by using the information > provided by the new GTDT driver of ACPI. > Also refactor original memory-mapped timer dt support for reusing some common > code. > &g

[PATCH v13 8/8] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-09-14 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 v13 7/8] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-09-14 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. Also refactor original memory-mapped timer dt support for reusing some common code. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 238

[PATCH v13 4/8] acpi/arm64: Add GTDT table parse driver

2016-09-14 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 v13 6/8] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-09-14 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 d

[PATCH v13 1/8] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-09-14 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. No functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 11

[PATCH v13 2/8] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-09-14 Thread fu . wei
From: Fu Wei This patch add a new enum "spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 4 ++-- include/clocksource/arm_arch_timer.h | 6 ++ 2 files changed, 8 insertions(

[PATCH v13 0/8] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-09-14 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. (2)Introduce ACPI GTDT parser: drivers/acpi/arm64/acpi_gtdt.c

[PATCH v13 3/8] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-09-14 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. Also delete some Blank Spaces in arch_timer_banner, according to the suggestion from checkpatch.pl. No functional change. Signed-off-by: Fu We

[PATCH v13 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-14 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

Re: [PATCH v11 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-13 Thread Fu Wei
Hi Timur On 09/13/2016 07:38 PM, Timur Tabi wrote: > Fu Wei wrote: >> I have prepared v12 (rebase to rc6 and on the top of IORT v11), >> should I send it now > > Yes. > > Please don't wait to release new versions of your patches. Time is running > out t

Re: [PATCH v12 7/8] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-09-13 Thread Fu Wei
Hi Marc, On 09/13/2016 07:00 PM, Marc Zyngier wrote: > Argh, new version... > > On 13/09/16 11:39, 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

[PATCH v12 6/8] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-09-13 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 d

[PATCH v12 4/8] acpi/arm64: Add GTDT table parse driver

2016-09-13 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 v12 8/8] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-09-13 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 v12 7/8] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-09-13 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 | 127 ++- 1 file changed, 124 insertions(+), 3 deletions(-) diff

[PATCH v12 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-13 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 v12 3/8] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-09-13 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. Also delete some Blank Spaces in arch_timer_banner, according to the suggestion from checkpatch.pl. No functional change. Signed-off-by: Fu We

[PATCH v12 2/8] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-09-13 Thread fu . wei
From: Fu Wei This patch add a new enum "spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 4 ++-- include/clocksource/arm_arch_timer.h | 6 ++ 2 files changed, 8 insertions(

[PATCH v12 0/8] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-09-13 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. (2)Introduce ACPI GTDT parser: drivers/acpi/arm64/acpi_gtdt.c

[PATCH v12 1/8] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-09-13 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. No functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 11

Re: [PATCH v11 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-13 Thread Fu Wei
Hi Mark, Marc, Sorry for missing you in the cc list Do you have any suggestion for the arm_arch_timer patches? Could you help me to review these patches ? Great thanks ! On 13 September 2016 at 17:22, Fu Wei wrote: > Hi Thomas, Daniel, > > For these arm_arch_timer patches, do you

Re: [PATCH v11 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-13 Thread Fu Wei
anything I can do to improve this patchset ? Thanks. On 7 September 2016 at 17:23, Fu Wei wrote: > Hi Thomas > > On 6 September 2016 at 22:36, Thomas Gleixner wrote: >> On Tue, 6 Sep 2016, fu@linaro.org wrote: >>> + if (timer_count < 0) >>> +

Re: [PATCH v11 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-07 Thread Fu Wei
s ), system still can work. So I thing we just need to print a error. > >> - arch_timer_init(); >> - return 0; >> + return arch_timer_init(); > > Thanks, > > tglx -- Best regards, Fu Wei Software Engineer Red Hat

[PATCH v11 7/8] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer

2016-09-06 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 | 127 ++- 1 file changed, 124 insertions(+), 3 deletions(-) diff

[PATCH v11 2/8] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-09-06 Thread fu . wei
From: Fu Wei This patch add a new enum "spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 4 ++-- include/clocksource/arm_arch_timer.h | 6 ++ 2 files changed, 8 insertions(

[PATCH v11 3/8] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-09-06 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. Also delete some Blank Spaces in arch_timer_banner, according to the suggestion from checkpatch.pl. No functional change. Signed-off-by: Fu We

[PATCH v11 1/8] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

2016-09-06 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. No functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 11

[PATCH v11 8/8] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-09-06 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 v11 0/8] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-09-06 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. (2)Introduce ACPI GTDT parser: drivers/acpi/arm64/acpi_gtdt.c

[PATCH v11 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-09-06 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 v11 6/8] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-09-06 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 d

[PATCH v11 4/8] acpi/arm64: Add GTDT table parse driver

2016-09-06 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

Re: [Linaro-acpi] [PATCH V8 1/8] ACPI: I/O Remapping Table (IORT) initial support

2016-09-05 Thread Fu Wei
Hi Tomasz, On 5 September 2016 at 14:12, Tomasz Nowicki wrote: > On 02.09.2016 13:52, Fu Wei wrote: >> >> Hi Tomasz, >> >> On 11 August 2016 at 18:06, Tomasz Nowicki wrote: >>> >>> IORT shows representation of IO topology for ARM based systems

Re: [Linaro-acpi] [PATCH V8 1/8] ACPI: I/O Remapping Table (IORT) initial support

2016-09-02 Thread Fu Wei
--- /dev/null > +++ b/include/linux/iort.h > @@ -0,0 +1,30 @@ > +/* > + * Copyright (C) 2016, Semihalf > + * Author: Tomasz Nowicki > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms and conditions of the GNU General Public License, > + * version 2, as published by the Free Software Foundation. > + * > + * This program is distributed in the hope 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. > + * > + * You should have received a copy of the GNU General Public License along > with > + * this program; if not, write to the Free Software Foundation, Inc., 59 > Temple > + * Place - Suite 330, Boston, MA 02111-1307 USA. > + */ > + > +#ifndef __IORT_H__ > +#define __IORT_H__ > + > +#include > + > +#ifdef CONFIG_IORT_TABLE > +void iort_table_detect(void); > +#else > +static inline void iort_table_detect(void) { } > +#endif > + > +#endif /* __IORT_H__ */ > -- > 1.9.1 > > ___ > Linaro-acpi mailing list > linaro-a...@lists.linaro.org > https://lists.linaro.org/mailman/listinfo/linaro-acpi -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v10 0/8] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-08-18 Thread Fu Wei
rce maintainer will take care of it, and GTDT patch has got Rafael's ACK. > > -- > Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm > Technologies, Inc. Qualcomm Technologies, Inc. is a member of the > Code Aurora Forum, a Linux Foundation Collaborative Project. -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v10 0/8] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-08-11 Thread Fu Wei
t on the master branch of upstream kernel (2)rebase my v10 on the top of Tomasz's v8 (3)git format-patch as v11, repost it Is that OK for everyone? :-) > > But both the GTDT patchset and IORT are in good shape now, can we > targeting both of them for 4.9 kernel (since all go via tip tree)? > > Thanks > Hanjun -- Best regards, Fu Wei Software Engineer Red Hat

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

2016-08-10 Thread Fu Wei
Hi Borislav, On 10 August 2016 at 18:45, Borislav Petkov wrote: > On Wed, Aug 10, 2016 at 06:40:53PM +0800, Fu Wei wrote: >> But the reason for a separate hest_ia32_init() is: >> For now(ACPI 6.1), we have three IA-32 Architecture-specific error >> source structures, may

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

2016-08-10 Thread fu . wei
CHECK) checking to a generic place. (3)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: improve && upstream] Signed-off-by: Tomasz Nowicki Tested-by: Jonathan (Zhixiong) Zhang Signed-off-by: Fu Wei

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

2016-08-10 Thread Fu Wei
Hi Borislav, On 10 August 2016 at 19:10, Borislav Petkov wrote: > On Wed, Aug 10, 2016 at 07:01:05PM +0800, Fu Wei wrote: >> - if (!acpi_disable_cmcff) >> - apei_hest_parse(hest_parse_cmc, NULL); >> + rc =apei_hest_parse(hest_parse_cmc, NULL); >> + if (rc

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

2016-08-10 Thread Fu Wei
gt; > (2)move HEST type (ACPI_HEST_TYPE_IA32_CORRECTED_CHECK) checking to >> > a generic place. >> > (3)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. >> > >> &

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

2016-08-10 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 --- This patchset has been tested on the following platforms: (1)ARM Foundation

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

2016-08-10 Thread Fu Wei
I_APEI when EFI and ACPI is set on ARM64, >> because arch_apei_get_mem_attribute is using efi_mem_attributes on ARM64. >> >> [Fu Wei: improve && upstream] >> >> Signed-off-by: Tomasz Nowicki >> Tested-by: Jonathan (Zhixiong) Zhang >> Signed-off-by: Fu Wei >&

Re: [PATCH v9 0/9] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-08-09 Thread Fu Wei
quire who is going to pull > it in ? the latest v10 patchset can apply on v4.8-rc1: https://lkml.org/lkml/2016/7/26/215 > > Thanks, > Tomasz > > > On 25.07.2016 17:26, fu....@linaro.org wrote: >> >> From: Fu Wei >> >> This patchset: >> (

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

2016-07-29 Thread fu . wei
CHECK) checking to a generic place. (3)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: improve && upstream] Signed-off-by: Tomasz Nowicki Tested-by: Jonathan (Zhixiong) Zhang Signed-off-by: Fu Wei

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

2016-07-28 Thread Fu Wei
PI is set on ARM64, >> because arch_apei_get_mem_attribute is using efi_mem_attributes on ARM64. >> >> [Fu Wei: improve && upstream] >> > > >> diff --git a/arch/arm64/include/asm/acpi.h b/arch/arm64/include/asm/acpi.h >> index 5420cb0..d3d02dc 1006

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

2016-07-27 Thread Fu Wei
ACPI is set on ARM64, >> because arch_apei_get_mem_attribute is using efi_mem_attributes on ARM64. >> >> [Fu Wei: improve && upstream] >> >> Signed-off-by: Tomasz Nowicki >> Tested-by: Jonathan (Zhixiong) Zhang >> Signed-off-by: Fu Wei >>

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

2016-07-27 Thread fu . wei
CHECK) checking to a generic place. (3)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: improve && upstream] Signed-off-by: Tomasz Nowicki Tested-by: Jonathan (Zhixiong) Zhang Signed-off-by: Fu Wei

Re: [PATCH v9 4/9] clocksource/drivers/arm_arch_timer: use readq to get 64-bit CNTVCT

2016-07-26 Thread Fu Wei
ately, because this GTDT patchset can work without it, this readq support is a optimizing. I also can see another arm-related driver are using readq in this way( #ifdef readq): bus/arm-ccn.c And some other drivers are also doing this. > > I measured the performance on berlin arm64 platforms: > > compared with original version, using readq_relaxed could reduce > time of arch_counter_get_cntvct_mem() by about 42%! Great thanks for your data, :-) > > Thanks, > Jisheng -- Best regards, Fu Wei Software Engineer Red Hat

Re: [PATCH v9 5/9] acpi/arm64: Add GTDT table parse driver

2016-07-26 Thread Fu Wei
Hi Rafael, On 26 July 2016 at 19:50, Rafael J. Wysocki wrote: > On Monday, July 25, 2016 11:27:03 PM fu@linaro.org wrote: >> From: Fu Wei >> >> This patch adds support for parsing arch timer in GTDT, >> provides some kernel APIs to parse all the PPIs and >>

[PATCH v10 8/8] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver

2016-07-26 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 v10 5/8] clocksource/drivers/arm_arch_timer: Simplify ACPI support code.

2016-07-26 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 v10 2/8] clocksource/drivers/arm_arch_timer: Add a new enum for spi type

2016-07-26 Thread fu . wei
From: Fu Wei This patch add a new enum "spi_nr" and use it in the driver. Just for code's readability, no functional change. Signed-off-by: Fu Wei --- drivers/clocksource/arm_arch_timer.c | 4 ++-- include/clocksource/arm_arch_timer.h | 6 ++ 2 files changed, 8 insertions(

[PATCH v10 3/8] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-07-26 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. Also delete some Blank Spaces in arch_timer_banner, according to the suggestion from checkpatch.pl. No functional change. Signed-off-by: Fu We

[PATCH v10 4/8] acpi/arm64: Add GTDT table parse driver

2016-07-26 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 v10 6/8] acpi/arm64: Add memory-mapped timer support in GTDT driver

2016-07-26 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 d

<    1   2   3   4   5   6   7   >