On 1/22/26 10:34 AM, Kancy Joe wrote: > > On 1/22/2026 5:25 PM, Neil Armstrong wrote: >> On 1/22/26 10:15, Konrad Dybcio wrote: >>> On 1/21/26 5:40 PM, Neil Armstrong wrote: >>>> From: KancyJoe <[email protected]> >>>> >>>> Add initial Device Tree for the Ayaneo Pocket S2 gaming console based >>>> on the Qualcomm Snapdragon 8 Gen 3 platform. >>>> >>>> The design is similar to a phone wihout the modem, the game control >>>> is handled via a standalone controller connected to a PCIe USB >>>> controller. >>>> >>>> Display support will be added in a second time. >>>> >>>> Signed-off-by: KancyJoe <[email protected]> >>>> Signed-off-by: Neil Armstrong <[email protected]> >>>> ---
[...] >>>> + pinctrl-names = "default", >>>> + "sleep"; >>>> + >>>> + pwms = <&pm8550_pwm 3 50000>; >>>> + >>>> + #cooling-cells = <2>; >>>> + cooling-levels = <0 16 32 45 60 80 105 130 155 180 205 230 255>; >>> >>> Does this come from a preexisting map? >> >> Kancy ? > > No it is not a preexisting map. I add it(and the thermal part) myself to get > dynamic fan speed control work. Perhaps you can also use userspace fan > control daemon instead of hardcode it here. In android the vendor control the > fan speed in userspace too. > > Following block is what the stock fw defined. I changed the granularity to > make fan speed (or noise actually) sounds more "smooth". > > ``` > > cooling-levels = <0 64 128 255>; > > ``` FWIW the corresponding pwm-backlight driver has this num-interpolated-steps property which computes a smooth map.. not sure how many cooling levels are resonable for a PWM fan, but then I would intuitively not object to having more as opposed to less.. Konrad
