Re: [PATCH V2 1/6] PM / QOS: Add default case to the switch

2017-02-12 Thread Viresh Kumar
On 10-02-17, 13:15, Pavel Machek wrote: > As soon as new QoS request type is added, this switch can be > fixed. There is no issue now, and there should be no issue in future. Sure. Will do it in V3. -- viresh

Re: [PATCH V2 1/6] PM / QOS: Add default case to the switch

2017-02-10 Thread Pavel Machek
On Fri 2017-02-10 11:30:30, Viresh Kumar wrote: > On 09-02-17, 15:24, Pavel Machek wrote: > > On Thu 2017-02-09 09:11:47, Viresh Kumar wrote: > > > The switch block handles all the QOS request types present today, but > > > starts giving compilation warnings as soon as a new type is added and > > >

Re: [PATCH V2 1/6] PM / QOS: Add default case to the switch

2017-02-10 Thread Pavel Machek
On Fri 2017-02-10 11:30:30, Viresh Kumar wrote: > On 09-02-17, 15:24, Pavel Machek wrote: > > On Thu 2017-02-09 09:11:47, Viresh Kumar wrote: > > > The switch block handles all the QOS request types present today, but > > > starts giving compilation warnings as soon as a new type is added and > > >

Re: [PATCH V2 1/6] PM / QOS: Add default case to the switch

2017-02-09 Thread Viresh Kumar
On 09-02-17, 15:24, Pavel Machek wrote: > On Thu 2017-02-09 09:11:47, Viresh Kumar wrote: > > The switch block handles all the QOS request types present today, but > > starts giving compilation warnings as soon as a new type is added and > > not handled in this. > > > > To prevent against that, ad

Re: [PATCH V2 1/6] PM / QOS: Add default case to the switch

2017-02-09 Thread Pavel Machek
On Thu 2017-02-09 09:11:47, Viresh Kumar wrote: > The switch block handles all the QOS request types present today, but > starts giving compilation warnings as soon as a new type is added and > not handled in this. > > To prevent against that, add the default case as well and do a WARN from > it.

[PATCH V2 1/6] PM / QOS: Add default case to the switch

2017-02-08 Thread Viresh Kumar
The switch block handles all the QOS request types present today, but starts giving compilation warnings as soon as a new type is added and not handled in this. To prevent against that, add the default case as well and do a WARN from it. Signed-off-by: Viresh Kumar --- drivers/base/power/qos.c