Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 09:42:39PM +0800, Fu Wei wrote: [...] > >> For calling acpi_gtdt_init() twice: > >> (1) 1st time: in early boot(bootmem), for init arch_timer and > >> memory-mapped timer, we initialize the acpi_gtdt_desc. > >> you can see that all the items in this struct are pointer. >

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 09:42:39PM +0800, Fu Wei wrote: [...] > >> For calling acpi_gtdt_init() twice: > >> (1) 1st time: in early boot(bootmem), for init arch_timer and > >> memory-mapped timer, we initialize the acpi_gtdt_desc. > >> you can see that all the items in this struct are pointer. >

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 10:31:42PM +0800, Fu Wei wrote: > On 29 March 2017 at 22:29, Fu Wei wrote: > > Hi Lorenzo, > > > > On 28 March 2017 at 19:35, Lorenzo Pieralisi > > wrote: > >> On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 10:31:42PM +0800, Fu Wei wrote: > On 29 March 2017 at 22:29, Fu Wei wrote: > > Hi Lorenzo, > > > > On 28 March 2017 at 19:35, Lorenzo Pieralisi > > wrote: > >> On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org wrote: > >>> From: Fu Wei > >>> > >>> This patch

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On 29 March 2017 at 22:29, Fu Wei wrote: > Hi Lorenzo, > > On 28 March 2017 at 19:35, Lorenzo Pieralisi > wrote: >> On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org wrote: >>> From: Fu Wei >>> >>> This patch adds

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On 29 March 2017 at 22:29, Fu Wei wrote: > Hi Lorenzo, > > On 28 March 2017 at 19:35, Lorenzo Pieralisi > wrote: >> On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org wrote: >>> From: Fu Wei >>> >>> This patch adds support for parsing arch timer info in GTDT, >>> provides some kernel

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, On 28 March 2017 at 19:35, Lorenzo Pieralisi wrote: > On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> This patch adds support for parsing arch timer info in GTDT, >> provides some kernel APIs to

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, On 28 March 2017 at 19:35, Lorenzo Pieralisi wrote: > On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org wrote: >> 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

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, On 29 March 2017 at 19:33, Lorenzo Pieralisi wrote: > On Wed, Mar 29, 2017 at 06:48:26PM +0800, Fu Wei wrote: >> Hi Lorenzo, >> >> On 29 March 2017 at 18:21, Lorenzo Pieralisi >> wrote: >> > On Wed, Mar 29, 2017 at 05:48:17PM

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, On 29 March 2017 at 19:33, Lorenzo Pieralisi wrote: > On Wed, Mar 29, 2017 at 06:48:26PM +0800, Fu Wei wrote: >> Hi Lorenzo, >> >> On 29 March 2017 at 18:21, Lorenzo Pieralisi >> wrote: >> > On Wed, Mar 29, 2017 at 05:48:17PM +0800, Fu Wei wrote: >> > >> > [...] >> > >> >> *

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 06:48:26PM +0800, Fu Wei wrote: > Hi Lorenzo, > > On 29 March 2017 at 18:21, Lorenzo Pieralisi > wrote: > > On Wed, Mar 29, 2017 at 05:48:17PM +0800, Fu Wei wrote: > > > > [...] > > > >> * @platform_timer_count: It points to a integer variable

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 06:48:26PM +0800, Fu Wei wrote: > Hi Lorenzo, > > On 29 March 2017 at 18:21, Lorenzo Pieralisi > wrote: > > On Wed, Mar 29, 2017 at 05:48:17PM +0800, Fu Wei wrote: > > > > [...] > > > >> * @platform_timer_count: It points to a integer variable which is used > >> *

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, On 29 March 2017 at 18:21, Lorenzo Pieralisi wrote: > On Wed, Mar 29, 2017 at 05:48:17PM +0800, Fu Wei wrote: > > [...] > >> * @platform_timer_count: It points to a integer variable which is used >> * for storing the number of

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, On 29 March 2017 at 18:21, Lorenzo Pieralisi wrote: > On Wed, Mar 29, 2017 at 05:48:17PM +0800, Fu Wei wrote: > > [...] > >> * @platform_timer_count: It points to a integer variable which is used >> * for storing the number of platform timers. >> *

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 05:48:17PM +0800, Fu Wei wrote: [...] > * @platform_timer_count: It points to a integer variable which is used > * for storing the number of platform timers. > * This pointer could be NULL, if the caller > *

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 29, 2017 at 05:48:17PM +0800, Fu Wei wrote: [...] > * @platform_timer_count: It points to a integer variable which is used > * for storing the number of platform timers. > * This pointer could be NULL, if the caller > *

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, Great thanks for your review and help, I will take most of your suggestions, But one or two comments have been discussed in previous patchset, please allow me to explain these. :-) On 28 March 2017 at 19:35, Lorenzo Pieralisi wrote: > On Wed, Mar 22, 2017

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

Hi Lorenzo, Great thanks for your review and help, I will take most of your suggestions, But one or two comments have been discussed in previous patchset, please allow me to explain these. :-) On 28 March 2017 at 19:35, Lorenzo Pieralisi wrote: > On Wed, Mar 22, 2017 at 12:31:18AM +0800,

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org wrote: > 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

Re: [PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

On Wed, Mar 22, 2017 at 12:31:18AM +0800, fu@linaro.org wrote: > 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

[PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

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.

[PATCH v22 07/11] acpi/arm64: Add GTDT table parse driver

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