Skip to content

NVIDIA Tegra clocksource driver improvements

Dmitry Osipenko linux-tegra lkml 4 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest10 patches 2 repliesIn review
  2. v38 patches 6 repliesSuperseded
  3. v27 patches 0 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v17 patches0 repliesSuperseded

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10clocksource/drivers/tegra: Support per-CPU timers on all Tegra'sdrivers/clocksource/timer-tegra20.c+43 -770 replies
2/10clocksource/drivers/tegra: Unify timer codedrivers/clocksource/timer-tegra20.c+60 -510 replies
3/10clocksource/drivers/tegra: Reset hardware state on initdrivers/clocksource/timer-tegra20.c+3 -00 replies
4/10clocksource/drivers/tegra: Replace readl/writel with relaxed versionsdrivers/clocksource/timer-tegra20.c+18 -170 replies
5/10clocksource/drivers/tegra: Release all IRQ's on request_irq() errordrivers/clocksource/timer-tegra20.c+3 -20 replies
6/10clocksource/drivers/tegra: Minor code clean updrivers/clocksource/timer-tegra20.c+24 -190 replies
7/10clocksource/drivers/tegra: Use SPDX identifierdrivers/clocksource/timer-tegra20.c+2 -130 replies
8/10clocksource/drivers/tegra: Support COMPILE_TEST universallydrivers/clocksource/Kconfig+1 -10 replies
9/10clocksource/drivers/tegra: Lower clocksource rating for some Tegra'sdrivers/clocksource/timer-tegra20.c+26 -40 replies
10/10clocksource/drivers/tegra: Rename timer-tegra20.c to timer-tegra.cdrivers/clocksource/Makefile drivers/clocksource/timer-tegra.c+1 -10 replies

Reviews trailers per patch

None of the 10 patches has a Reviewed-by or Acked-by.

PatchPeter De Schrijver
1/10
2/10
3/10
4/10
5/10
6/10
7/10
8/10
9/10Pointed-out-by
10/10

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v4 thread

Cover letter [PATCH v4 00/10] NVIDIA Tegra clocksource driver improvements

Hello,

This series primarily unifies the driver code across all Tegra SoC generations. In a result the clocksources are allocated per-CPU on older Tegra's and have a higher rating than the arch-timer where appropriate, the newer Tegra210 is getting support for microsecond clocksource and the driver's code is getting much cleaner.

The series was extensively tested on Tegra20 and Tegra30.

Changelog:

v4: In the comment to v3 Peter De Schrijver pointed out that arch-timer
isn't affected by DVFS changes and thus it is preferred over tegra-timer on [T114, T210). Added new patch to address that: "Lower clocksource rating for some Tegra's".

Daniel Lezcano suggested that it will be worthwhile to rename driver's source file as driver now covers more SoC generations than it initially did. Hence the new "Rename timer-tegra20.c to timer-tegra.c" patch.

v3: Fixed compilation on ARM64. Turned out that it doesn't have the
delay-timer, thanks to Nicolas Chauvet for the report.

Added new "Support COMPILE_TEST universally" patch for better compile-test coverage.

v2: Rebased on recent linux-next. Now all of #ifdef's are removed from the
code due to the recent patch that generalized persistent clocksource.

Couple other minor cosmetic changes.

Dmitry Osipenko (10):
clocksource/drivers/tegra: Support per-CPU timers on all Tegra's clocksource/drivers/tegra: Unify timer code clocksource/drivers/tegra: Reset hardware state on init clocksource/drivers/tegra: Replace readl/writel with relaxed versions clocksource/drivers/tegra: Release all IRQ's on request_irq() error clocksource/drivers/tegra: Minor code clean up clocksource/drivers/tegra: Use SPDX identifier clocksource/drivers/tegra: Support COMPILE_TEST universally clocksource/drivers/tegra: Lower clocksource rating for some Tegra's clocksource/drivers/tegra: Rename timer-tegra20.c to timer-tegra.c

 drivers/clocksource/Kconfig                   |   2 +-
 drivers/clocksource/Makefile                  |   2 +-
 .../{timer-tegra20.c => timer-tegra.c}        | 296 +++++++++---------
 3 files changed, 148 insertions(+), 152 deletions(-)
 rename drivers/clocksource/{timer-tegra20.c => timer-tegra.c} (54%)
Signature · 1 line
2.21.0