]> git.baikalelectronics.ru Git - kernel.git/commit
can: c_can: use regmap_update_bits() to modify RAMINIT register
authorRoger Quadros <rogerq@ti.com>
Tue, 13 Jan 2015 14:23:11 +0000 (16:23 +0200)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Thu, 15 Jan 2015 15:58:00 +0000 (16:58 +0100)
commit13ef5c03c1b442f6b4a406c0605295c51d359964
treeaba6582cd3bf0a8f2fd58b5f47ffd3c60486a527
parent4b8361adbd55b916ca87acd59d102f9c51835e5a
can: c_can: use regmap_update_bits() to modify RAMINIT register

use of regmap_read() and regmap_write() in c_can_hw_raminit_syscon()
is not safe as the RAMINIT register can be shared between different drivers
at least for TI SoCs.

To make the modification atomic we switch to using regmap_update_bits().

regmap_update_bits() skips writing to the register if it's read content is the
same as what is going to be written. This causes an issue for us when we
need to clear the DONE bit with the initial condition START:0, DONE:1 as
DONE bit must be written with 1 to clear it.

So we defer the clearing of DONE bit to later when we set the START bit.
There we are sure that START bit is changed from 0 to 1 so the write of
1 to already set DONE bit will happen.

Signed-off-by: Roger Quadros <rogerq@ti.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/c_can/c_can_platform.c