It sounds good to me except you need to add an GEC.
250.30(A)(4) Grounding Electrode. The grounding electrode shall be as near as practicable to and preferably in the same area as the grounding electrode conductor connection to the system. The grounding electrode shall be the nearest one of the following:
(1) An effectively grounded structural metal member of the structure
(2) An effectively grounded metal water pipe within 1.5 m (5 ft) from the point of entrance into the building
You should take a look through all of Article 250.30 Grounding Separately Derived Alternating-Current Systems.
In there you will find how to size your bonding jumper and the grounding electrode conductor.
