Re: [PATCH v2 1/2] arm64: dts: qcom: ipq5210: Add nodes for multiple peripherals

[Konrad Dybcio]

On 7/15/26 9:11 AM, Varadarajan Narayanan wrote:
> Describe common IPQ5210 SoC peripherals in the base DTSI so board DTS files
> can enable the hardware they use.
> 
> Add nodes for USB, PCIe, QPIC NAND/BAM, QFPROM, TSENS and thermal zones.
> Also describe the CPU OPP table and CPU cooling cells needed for CPU
> frequency scaling and thermal throttling.
> 
> Signed-off-by: Varadarajan Narayanan <varadarajan.narayanan@xxxxxxxxxxxxxxxx>
> ---

[...]

> +			phys = <&qusb_phy_0>,
> +			       <&ssphy_0>;
> +			phy-names = "usb2-phy",
> +				    "usb3-phy";
> +
> +			resets = <&gcc GCC_USB_BCR>;
> +
> +			tx-fifo-resize;
> +			snps,is-utmi-l1-suspend;
> +			snps,hird-threshold = /bits/ 8 <0x0>;
> +			snps,dis_u2_susphy_quirk;
> +			snps,dis_u3_susphy_quirk;
> +
> +			dr_mode = "host";

Is it host-only at the platform level?

[...]

> +	thermal_zones: thermal-zones {
> +		ethphy-thermal {
> +			polling-delay-passive = <0>;
> +			polling-delay = <0>;

Drop delays = 0, it's the default value

[...]

> +			cooling-maps {
> +				map0 {
> +					trip = <&cpu0_thermal_alert>;
> +					/* Set maximum frequency as 816 MHz */
> +					cooling-device = <&cpu0 3 3>,
> +							 <&cpu1 3 3>,
> +							 <&cpu2 3 3>,
> +							 <&cpu3 3 3>;

I think THERMAL_NO_LIMIT is generally better.

Konrad