提交 5379a5fd 编写于 作者: P Phil Sutter 提交者: Ralf Baechle

MIPS: RB532: Fix bit swapping in rb532_set_bit()

The algorithm works unconditionally. If bitval is one, the first line is
a no op and the second line sets the bit at offset position. Vice versa,
if bitval is zero, the first line clears the bit at offset position and
the second line is a no op.
Signed-off-by: NPhil Sutter <n0-1@freewrt.org>
Signed-off-by: NRalf Baechle <ralf@linux-mips.org>
上级 f839490a
...@@ -119,13 +119,11 @@ static inline void rb532_set_bit(unsigned bitval, ...@@ -119,13 +119,11 @@ static inline void rb532_set_bit(unsigned bitval,
unsigned long flags; unsigned long flags;
u32 val; u32 val;
bitval = !!bitval; /* map parameter to {0,1} */
local_irq_save(flags); local_irq_save(flags);
val = readl(ioaddr); val = readl(ioaddr);
val &= ~( ~bitval << offset ); /* unset bit if bitval == 0 */ val &= ~(!bitval << offset); /* unset bit if bitval == 0 */
val |= ( bitval << offset ); /* set bit if bitval == 1 */ val |= (!!bitval << offset); /* set bit if bitval == 1 */
writel(val, ioaddr); writel(val, ioaddr);
local_irq_restore(flags); local_irq_restore(flags);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册