Commit dca17146 authored by Ben Gardner's avatar Ben Gardner Committed by Linus Torvalds

w1_ds2433: clear the validcrc flag after a write

The w1_ds2433 driver does not read from the hardware if the CRC was valid
on the last read.  The validcrc flag should be cleared after a write so
that the new value can be read.
Signed-off-by: default avatarBen Gardner <gardner.ben@gmail.com>
Signed-off-by: default avatarEvgeniy Polyakov <johnpol@2ka.mipt.ru>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 221be177
...@@ -156,6 +156,9 @@ out_up: ...@@ -156,6 +156,9 @@ out_up:
*/ */
static int w1_f23_write(struct w1_slave *sl, int addr, int len, const u8 *data) static int w1_f23_write(struct w1_slave *sl, int addr, int len, const u8 *data)
{ {
#ifdef CONFIG_W1_SLAVE_DS2433_CRC
struct w1_f23_data *f23 = sl->family_data;
#endif
u8 wrbuf[4]; u8 wrbuf[4];
u8 rdbuf[W1_PAGE_SIZE + 3]; u8 rdbuf[W1_PAGE_SIZE + 3];
u8 es = (addr + len - 1) & 0x1f; u8 es = (addr + len - 1) & 0x1f;
...@@ -196,7 +199,9 @@ static int w1_f23_write(struct w1_slave *sl, int addr, int len, const u8 *data) ...@@ -196,7 +199,9 @@ static int w1_f23_write(struct w1_slave *sl, int addr, int len, const u8 *data)
/* Reset the bus to wake up the EEPROM (this may not be needed) */ /* Reset the bus to wake up the EEPROM (this may not be needed) */
w1_reset_bus(sl->master); w1_reset_bus(sl->master);
#ifdef CONFIG_W1_SLAVE_DS2433_CRC
f23->validcrc &= ~(1 << (addr >> W1_PAGE_BITS));
#endif
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment