[PATCH 01/23] ARM: OMAP2+: clock: move clock provider infrastructure to clock driver
Tomi Valkeinen
tomi.valkeinen at ti.com
Tue Jan 27 03:34:38 PST 2015
On 26/01/15 17:49, Tony Lindgren wrote:
>> I'm not sure if I miss something, but regmap_write does not protect from
>> problems if there are multiple users for the same registers. You need to
>> use regmap_update_bits() to get a protected read/write sequence, in
>> which you can change only the bits that you want to change.
>
> To me it seems that issue can be fixed by making all the code use regmap.
> AFAIK that should work for the legacy code too.
Even if everybody uses regmap, doing
v = regmap_read()
modify v
regmap_write(v)
is racy. regmap_update_bits() has to be used to protect the read/write
sequence. Which may be somewhat challenging at times with some strange
registers, the like Roger Q encountered recently related to CAN.
Tomi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150127/9bd4d3be/attachment.sig>
More information about the linux-arm-kernel
mailing list