Skip to content

arch_topology: Minor cleanups

Viresh Kumar linux-arm-kernel lkml 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest5 patches 1 replyIn review
  2. v25 patches 6 repliesSuperseded

Changes in v3 from the cover letter

- topology_parse_cpu_capacity() returns bool now (Juri).
- "Get rid of cap_parsing_done" is the last patch now and its implementation is change a bit (Juri).

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5arch_topology: Don't break lines unnecessarilydrivers/base/arch_topology.c+3 -60 replies
2/5arch_topology: Convert switch block to if blockdrivers/base/arch_topology.c+22 -190 replies
3/5arch_topology: Change return type of topology_parse_cpu_capacity() to booldrivers/base/arch_topology.c include/linux/arch_topology.h+7 -50 replies
4/5arch_topology: Localize cap_parsing_failed to topology_parse_cpu_capacity()drivers/base/arch_topology.c+13 -110 replies
5/5arch_topology: Get rid of cap_parsing_donedrivers/base/arch_topology.c+1 -30 replies

Reviews trailers per patch

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

PatchJuri Lelli
SeriesReviewed-and-tested-by
1/5
2/5
3/5
4/5
5/5

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

Read the v3 thread

Cover letter [PATCH V3 0/5] arch_topology: Minor cleanups

Hi Greg,

You weren't included in the first [1] version of this series, as it was targeting arch/arm*/ directories then.

Here are some cleanups for the arch_topology core.

Tested on ARM64 Hikey board by setting following in cpu nodes in DT:

        capacity-dmips-mhz = <1000>;

V2->V3:
- topology_parse_cpu_capacity() returns bool now (Juri).
- "Get rid of cap_parsing_done" is the last patch now and its implementation is change a bit (Juri).

V1->V2:
- based over linux-next/master (to get Juri's recent changes)
- More cleanups included. V1 only had the first patch.
- Rename of cap_parsing_failed isn't required anymore (as it is localized to a single routine now).

--
viresh

[1] https://marc.info/?l=linux-arm-kernel&m=149795553024005

Viresh Kumar (5):
arch_topology: Don't break lines unnecessarily
arch_topology: Convert switch block to if block
arch_topology: Change return type of topology_parse_cpu_capacity() to
bool
arch_topology: Localize cap_parsing_failed to
topology_parse_cpu_capacity()
arch_topology: Get rid of cap_parsing_done

 drivers/base/arch_topology.c  | 78 +++++++++++++++++++++----------------------
 include/linux/arch_topology.h |  4 ++-
 2 files changed, 42 insertions(+), 40 deletions(-)
Signature · 1 line
2.13.0.71.gd7076ec9c9cb