hal fhss timer: removed unnecessary and potentially unsafe memset

[JarkkoPaso]

Description

hal fhss timer: removed unnecessary and potentially unsafe memset

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Release Notes

[ciarmcom]

@JarkkoPaso, thank you for your changes.
@ARMmbed/mbed-os-ipcore @ARMmbed/mbed-os-maintainers please review.

[kjbracey]

While you're here, could you make the structure type and array not global?

Stick namespace { } around both the type and array to put them in an anonymous namespace. You could actually do that for all your other objects above, instead of the statics.

See bit about anonymous namespaces in the Organization section of https://os.mbed.com/docs/mbed-os/v5.13/contributing/software-design.html

static is fine for objects, but doesn't work for the types - namespace { } handles both.

[kjbracey]

Actually, all your currently-static functions and the using declarations could be in the anonymous namespace rather than static. Would keep the style consistent within the file. Only the fhss_functions needs to be global.

Technically fhss_functions should be declared extern "C", if you're accessing it from C code, but I don't think any of our toolchains care.

[0xc0170]

Above ^^

[JarkkoPaso]

@kjbracey-arm please review again

[0xc0170]

CI started

[mbed-ci]

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

[0xc0170]

@JarkkoPaso always provide description please. It is required. I fixed it here