I think you made a nice attempt here. The faces and hair I think turned out the best, along perhaps with the design of Fingon's robes and Maedhros' cloak. However the heraldry behind Maedhros looks very sketchy, and you have some trouble with how you draw the arms attaching.
Nice drawing, nice expressions on each of these guys, and I like how you centered their heraldry. However something that bothers me here is that Fingon and Maedhros both have the heraldry of their father's behind them, while Finrod has his own. I do not think that we know the heraldry of Fingon or Maedhros though, nor do I think we know the herladry of Finarfin, so I think this idea works.