Re: [PATCH 1/2] PM / OPP: compatible is an optional property

2016-09-25 Thread Viresh Kumar
On 23-09-16, 14:55, Rob Herring wrote: > On Fri, Sep 23, 2016 at 10:45:26AM +0530, Viresh Kumar wrote: > > On 22-09-16, 12:24, Stephen Boyd wrote: > > > On 09/21/2016 02:32 AM, Viresh Kumar wrote: > > > > It was never compulsory to have a compatible string in the OPP table. > > > > Fix the document

Re: [PATCH 1/2] PM / OPP: compatible is an optional property

2016-09-23 Thread Rob Herring
On Fri, Sep 23, 2016 at 10:45:26AM +0530, Viresh Kumar wrote: > On 22-09-16, 12:24, Stephen Boyd wrote: > > On 09/21/2016 02:32 AM, Viresh Kumar wrote: > > > It was never compulsory to have a compatible string in the OPP table. > > > Fix the documentation to mark it optional. > > > NAK. > > > Als

Re: [PATCH 1/2] PM / OPP: compatible is an optional property

2016-09-22 Thread Viresh Kumar
On 22-09-16, 12:24, Stephen Boyd wrote: > On 09/21/2016 02:32 AM, Viresh Kumar wrote: > > It was never compulsory to have a compatible string in the OPP table. > > Fix the documentation to mark it optional. > > > > Also update its description a bit. > > > > Signed-off-by: Viresh Kumar > > --- > >

Re: [PATCH 1/2] PM / OPP: compatible is an optional property

2016-09-22 Thread Stephen Boyd
On 09/21/2016 02:32 AM, Viresh Kumar wrote: > It was never compulsory to have a compatible string in the OPP table. > Fix the documentation to mark it optional. > > Also update its description a bit. > > Signed-off-by: Viresh Kumar > --- Why? I'd prefer the compatible string to be required so we

[PATCH 1/2] PM / OPP: compatible is an optional property

2016-09-21 Thread Viresh Kumar
It was never compulsory to have a compatible string in the OPP table. Fix the documentation to mark it optional. Also update its description a bit. Signed-off-by: Viresh Kumar --- Documentation/devicetree/bindings/opp/opp.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --g