| Message ID | 20190709080015.43442-1-Anson.Huang@nxp.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | viresh kumar |
| Headers | show |
| Series | [V2,1/4] dt-bindings: opp: Support multiple opp-suspend properties | expand |
On 09-07-19, 16:00, Anson.Huang@nxp.com wrote: > From: Anson Huang <Anson.Huang@nxp.com> > > Update opp-suspend property's description to support multiple > opp-suspend properties defined in DT, the OPP with highest opp-hz > and with opp-suspend property present will be used as suspend opp. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > --- > New patch. > --- > Documentation/devicetree/bindings/opp/opp.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/opp/opp.txt b/Documentation/devicetree/bindings/opp/opp.txt > index 76b6c79..6859227 100644 > --- a/Documentation/devicetree/bindings/opp/opp.txt > +++ b/Documentation/devicetree/bindings/opp/opp.txt > @@ -140,8 +140,8 @@ Optional properties: > frequency for a short duration of time limited by the device's power, current > and thermal limits. > > -- opp-suspend: Marks the OPP to be used during device suspend. Only one OPP in > - the table should have this. > +- opp-suspend: Marks the OPP to be used during device suspend. If multiple OPPs > + in the table have this, the OPP with highest opp-hz will be used. > > - opp-supported-hw: This enables us to select only a subset of OPPs from the > larger OPP table, based on what version of the hardware we are running on. We LGTM. Once Rob Acks it, I will apply the first two patches to the OPP tree.
On Tue, 9 Jul 2019 16:00:12 +0800, Anson.Huang@nxp.com wrote: > From: Anson Huang <Anson.Huang@nxp.com> > > Update opp-suspend property's description to support multiple > opp-suspend properties defined in DT, the OPP with highest opp-hz > and with opp-suspend property present will be used as suspend opp. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > --- > New patch. > --- > Documentation/devicetree/bindings/opp/opp.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Reviewed-by: Rob Herring <robh@kernel.org>
On 09-07-19, 16:00, Anson.Huang@nxp.com wrote: > From: Anson Huang <Anson.Huang@nxp.com> > > Update opp-suspend property's description to support multiple > opp-suspend properties defined in DT, the OPP with highest opp-hz > and with opp-suspend property present will be used as suspend opp. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > --- > New patch. > --- > Documentation/devicetree/bindings/opp/opp.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Applied. Thanks.
Hi, Shawn > On 09-07-19, 16:00, Anson.Huang@nxp.com wrote: > > From: Anson Huang <Anson.Huang@nxp.com> > > > > Update opp-suspend property's description to support multiple > > opp-suspend properties defined in DT, the OPP with highest opp-hz and > > with opp-suspend property present will be used as suspend opp. > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > --- > > New patch. > > --- > > Documentation/devicetree/bindings/opp/opp.txt | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > Applied. Thanks. Would you pick up the DT patches in this series? Thanks. Anson
diff --git a/Documentation/devicetree/bindings/opp/opp.txt b/Documentation/devicetree/bindings/opp/opp.txt index 76b6c79..6859227 100644 --- a/Documentation/devicetree/bindings/opp/opp.txt +++ b/Documentation/devicetree/bindings/opp/opp.txt @@ -140,8 +140,8 @@ Optional properties: frequency for a short duration of time limited by the device's power, current and thermal limits. -- opp-suspend: Marks the OPP to be used during device suspend. Only one OPP in - the table should have this. +- opp-suspend: Marks the OPP to be used during device suspend. If multiple OPPs + in the table have this, the OPP with highest opp-hz will be used. - opp-supported-hw: This enables us to select only a subset of OPPs from the larger OPP table, based on what version of the hardware we are running on. We