Simplify ESP32 atomic access logic

[davetcc]

We now use the inbuilt atomic swap for boolean writes, and have made reads volatile to ensure memory is always used even when read only

We now use volatile for pointers instead of atomic functions for the moment. The critical work is now done by the booleans, pointers just must never be cached. This is a small improvement for esp32.