I'm not 100% sure but I am 99% sure that you only used a single grass tile in the entire map, that's a BIG... ummm "no-no" doesn't sound serious enough, just don't do it it's called tile spam in case anyone says it like that.
The river that crosses the map is very boring, and VERY straight, that's also "wrong" (since there's not really something right). Try to make it curve and flow from one end of the map to the other instead of just making a straight line. Preemptive advice here, when you add paths don't make them straight either since old roads weren't straight or even planned most of the time.
Add more stuff. Paths to start off, the middle of the map is very empty, if you're gonna have all of those bridges there (I actually suggest taking one out), then there should be a road leading to it to show that they use them. In the places where there isn't a bridge you should try to add a reason for it being there and not somewhere else.
Also you have to many villages in the top. I'd say the total number is too high for the map all on it's own, but if you have them all at the top then you really have to many villages in that map. My advice would be to leave 2 of the ones on the top but to put them in different places.
The first cliff tile (from the top) in the cliff segment to the right is in the wrong place, it should be one square to the right.
TECHNICALLY there is nothing wrong with your map (except for the cliff thing and the tile spam) but it's really boring and empty as a whole. Basically try to make it more interesing, add paths, add some more forests, light grass and stuff like that. Try reading Primes Mapping Tutorial, it's really helpful when you're just starting to map, and if you want to try making mountains check out my tutorial, they're both in the Spriters Resource thread on SF (I think the links for my tutorial in this thread got deleted).
EDIT: Hatless reminded me that I forgot to mention that the snags looks kinda funny XD. Maybe if you took out the splinters in the tile below?