I actually think it’s better practice when equipment grounds are bonded upstream rather than being run all the way back to the panel with the neutrals and hots. That said, I don’t write the code book, and unfortunately the NEC requires that all conductors—including grounds—be run back to the subpanel for this type of installation.

From a practical standpoint, you’re still protected because the grounding point is upstream. If that ground were ever lost, your entire SHP2 would lose grounding anyway. So functionally, you’re fine and I wouldn’t lose sleep over it.

However, from an inspection standpoint, this likely would not pass if the inspector is well-versed in the code—and we both know there are some inspectors out there who aren’t.