The GEC is required by 250.24 to be connected to the service disconnect. That's what this guy didn't do.

The supplemental ground from the sub panel ground bus in the building to the grid is not required by the NEC, but is a design issue that I'm sure has been proven to be beneficial in all of these installations else it wouldn't be required.

-Hal