i2c_ecc,random: remove duplicate definitions

[benma]

These symbols are already defined in driver_init.c.

The current GCC version doesn't complain, I discovered this when trying to build with an updated GCC (arm-gnu-toolchain-11.3.rel1-x86_64) and getting errors about these redefinitions.

The reason that the current GCC version (GCC 8) didn't complain and the updated one (GCC 11) did complain is that the default for the -fcommon flag changed from on to off in GCC 10.

See:

• https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85678
• https://gcc.gnu.org/onlinedocs/gcc/Code-Gen-Options.html (-fcommon)

[benma]

@NickeZ any idea why the old compiler would not complain about this but the new would? duplicate definition should not work on the old compiler (the one currently in use) :thinking:

[benma]

@NickeZ any idea why the old compiler would not complain about this but the new would? duplicate definition should not work on the old compiler (the one currently in use) thinking

Found the reason and documented it in the commit msg.