On Fri, 4 Jul 2014, Nicholas Krause wrote:
This patch changes this function to check for if v is returning value from the
called function. If not it does the else. Furthermore fixes the FIXME message
above
if statement for holding INVERT_ENABLE bit when not needing to be checked for
in if statement by just checking for if v is not Null.
Signed-off-by: Nicholas Krause xerofo...@gmail.com
This patch doesn't make any sense. Neither does the commit message.
Please explain exactly what this patch is intended to do in terms of the
clock framework or the hardware.
- Paul
---
arch/arm/mach-omap2/clock.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
index 591581a..626363d 100644
--- a/arch/arm/mach-omap2/clock.c
+++ b/arch/arm/mach-omap2/clock.c
@@ -379,7 +379,7 @@ void omap2_dflt_clk_disable(struct clk_hw *hw)
}
v = omap2_clk_readl(clk, clk-enable_reg);
- if (clk-flags INVERT_ENABLE)
+ if (v)
v |= (1 clk-enable_bit);
else
v = ~(1 clk-enable_bit);
--
1.9.1
- Paul
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html