Re: [PATCH] arm: omap2: vc: fix 'or' always true warning

2015-09-01 Thread Frans Klaver
On Tue, Sep 1, 2015 at 5:46 PM, Tony Lindgren wrote: > * Frans Klaver [150828 03:14]: >> From: Frans Klaver >> >> Fix the warning: >> arch/arm/mach-omap2/vc.c:302:47: warning: logical ‘or’ of collectively >> exhaustive tests is always true [-Wlogical-op] >> >> As we're toggling both CLKREQ and

Re: [PATCH] arm: omap2: vc: fix 'or' always true warning

2015-09-01 Thread Tony Lindgren
* Frans Klaver [150828 03:14]: > From: Frans Klaver > > Fix the warning: > arch/arm/mach-omap2/vc.c:302:47: warning: logical ‘or’ of collectively > exhaustive tests is always true [-Wlogical-op] > > As we're toggling both CLKREQ and OFFMODE, we should also be checking > OFFMODE. > >

Re: [PATCH] arm: omap2: vc: fix 'or' always true warning

2015-09-01 Thread Frans Klaver
On Tue, Sep 1, 2015 at 5:46 PM, Tony Lindgren wrote: > * Frans Klaver [150828 03:14]: >> From: Frans Klaver >> >> Fix the warning: >> arch/arm/mach-omap2/vc.c:302:47: warning: logical ‘or’ of collectively >> exhaustive tests is

Re: [PATCH] arm: omap2: vc: fix 'or' always true warning

2015-09-01 Thread Tony Lindgren
* Frans Klaver [150828 03:14]: > From: Frans Klaver > > Fix the warning: > arch/arm/mach-omap2/vc.c:302:47: warning: logical ‘or’ of collectively > exhaustive tests is always true [-Wlogical-op] > > As we're toggling both CLKREQ and OFFMODE, we

[PATCH] arm: omap2: vc: fix 'or' always true warning

2015-08-28 Thread Frans Klaver
From: Frans Klaver Fix the warning: arch/arm/mach-omap2/vc.c:302:47: warning: logical ‘or’ of collectively exhaustive tests is always true [-Wlogical-op] As we're toggling both CLKREQ and OFFMODE, we should also be checking OFFMODE. Signed-off-by: Frans Klaver --- arch/arm/mach-omap2/vc.c |

[PATCH] arm: omap2: vc: fix 'or' always true warning

2015-08-28 Thread Frans Klaver
From: Frans Klaver franskla...@gmail.com Fix the warning: arch/arm/mach-omap2/vc.c:302:47: warning: logical ‘or’ of collectively exhaustive tests is always true [-Wlogical-op] As we're toggling both CLKREQ and OFFMODE, we should also be checking OFFMODE. Signed-off-by: Frans Klaver