Re: [PATCH v5] HID: corsair-void: Add Corsair Void headset family driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11. 02. 25, 13:44, Stuart wrote:
E.g. set_bit() (one bit per OP) to something like 'unsigned long flags'
in 'struct corsair_void_drvdata'. But it depends if you want to preserve
order of actions... (Which was not guaranteed with separate queues
anyway, so I assume not?)

Wouldn't using one variable for the flags risk overwriting each other, unless
it was atomic? I might be misunderstanding, I haven't played with the kernel's
work much.

set/clear/test/test_and_clear/..._bit() are guaranteed to be atomic.

--
js
suse labs




[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux