Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40126 )
Change subject: src: drop 'scan_bus = NULL' and 'scan_bus = 0' ......................................................................
Patch Set 4:
Patch Set 2: Code-Review+1
Patch Set 2:
How about dropping these lines? `struct device_operations` has 21 fields, I don't see what makes `.scan_bus` so special that it needs an explicit NULL while 15 other fields don't.
I've been replacing them with NULL on some changes, but if the structs are zero'd automatically, then it shouldn't change a thing.
Not the whole structs but the missing fields are zero'd:
"If there are fewer initializers in a brace-enclosed list than there are elements or members of an aggregate, or fewer characters in a string literal used to initialize an array of known size than there are elements in the array, the remainder of the aggregate shall be initialized implicitly the same as objects that have static storage duration."