On Thu, 8 Feb 2024 at 08:29, Taniya Das <quic_tdas@xxxxxxxxxxx> wrote: > > When remoteproc is used to boot the LPASS the ADSP PLL should not be > configured from the high level OS. Why? > Thus add support for property to > avoid configuring the LPASS PLL. > > Signed-off-by: Taniya Das <quic_tdas@xxxxxxxxxxx> > --- > .../devicetree/bindings/clock/qcom,sc7280-lpasscorecc.yaml | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/qcom,sc7280-lpasscorecc.yaml b/Documentation/devicetree/bindings/clock/qcom,sc7280-lpasscorecc.yaml > index deee5423d66e..358eb4a1cffd 100644 > --- a/Documentation/devicetree/bindings/clock/qcom,sc7280-lpasscorecc.yaml > +++ b/Documentation/devicetree/bindings/clock/qcom,sc7280-lpasscorecc.yaml > @@ -49,6 +49,11 @@ properties: > peripheral loader. > type: boolean > > + qcom,adsp-skip-pll: > + description: > + Indicates if the LPASS PLL configuration is to be skipped. > + type: boolean This property describes OS behaviour rather than the hardware. Such things are generally not acceptable. > + > required: > - compatible > - reg > -- > 2.17.1 > > -- With best wishes Dmitry