Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-24 Thread Yingjoe Chen
On Mon, 2015-08-24 at 09:51 +0200, Daniel Lezcano wrote: > On 08/21/2015 04:39 PM, Yingjoe Chen wrote: > > [ ... ] > > >>- Does the spurious interrupt occurs *every* time ? at each boot ? > > > > Yes. If you applied this series to enable mtk timer without this fix on > > mt8173 or mt8135 you

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-24 Thread Daniel Lezcano
On 08/21/2015 04:39 PM, Yingjoe Chen wrote: [ ... ] - Does the spurious interrupt occurs *every* time ? at each boot ? Yes. If you applied this series to enable mtk timer without this fix on mt8173 or mt8135 you can reproduce it. It occurs for every boot. It crash before uart driver is

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-24 Thread Yingjoe Chen
On Mon, 2015-08-24 at 09:51 +0200, Daniel Lezcano wrote: On 08/21/2015 04:39 PM, Yingjoe Chen wrote: [ ... ] - Does the spurious interrupt occurs *every* time ? at each boot ? Yes. If you applied this series to enable mtk timer without this fix on mt8173 or mt8135 you can

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-24 Thread Daniel Lezcano
On 08/21/2015 04:39 PM, Yingjoe Chen wrote: [ ... ] - Does the spurious interrupt occurs *every* time ? at each boot ? Yes. If you applied this series to enable mtk timer without this fix on mt8173 or mt8135 you can reproduce it. It occurs for every boot. It crash before uart driver is

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-21 Thread Yingjoe Chen
On Thu, 2015-08-20 at 16:28 +0200, Daniel Lezcano wrote: > On 08/17/2015 04:10 PM, Yingjoe Chen wrote: > > On Thu, 2015-08-13 at 10:35 +0200, Daniel Lezcano wrote: > >> On 07/22/2015 10:14 AM, Yingjoe Chen wrote: > >>> Spurious mtk timer interrupt is noticed at boot and cause kernel > >>> crash.

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-21 Thread Yingjoe Chen
On Thu, 2015-08-20 at 16:28 +0200, Daniel Lezcano wrote: On 08/17/2015 04:10 PM, Yingjoe Chen wrote: On Thu, 2015-08-13 at 10:35 +0200, Daniel Lezcano wrote: On 07/22/2015 10:14 AM, Yingjoe Chen wrote: Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-20 Thread Daniel Lezcano
On 08/17/2015 04:10 PM, Yingjoe Chen wrote: On Thu, 2015-08-13 at 10:35 +0200, Daniel Lezcano wrote: On 07/22/2015 10:14 AM, Yingjoe Chen wrote: Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-20 Thread Daniel Lezcano
On 08/17/2015 04:10 PM, Yingjoe Chen wrote: On Thu, 2015-08-13 at 10:35 +0200, Daniel Lezcano wrote: On 07/22/2015 10:14 AM, Yingjoe Chen wrote: Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-17 Thread Yingjoe Chen
On Thu, 2015-08-13 at 10:35 +0200, Daniel Lezcano wrote: > On 07/22/2015 10:14 AM, Yingjoe Chen wrote: > > Spurious mtk timer interrupt is noticed at boot and cause kernel > > crash. It seems if GPT is enabled, it will latch irq status even > > when its IRQ is disabled. When irq is enabled

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-17 Thread Yingjoe Chen
On Thu, 2015-08-13 at 10:35 +0200, Daniel Lezcano wrote: On 07/22/2015 10:14 AM, Yingjoe Chen wrote: Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is disabled. When irq is enabled afterward, we

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-13 Thread Daniel Lezcano
On 07/22/2015 10:14 AM, Yingjoe Chen wrote: Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is disabled. When irq is enabled afterward, we see spurious interrupt. Change init flow to only enable

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-08-13 Thread Daniel Lezcano
On 07/22/2015 10:14 AM, Yingjoe Chen wrote: Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is disabled. When irq is enabled afterward, we see spurious interrupt. Change init flow to only enable

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-07-30 Thread Yingjoe Chen
On Wed, 2015-07-22 at 16:14 +0800, Yingjoe Chen wrote: > Spurious mtk timer interrupt is noticed at boot and cause kernel > crash. It seems if GPT is enabled, it will latch irq status even > when its IRQ is disabled. When irq is enabled afterward, we see > spurious interrupt. > Change init flow to

Re: [PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-07-30 Thread Yingjoe Chen
On Wed, 2015-07-22 at 16:14 +0800, Yingjoe Chen wrote: Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is disabled. When irq is enabled afterward, we see spurious interrupt. Change init flow to only

[PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-07-22 Thread Yingjoe Chen
Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is disabled. When irq is enabled afterward, we see spurious interrupt. Change init flow to only enable GPT_CLK_SRC at mtk_timer_init. Acked-by: Matthias

[PATCH v2 1/5] clocksource: mediatek: do not enable GPT_CLK_EVT when setup

2015-07-22 Thread Yingjoe Chen
Spurious mtk timer interrupt is noticed at boot and cause kernel crash. It seems if GPT is enabled, it will latch irq status even when its IRQ is disabled. When irq is enabled afterward, we see spurious interrupt. Change init flow to only enable GPT_CLK_SRC at mtk_timer_init. Acked-by: Matthias