Not sure why you left the crossing=controlled tag instead of changing it to crossing=traffic_signals. crossing=controlled is not documented on the OSM wiki, has just 924 uses worldwide on taginfo, and seems to mean the exact same thing as crossing=traffic_signals, so I think it should not be used and maybe eventually eliminated.