Hey, really love the environment above. Environments aren't really my focus, but from what I've learned, maybe you could try adding some storytelling elements. Maybe a boat loading or unloading cargo onto the bay in the distance (the type of ship could determine what they are doing i,e fishing, pirates) then the viewer knows that its a fishing town for example.
Or with the bridge, why is there a bridge? if its for protection from invaders, maybe it should be a drawer bridge? or a sleeping guard! if there is a mine or something underneath, add a few lanterns or an broken mining kart. Could be that there is some shady business going on down there like black market transaction - add a few characters handing over some items.
These kind of elements tell the viewer a bit more about what they are looking at and what this place is like. Makes the piece a bit more interesting
Composition is great, values are great, the perspective I can't see any glaring issues with
I hope this gives you another avenue to explore! really good stuff!