Re: [PATCH] arm: omap2: vc: fix 'or' always true warning
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 OFFMODE, we should also be checking >> OFFMODE. >> >> Signed-off-by: Frans Klaver > > Thanks apppying into omap-for-v4.3/fixes. Thanks! Frans -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] arm: omap2: vc: fix 'or' always true warning
* 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. > > Signed-off-by: Frans Klaver Thanks apppying into omap-for-v4.3/fixes. Tony > --- > arch/arm/mach-omap2/vc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/mach-omap2/vc.c b/arch/arm/mach-omap2/vc.c > index 076fd20d7e5a..807bc79e3e3d 100644 > --- a/arch/arm/mach-omap2/vc.c > +++ b/arch/arm/mach-omap2/vc.c > @@ -300,7 +300,7 @@ static void __init omap3_vc_init_pmic_signaling(struct > voltagedomain *voltdm) > > val = voltdm->read(OMAP3_PRM_POLCTRL_OFFSET); > if (!(val & OMAP3430_PRM_POLCTRL_CLKREQ_POL) || > - (val & OMAP3430_PRM_POLCTRL_CLKREQ_POL)) { > + (val & OMAP3430_PRM_POLCTRL_OFFMODE_POL)) { > val |= OMAP3430_PRM_POLCTRL_CLKREQ_POL; > val &= ~OMAP3430_PRM_POLCTRL_OFFMODE_POL; > pr_debug("PM: fixing sys_clkreq and sys_off_mode polarity to > 0x%x\n", > -- > 2.3.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] arm: omap2: vc: fix 'or' always true warning
On Tue, Sep 1, 2015 at 5:46 PM, Tony Lindgrenwrote: > * 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. >> >> Signed-off-by: Frans Klaver > > Thanks apppying into omap-for-v4.3/fixes. Thanks! Frans -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] arm: omap2: vc: fix 'or' always true warning
* 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. > > Signed-off-by: Frans Klaver Thanks apppying into omap-for-v4.3/fixes. Tony > --- > arch/arm/mach-omap2/vc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/mach-omap2/vc.c b/arch/arm/mach-omap2/vc.c > index 076fd20d7e5a..807bc79e3e3d 100644 > --- a/arch/arm/mach-omap2/vc.c > +++ b/arch/arm/mach-omap2/vc.c > @@ -300,7 +300,7 @@ static void __init omap3_vc_init_pmic_signaling(struct > voltagedomain *voltdm) > > val = voltdm->read(OMAP3_PRM_POLCTRL_OFFSET); > if (!(val & OMAP3430_PRM_POLCTRL_CLKREQ_POL) || > - (val & OMAP3430_PRM_POLCTRL_CLKREQ_POL)) { > + (val & OMAP3430_PRM_POLCTRL_OFFMODE_POL)) { > val |= OMAP3430_PRM_POLCTRL_CLKREQ_POL; > val &= ~OMAP3430_PRM_POLCTRL_OFFMODE_POL; > pr_debug("PM: fixing sys_clkreq and sys_off_mode polarity to > 0x%x\n", > -- > 2.3.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] arm: omap2: vc: fix 'or' always true warning
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 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-omap2/vc.c b/arch/arm/mach-omap2/vc.c index 076fd20d7e5a..807bc79e3e3d 100644 --- a/arch/arm/mach-omap2/vc.c +++ b/arch/arm/mach-omap2/vc.c @@ -300,7 +300,7 @@ static void __init omap3_vc_init_pmic_signaling(struct voltagedomain *voltdm) val = voltdm->read(OMAP3_PRM_POLCTRL_OFFSET); if (!(val & OMAP3430_PRM_POLCTRL_CLKREQ_POL) || - (val & OMAP3430_PRM_POLCTRL_CLKREQ_POL)) { + (val & OMAP3430_PRM_POLCTRL_OFFMODE_POL)) { val |= OMAP3430_PRM_POLCTRL_CLKREQ_POL; val &= ~OMAP3430_PRM_POLCTRL_OFFMODE_POL; pr_debug("PM: fixing sys_clkreq and sys_off_mode polarity to 0x%x\n", -- 2.3.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] arm: omap2: vc: fix 'or' always true warning
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 franskla...@gmail.com --- arch/arm/mach-omap2/vc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-omap2/vc.c b/arch/arm/mach-omap2/vc.c index 076fd20d7e5a..807bc79e3e3d 100644 --- a/arch/arm/mach-omap2/vc.c +++ b/arch/arm/mach-omap2/vc.c @@ -300,7 +300,7 @@ static void __init omap3_vc_init_pmic_signaling(struct voltagedomain *voltdm) val = voltdm-read(OMAP3_PRM_POLCTRL_OFFSET); if (!(val OMAP3430_PRM_POLCTRL_CLKREQ_POL) || - (val OMAP3430_PRM_POLCTRL_CLKREQ_POL)) { + (val OMAP3430_PRM_POLCTRL_OFFMODE_POL)) { val |= OMAP3430_PRM_POLCTRL_CLKREQ_POL; val = ~OMAP3430_PRM_POLCTRL_OFFMODE_POL; pr_debug(PM: fixing sys_clkreq and sys_off_mode polarity to 0x%x\n, -- 2.3.4 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/