kernel/drivertools.h: avoid maybe-uninitialized compile warnings

[ldoolitt]

Initialize "unsigned int inner" in hash() functions
Includes a log_assert() that might help catch corrupted data structures or future incomplete modification of DriveType definition

See Issue #4713, Reduce compiler warnings when building yosys

[KrystalDelusion]

Huh, this actually doesn't affect the number of warnings in the git actions. I wonder if this is because we only test gcc-10 and gcc (which is apparently gcc-11) where your issue mentions gcc-12. Could indicate that we could/should declare a newer gcc target.

[ldoolitt]

Huh, this actually doesn't affect the number of warnings in the git actions. I wonder if this is because we only test gcc-10 and gcc (which is apparently gcc-11) where your issue mentions gcc-12. Could indicate that we could/should declare a newer gcc target.

gcc-12 is default in Debian Stable, so not exactly bleeding-edge. I'm starting to test code with gcc-14 in Debian Testing, which is likely to ship (as the new Stable) next Spring.