many thanks to @sh83 for tracking down this bug. When write_all() is
called with non-zero offset in init it ends up calling write with an
invalid offset, triggering a full reset
This fixes issue #10874
this fixes two issues found by night-ghost. The first occurs if a 16
bit flash write is not atomic and only the first byte is written. Then
we could end up declaring the data invalid on reboot and erasing.
The second is a race in switching sectors. If power off occurs at the
wrong time we would trigger en erase on reboot.
Andrew Tridgell
Peter Barker