You have a FAS2240-4 and some DS4243 or DS4246 disk shelves. (Though technically the 2000 series is usually seen a shelf too)
That's the correct way to cable them, you can verify it if you run sysconfig on each, you're see something like "MULTIPATH HA" and ACP Full Connectivity near the top of the output. You can even run Config Advisor for a more detailed system overview.
"HA" is there should one of the controllers dies. If the second controller dies, the first will "takeover" it and take over things like disk and aggr ownership as well as some network connections. You do a "giveback" to return the ownership of various things back to it's partner.
https://library.netapp.com/ecmdocs/ECMP1210206/html/GUID-3D65CDAA-77DC-4D44-90D9-6F02787738B1.html