[PATCH 1/7] media: aspeed: fix a kernel warning on clk control
benh at kernel.crashing.org
Fri May 10 09:19:19 AEST 2019
On Thu, 2019-05-09 at 10:19 -0700, Jae Hyun Yoo wrote:
> Okay. Probably I need to add one another patch in this series to address
> what you pointed out.
> I have one question. I reviewed again all bitops in this driver and
> checked that some bitops are protected by a spinlock and some others
> are not. In this case, can I mix use atomic and non-atomic bitops
> depend on each bitop's protection state by the spinlock? Or, would it be
> better to change all of them to bool in this case?
No, if some aren't protected by a lock and some are, then they need to
The question then becomes whether the unprotected ones are actually
correct or just exposing more races.
More information about the Linux-aspeed