Skip to content

RFC qcom: Add support for tsens driver

Rajendra Nayak linux-arm-kernel linux-arm-msm linux-pm 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest0 of 7 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 7 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [RFC 0/7] qcom: Add support for tsens driver

Hi Kevin,

Quoted 1 line

Curious how are you testing this? with which thermal governor?

I have been mostly testing this with step wise on the db410c target board.

Quoted 10 lines

I'm experimenting with the new thermal power_allocator governor currently in linux-next, and was hoping to use this to test it, but I'm noticing it doesn't work with the power_allocator governor.

The first thing is that the power_allator goveror expects each thermal zone to have a sustainable-power property in the DT, which this series doesn't provide. Hacking some dummy values in there to see what happens next, I then hit the "thermal zone cpu-thermal0 has wrong trip setup for power allocator" error from the power_allocator driver.

Right, the RFC I posted wasn't tested with IPA and hence does not have some of the additional properties in DT which IPA expects. The wrong trip setup error that you are seeing is because IPA expects 2 passive trip points, a "switch on" trip point and a "desired temperature" trip point and the dts I posted with the RFC had just one passive trip point.

I am in the process of testing this with IPA myself (on db410c mainly, but will also test on db8074 and IFC6410/6540 as well) and will repost in a few days with updated dts entries and a few other minor fixes to the driver itself. Let me know what target board you are interested to get this working on and I will make sure I test on it before I repost.

regards,
Rajendra

Read the cover letter in its thread