Skip to content

thermal: qcom: tsens: small fixup for debugfs

Christian Marangi linux-arm-msm linux-pm lkml 2 versions – Applied

Status from this message: The following commit has been merged into the thermal/next branch of thermal:

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest3 patches 4 repliesApplied
  2. v22 patches 0 repliesSuperseded

Changes in v3 from the cover letter

- remove extra space from patch 1
- split patch 2 to 2 different patch
- patch 3 rework wrong debugfs structure

Patches in v3 Applied

#Subject and filesSizeRepliesApplied
1/3thermal: qcom: tsens: init debugfs only with successful probedrivers/thermal/qcom/tsens.c+5 -30 replies
2/3thermal: qcom: tsens: fix wrong version id dbg_version_showdrivers/thermal/qcom/tsens.c+1 -10 replies
3/3thermal: qcom: tsens: rework debugfs file structuredrivers/thermal/qcom/tsens.c+3 -103 replies

Reviews trailers per patch

1 of 3 patches has a Reviewed-by or Acked-by.

PatchThara Gopinath
1/3A-b
2/3
3/3

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/3] thermal: qcom: tsens: small fixup for debugfs

This is a small series to fixup some bug in how tsens init debufs.

The first patch just handle situation where tsens fails to register and debugfs are getting registred anyway. When tsens is tried to reprobe debugs will print a warning as the directory are already there.

The second patch is a fixup for wrong version when the ancient VER_0 is used.

The third patch is a rework of debugfs structure moving version in the tsens istance instead of ignoring any other tsens istance if it will ever be the case in the future of having multiple tsens instance with different version. It's just futureproof on it's own and also removed one additional check.

changes v3:
- remove extra space from patch 1
- split patch 2 to 2 different patch
- patch 3 rework wrong debugfs structure
changes v2:
- Changed sob name to new one.

(the resend was actually v2 but i totally forgot that I sent it as v2 with the sob name fixed... but everything should be good now...)

Christian Marangi (3):
thermal: qcom: tsens: init debugfs only with successful probe
thermal: qcom: tsens: fix wrong version id dbg_version_show
thermal: qcom: tsens: rework debugfs file structure

 drivers/thermal/qcom/tsens.c | 23 +++++++++--------------
 1 file changed, 9 insertions(+), 14 deletions(-)
Signature · 1 line
2.37.2