It looks like it's working okay now - was following mix of examples/had remnants of old code
When I changed the slave side with the following (and removed the previous lines which are covered by SPI.begin())
things seem to be working now:
#include "pins_arduino.h"
...
pinMode(MISO, OUTPUT);