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

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 to get these into 4.9. > yes, v12 is

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

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 to get these into 4.9. > yes, v12 is

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

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 to get these into 4.9. -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm

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

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 to get these into 4.9. -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm

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

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,

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

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 have any

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

Hi Thomas, Daniel, For these arm_arch_timer patches, do you have any other suggestion or comment? I have deleted "skipping" in the error message. I have prepared v12 (rebase to rc6 and on the top of IORT v11), should I send it now (if you are OK with my arm_arch_timer patches ), or anything I

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

Hi Thomas, Daniel, For these arm_arch_timer patches, do you have any other suggestion or comment? I have deleted "skipping" in the error message. I have prepared v12 (rebase to rc6 and on the top of IORT v11), should I send it now (if you are OK with my arm_arch_timer patches ), or anything I

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

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) >> + pr_err("Failed to get platform timer info, skipping.\n"); > > So this prints something about skipping. But then it

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

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) >> + pr_err("Failed to get platform timer info, skipping.\n"); > > So this prints something about skipping. But then it continues as if >

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

On Tue, 6 Sep 2016, fu@linaro.org wrote: > + if (timer_count < 0) > + pr_err("Failed to get platform timer info, skipping.\n"); So this prints something about skipping. But then it continues as if nothing went wrong. That's either wrong or confusing or both. > -

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

On Tue, 6 Sep 2016, fu@linaro.org wrote: > + if (timer_count < 0) > + pr_err("Failed to get platform timer info, skipping.\n"); So this prints something about skipping. But then it continues as if nothing went wrong. That's either wrong or confusing or both. > -

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

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

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

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 ---