Tidy up hm2_soc_ol and amend 5i25_socfpga sims to use new format

Move logical test for config to earlier in function and catch all
Local values used to shorten iterations
Change LL_ERR used in testing because they always print, to LL_DBG

Change call to hm2_soc_register.
If we are passing the hm2_soc_t *brd, there is no point passing
values separately that are already contained within it

Modify socfpga sim configs to use new param passing protocols
Uses argc/argv mechanism for passing strings to hm2_soc_ol

Signed-off-by: Mick <arceye@mgware.co.uk>

Fixup for 17b665d: remove 'sleep' hack & fix race condition in tests

17b665d was a failed attempt to hack around a race condition by
lengthening a 'sleep'; this simply made the problem appear less
frequently.

The problem only occurred in the toolchanger/toolno-pocket-differ/*
tests.  Comparing with other successful linuxcncrsh-based tests turned
up 9748fe9 and the "set wait done" command, which clears up the
problem, even with the original 'sleep' hack removed.