Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Fu Wei
Hi Mark, On 1 February 2017 at 02:49, Mark Rutland wrote: > On Wed, Feb 01, 2017 at 02:43:02AM +0800, Fu Wei wrote: >> On 31 January 2017 at 01:49, Mark Rutland wrote: >> > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: >> >> On 26 January

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Fu Wei
Hi Mark, On 1 February 2017 at 02:49, Mark Rutland wrote: > On Wed, Feb 01, 2017 at 02:43:02AM +0800, Fu Wei wrote: >> On 31 January 2017 at 01:49, Mark Rutland wrote: >> > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: >> >> On 26 January 2017 at 01:25, Mark Rutland wrote: >> >> > On

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Mark Rutland
On Wed, Feb 01, 2017 at 02:43:02AM +0800, Fu Wei wrote: > On 31 January 2017 at 01:49, Mark Rutland wrote: > > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: > >> On 26 January 2017 at 01:25, Mark Rutland wrote: > >> > On Wed, Jan 25, 2017 at

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Mark Rutland
On Wed, Feb 01, 2017 at 02:43:02AM +0800, Fu Wei wrote: > On 31 January 2017 at 01:49, Mark Rutland wrote: > > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: > >> On 26 January 2017 at 01:25, Mark Rutland wrote: > >> > On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: > >> >> On

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Fu Wei
Hi Mark, On 31 January 2017 at 01:49, Mark Rutland wrote: > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: >> On 26 January 2017 at 01:25, Mark Rutland wrote: >> > On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: >> >> On 25 January

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Fu Wei
Hi Mark, On 31 January 2017 at 01:49, Mark Rutland wrote: > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: >> On 26 January 2017 at 01:25, Mark Rutland wrote: >> > On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: >> >> On 25 January 2017 at 01:24, Mark Rutland wrote: >> >> > On

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Mark Rutland
On Mon, Jan 30, 2017 at 05:49:59PM +, Mark Rutland wrote: > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: > > And because Linux kernel is running on Non-secure EL1, so should we > > skip "SECURE" timer in Linux? > > I guess you mean by checking the GTx Common flags, to see if the

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-31 Thread Mark Rutland
On Mon, Jan 30, 2017 at 05:49:59PM +, Mark Rutland wrote: > On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: > > And because Linux kernel is running on Non-secure EL1, so should we > > skip "SECURE" timer in Linux? > > I guess you mean by checking the GTx Common flags, to see if the

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-30 Thread Mark Rutland
On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: > On 26 January 2017 at 01:25, Mark Rutland wrote: > > On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: > >> On 25 January 2017 at 01:24, Mark Rutland wrote: > >> > On Wed, Jan 18, 2017 at

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-30 Thread Mark Rutland
On Thu, Jan 26, 2017 at 01:49:03PM +0800, Fu Wei wrote: > On 26 January 2017 at 01:25, Mark Rutland wrote: > > On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: > >> On 25 January 2017 at 01:24, Mark Rutland wrote: > >> > On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Fu Wei
Hi Mark, Christopher, On 26 January 2017 at 01:36, Mark Rutland wrote: > On Wed, Jan 25, 2017 at 10:38:01AM -0500, Christopher Covington wrote: >> On 01/25/2017 01:46 AM, Fu Wei wrote: >> > On 25 January 2017 at 01:24, Mark Rutland wrote: >> >> On

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Fu Wei
Hi Mark, Christopher, On 26 January 2017 at 01:36, Mark Rutland wrote: > On Wed, Jan 25, 2017 at 10:38:01AM -0500, Christopher Covington wrote: >> On 01/25/2017 01:46 AM, Fu Wei wrote: >> > On 25 January 2017 at 01:24, Mark Rutland wrote: >> >> On Wed, Jan 18, 2017 at 09:25:32PM +0800,

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Fu Wei
Hi Mark, On 26 January 2017 at 01:25, Mark Rutland wrote: > On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: >> Hi Mark, > > Hi, > >> On 25 January 2017 at 01:24, Mark Rutland wrote: >> > On Wed, Jan 18, 2017 at 09:25:32PM +0800,

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Fu Wei
Hi Mark, On 26 January 2017 at 01:25, Mark Rutland wrote: > On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: >> Hi Mark, > > Hi, > >> On 25 January 2017 at 01:24, Mark Rutland wrote: >> > On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >> >> From: Fu Wei >> >> >> >>

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Mark Rutland
On Wed, Jan 25, 2017 at 10:38:01AM -0500, Christopher Covington wrote: > On 01/25/2017 01:46 AM, Fu Wei wrote: > > On 25 January 2017 at 01:24, Mark Rutland wrote: > >> On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: > >>> From: Fu Wei

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Mark Rutland
On Wed, Jan 25, 2017 at 10:38:01AM -0500, Christopher Covington wrote: > On 01/25/2017 01:46 AM, Fu Wei wrote: > > On 25 January 2017 at 01:24, Mark Rutland wrote: > >> On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: > >>> From: Fu Wei > > And for CNTFRQ(in CNTCTLBase and

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Mark Rutland
On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: > Hi Mark, Hi, > On 25 January 2017 at 01:24, Mark Rutland wrote: > > On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: > >> From: Fu Wei > >> > >> The counter frequency detection

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Mark Rutland
On Wed, Jan 25, 2017 at 02:46:12PM +0800, Fu Wei wrote: > Hi Mark, Hi, > On 25 January 2017 at 01:24, Mark Rutland wrote: > > On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: > >> From: Fu Wei > >> > >> The counter frequency detection call(arch_timer_detect_rate) combines two

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Christopher Covington
Hi Fu, On 01/25/2017 01:46 AM, Fu Wei wrote: > Hi Mark, > > On 25 January 2017 at 01:24, Mark Rutland wrote: >> On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >>> From: Fu Wei >>> >>> The counter frequency detection

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-25 Thread Christopher Covington
Hi Fu, On 01/25/2017 01:46 AM, Fu Wei wrote: > Hi Mark, > > On 25 January 2017 at 01:24, Mark Rutland wrote: >> On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >>> From: Fu Wei >>> >>> The counter frequency detection call(arch_timer_detect_rate) combines two >>> ways to get

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-24 Thread Fu Wei
Hi Mark, On 25 January 2017 at 14:46, Fu Wei wrote: > Hi Mark, > > On 25 January 2017 at 01:24, Mark Rutland wrote: >> On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >>> From: Fu Wei >>> >>> The counter frequency

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-24 Thread Fu Wei
Hi Mark, On 25 January 2017 at 14:46, Fu Wei wrote: > Hi Mark, > > On 25 January 2017 at 01:24, Mark Rutland wrote: >> On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >>> From: Fu Wei >>> >>> The counter frequency detection call(arch_timer_detect_rate) combines two >>> ways

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-24 Thread Fu Wei
Hi Mark, On 25 January 2017 at 01:24, Mark Rutland wrote: > On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The counter frequency detection call(arch_timer_detect_rate) combines two >> ways to get counter

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-24 Thread Fu Wei
Hi Mark, On 25 January 2017 at 01:24, Mark Rutland wrote: > On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: >> From: Fu Wei >> >> The counter frequency detection call(arch_timer_detect_rate) combines two >> ways to get counter frequency: system coprocessor register and MMIO

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-24 Thread Mark Rutland
On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: > From: Fu Wei > > The counter frequency detection call(arch_timer_detect_rate) combines two > ways to get counter frequency: system coprocessor register and MMIO timer. > But in a specific timer init code, we

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-24 Thread Mark Rutland
On Wed, Jan 18, 2017 at 09:25:32PM +0800, fu@linaro.org wrote: > From: Fu Wei > > The counter frequency detection call(arch_timer_detect_rate) combines two > ways to get counter frequency: system coprocessor register and MMIO timer. > But in a specific timer init code, we only need one way

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-19 Thread Hanjun Guo
On 2017/1/19 17:44, Fu Wei wrote: Hi Hanjun, On 19 January 2017 at 16:02, Hanjun Guo wrote: Hi Fuwei, One comments below. On 2017/1/18 21:25, fu@linaro.org wrote: From: Fu Wei The counter frequency detection call(arch_timer_detect_rate)

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-19 Thread Hanjun Guo
On 2017/1/19 17:44, Fu Wei wrote: Hi Hanjun, On 19 January 2017 at 16:02, Hanjun Guo wrote: Hi Fuwei, One comments below. On 2017/1/18 21:25, fu@linaro.org wrote: From: Fu Wei The counter frequency detection call(arch_timer_detect_rate) combines two ways to get counter frequency:

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-19 Thread Fu Wei
Hi Hanjun, On 19 January 2017 at 16:02, Hanjun Guo wrote: > Hi Fuwei, > > One comments below. > > > On 2017/1/18 21:25, fu@linaro.org wrote: >> >> From: Fu Wei >> >> The counter frequency detection call(arch_timer_detect_rate) combines two >> ways

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-19 Thread Fu Wei
Hi Hanjun, On 19 January 2017 at 16:02, Hanjun Guo wrote: > Hi Fuwei, > > One comments below. > > > On 2017/1/18 21:25, fu@linaro.org wrote: >> >> From: Fu Wei >> >> The counter frequency detection call(arch_timer_detect_rate) combines two >> ways to get counter frequency: system

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-19 Thread Hanjun Guo
Hi Fuwei, One comments below. On 2017/1/18 21:25, fu@linaro.org wrote: From: Fu Wei The counter frequency detection call(arch_timer_detect_rate) combines two ways to get counter frequency: system coprocessor register and MMIO timer. But in a specific timer init code,

Re: [PATCH v20 08/17] clocksource/drivers/arm_arch_timer: Rework counter frequency detection.

2017-01-19 Thread Hanjun Guo
Hi Fuwei, One comments below. On 2017/1/18 21:25, fu@linaro.org wrote: From: Fu Wei The counter frequency detection call(arch_timer_detect_rate) combines two ways to get counter frequency: system coprocessor register and MMIO timer. But in a specific timer init code, we only need one way