| Message ID | 20230720054100.9940-3-manivannan.sadhasivam@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | viresh kumar |
| Headers | show |
| Series | UFS: Add OPP and interconnect support | expand |
On Thu, 20 Jul 2023 11:10:47 +0530, Manivannan Sadhasivam wrote: > Current limit of 16 will be exhausted by platforms specifying the frequency > for 9 clocks using opp-hz, like Qcom SDM845 SoC. For instance, specifying > the frequency for 9 clocks with 64bit specifier as below would consume > (9 * 2 = 18) items. > > opp-50000000 { > opp-hz = /bits/ 64 <50000000>, > /bits/ 64 <0>, > /bits/ 64 <0>, > /bits/ 64 <37500000>, > /bits/ 64 <0>, > /bits/ 64 <0>, > /bits/ 64 <0>, > /bits/ 64 <0>, > /bits/ 64 <75000000>; > }; > > So let's increase the limit to 32 which should be enough for most platforms > (hopefully). > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > --- > Documentation/devicetree/bindings/opp/opp-v2-base.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Acked-by: Rob Herring <robh@kernel.org>
On 24-07-23, 09:51, Rob Herring wrote: > > On Thu, 20 Jul 2023 11:10:47 +0530, Manivannan Sadhasivam wrote: > > Current limit of 16 will be exhausted by platforms specifying the frequency > > for 9 clocks using opp-hz, like Qcom SDM845 SoC. For instance, specifying > > the frequency for 9 clocks with 64bit specifier as below would consume > > (9 * 2 = 18) items. > > > > opp-50000000 { > > opp-hz = /bits/ 64 <50000000>, > > /bits/ 64 <0>, > > /bits/ 64 <0>, > > /bits/ 64 <37500000>, > > /bits/ 64 <0>, > > /bits/ 64 <0>, > > /bits/ 64 <0>, > > /bits/ 64 <0>, > > /bits/ 64 <75000000>; > > }; > > > > So let's increase the limit to 32 which should be enough for most platforms > > (hopefully). > > > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > > --- > > Documentation/devicetree/bindings/opp/opp-v2-base.yaml | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Acked-by: Rob Herring <robh@kernel.org> Applied. Thanks.
diff --git a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml index 47e6f36b7637..e2f8f7af3cf4 100644 --- a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml +++ b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml @@ -56,7 +56,7 @@ patternProperties: need to be configured and that is left for the implementation specific binding. minItems: 1 - maxItems: 16 + maxItems: 32 items: maxItems: 1
Current limit of 16 will be exhausted by platforms specifying the frequency for 9 clocks using opp-hz, like Qcom SDM845 SoC. For instance, specifying the frequency for 9 clocks with 64bit specifier as below would consume (9 * 2 = 18) items. opp-50000000 { opp-hz = /bits/ 64 <50000000>, /bits/ 64 <0>, /bits/ 64 <0>, /bits/ 64 <37500000>, /bits/ 64 <0>, /bits/ 64 <0>, /bits/ 64 <0>, /bits/ 64 <0>, /bits/ 64 <75000000>; }; So let's increase the limit to 32 which should be enough for most platforms (hopefully). Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> --- Documentation/devicetree/bindings/opp/opp-v2-base.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)