Good work!
The only think I would question is cutting off the top of the castle - it makes the piece feel cramped and claustrophic, and generally that kind of cropping is done to put the focal point on something happening in the scene, like people crossing the bridge - or, a dragon attacking. If you showed the entire castle, with space above it, it would have more of an epic, landscape feel to it, and make the castle itself the focal point.
I’d look at your colors, it looks a bit muddy overall - I’d boost more cool colors in the distance, and in the shadows - since you have a lot of reds and yellows, I’d say use more purples, than green or blue, and the juxtaposition of red/yellow against purple will accentuate the 3d space. (The eye can’t focus on two colors that are across the color wheel from each other, which is why they use red/blue for 3D. Putting them together “vibrates” in the eye, and can give you a feeling of depth) (Don’t you love color theory?) Also, castles are typically built with local stone, I’d add more of the local color to the castle, to make it blend in a little more. Not a lot - just a little to blend.
I would also remove one of the rivers. Not only is it impossible looking, but the triangle they create distracts from the composition, and traps the eye. The human eye looks for shapes first, and that triangle literally screams at you. You don’t need to justify the bridge, as the height of the castle, and being on an outcropping does - I’d remove the rivers in front, and have it in the background behind the rocks.
The roofs on the village are’nt working, the level of detail is off - perhaps some faint indications of rows of tiles? They look out of focus.
The only other thing I’d add is signs that people actually live there. Props. Banners. Horses. People walking. A castle like that would have people going back and forth across the bridge all day. Watchmen in the ramparts. Stuff like that. Maybe some smoke from chimneys - castles are cold, dank places.
Hope this helps!
