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

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

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. > > >

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. > > >

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 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

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