Thank you for everyone's helpful input. Ive managed to get it up and running. This is the updated config I used for everyones interest. Main change was:

ORPort xxx.xxx.xxx:9001 No Listen (static public ip) ------> This was removed After this, ORPort could reach external, and form connections.

Address xxx.xxx.xxx.xxx (static public ip) -------> Removed

AddressDisableIPv6 1 ------->  Added as I'm not using IPv6 at this stage

There was low performance for a few days, however i've read
https://blog.torproject.org/lifecycle-new-relay and can see a steady increase in speed over the last few days.

@Zakwan Kalb: My relay hasn't been marked as bad, just initial misconfig due to lack of understanding, should be all sorted now.

@Georg Koppen: Not sure, but seems to be all sorted now

@Toralf: Yeah i've noticed just habit i gues :D

@John: Thx for the head up with ports, it helped with the fault finding!