Few more cleanups for tegra-timer
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v3 Latest8 patches 14 repliesIn review
- v26 patches 13 repliesSuperseded
- v13 patches 2 repliesSuperseded
Patches in v3 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | clocksource/drivers/tegra: Restore timer rate on Tegra210drivers/ | +2 | 1 reply | |
| 2/8 | clocksource/drivers/tegra: Remove duplicated use of per_cpu_ptrdrivers/ | +25 | 1 reply | |
| 3/8 | clocksource/drivers/tegra: Set and use timer's perioddrivers/ | +3 | 3 replies | |
| 4/8 | clocksource/drivers/tegra: Drop unneeded typecasting in one placedrivers/ | +1 | 1 reply | |
| 5/8 | clocksource/drivers/tegra: Add verbose definition for 1MHz constantdrivers/ | +7 | 1 reply | |
| 6/8 | clocksource/drivers/tegra: Restore base address before cleanupdrivers/ | +2 | 2 replies | |
| 7/8 | clocksource/drivers/tegra: Cycles can't be 0drivers/ | +10 | 3 replies | |
| 8/8 | clocksource/drivers/tegra: Set up maximum-ticks limit properlydrivers/ | +9 | 2 replies |
Reviews trailers per patch
All 8 patches have a Reviewed-by or Acked-by.
| Patch | Jon Hunter | Thierry Reding |
|---|---|---|
| 1/8 | A-b T-b | A-b |
| 2/8 | A-b | A-b |
| 3/8 | A-b | A-b |
| 4/8 | A-b | A-b |
| 5/8 | A-b | A-b |
| 6/8 | A-b | A-b |
| 7/8 | A-b | A-b |
| 8/8 | A-b | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v3 0/8] Few more cleanups for tegra-timer
Hello,
This a followup to [0] that includes some more fixes and further prettifies the driver's code.
[0] https://patchwork.ozlabs.org/project/linux-tegra/list/?series=111529
Changelog:
v3: Addressed request from Jon Hunter that was made in a review comment
to v2 by dropping the timer's period rounding-up in the "Set and use timer' period" patch.
Appended two new patches to this series that were already sent out and reviewed after the v2 of this series:
clocksource/drivers/tegra: Cycles can't be 0 clocksource/drivers/tegra: Set up maximum-ticks limit properly
In this two new patches I addressed review comments that were made by Thierry Reding by adding clarifying comments to the code and extending the commit messages a tad.
Corrected the "Fixes" tag in a "Restore timer rate on Tegra210" patch such that linux-next checker won't complain about the shortened commit's subject.
v2: Fixed a bug that was introduced by [0] in a newly added patch:
"Restore timer rate on Tegra210".
Fixed potential problem in regards to error handling in another new
patch: "Restore base address before cleanup".
Added new patch "Add verbose definition for 1MHz constant" as per Daniel's Lezcano recommendation.
Fixed a code typo that was made in "Remove duplicated use of per_cpu_ptr" of v1.
Dmitry Osipenko (8):
clocksource/drivers/tegra: Restore timer rate on Tegra210 clocksource/drivers/tegra: Remove duplicated use of per_cpu_ptr clocksource/drivers/tegra: Set and use timer's period clocksource/drivers/tegra: Drop unneeded typecasting in one place clocksource/drivers/tegra: Add verbose definition for 1MHz constant clocksource/drivers/tegra: Restore base address before cleanup clocksource/drivers/tegra: Cycles can't be 0 clocksource/drivers/tegra: Set up maximum-ticks limit properly
drivers/clocksource/timer-tegra.c | 82 +++++++++++++++++++++---------- 1 file changed, 56 insertions(+), 26 deletions(-)
Signature · 1 line
2.22.0