Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615: Add CPU scaling clock node

[Dmitry Baryshkov]

On 27/06/2025 06:52, Taniya Das wrote:
> On 6/25/2025 5:06 PM, Dmitry Baryshkov wrote:
> > On Wed, Jun 25, 2025 at 04:44:01PM +0530, Taniya Das wrote:
> > > Add cpufreq-hw node to support CPU frequency scaling.
> > >
> > > Signed-off-by: Taniya Das <quic_tdas@xxxxxxxxxxx>
> > > ---
> > >   arch/arm64/boot/dts/qcom/qcs615.dtsi | 29 +++++++++++++++++++++++++++++
> > >   1 file changed, 29 insertions(+)
> > >
> > > @@ -3891,6 +3907,19 @@ glink_edge: glink-edge {
> > >   				qcom,remote-pid = <2>;
> > >   			};
> > >   		};
> > > +
> > > +		cpufreq_hw: cpufreq@18323000 {
> > > +			compatible = "qcom,sc7180-cpufreq-hw", "qcom,cpufreq-hw";
> >
> > Why? Other platforms use a true SoC as the first entry.
> Dmitry, from cpufreq-hw perspective SC7180 is a exact match for QCS615
> and that was the reason to use the same.

Please look around. A quick `git grep` would show that every SoC uses 
SoC-specific compatible (although some of them are definitely 
compatible). The reason is pretty simple: each platform might have 
SoC-specific tunings and quirks.

--
With best wishes
Dmitry