[v2,0/6] Proper arch timer support for Exynos5433-based TM2(e) boards

Message ID 20181015123112.9379-1-m.szyprowski@samsung.com
Headers show
Series
  • Proper arch timer support for Exynos5433-based TM2(e) boards
Related show

Message

Marek Szyprowski Oct. 15, 2018, 12:31 p.m.
Dear All,

This patchset is an attempt to submit the last piece of missing code
to have proper support for Exynos5433 SoCs based TM2(e) boards. It
performs a cleanup of timer configuration, which so far needed various
out-of-tree workarounds. The fixes provided by this patchset are also
needed for add proper support for system suspend/resume.

Best regards
Marek Szyprowski
Samsung R&D Institute Poland


Changelog:

v2:
- dropped arch timer patch, it will be discussed separately
- fixed issues pointed by Krzysztof Kozlowski

v1: https://patchwork.kernel.org/project/linux-samsung-soc/list/?series=27965&state=*&archive=both
- initial version


Patch summary:

Marek Szyprowski (6):
  clocksource: exynos_mct: Remove dead code
  clocksource: exynos_mct: Fix error path in timer resources
    initialization
  clocksource: exynos_mct: Add arch_timer cooperation mode for ARM64
  clocksource: Change CPU hotplug priority of exynos_mct driver
  arm64: dts: exynos: Move arch-timer node to right place
  arm64: platform: Add enable Exynos Multi-Core Timer driver

 arch/arm64/Kconfig.platforms               |  1 +
 arch/arm64/boot/dts/exynos/exynos5433.dtsi | 23 ++++----
 drivers/clocksource/exynos_mct.c           | 68 +++++++++++++++-------
 include/linux/cpuhotplug.h                 |  2 +-
 4 files changed, 61 insertions(+), 33 deletions(-)

-- 
2.17.1

Comments

Krzysztof Kozlowski Oct. 16, 2018, 1:06 p.m. | #1
On Mon, 15 Oct 2018 at 14:31, Marek Szyprowski <m.szyprowski@samsung.com> wrote:
>

> On Exynos SoCs enabling MCT driver is required even if ARM Architected

> Timer driver is used to for managing timer hardware and clock source

> events.

>

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

> ---

>  arch/arm64/Kconfig.platforms | 1 +

>  1 file changed, 1 insertion(+)


Looks okay, I'll take it after this merge window.

In case this was applied directly to arm-soc:
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>


Best regards,
Krzysztof