well it looks good to me! Of course, unless its a specialist LED, like a power LED, then there aren't any real restrictions on driving them.
I must say, 4.1Vf fora green LED seems high to me... highest I've measured/seen is 2.2V, but then again there are different materials used for different greens.
Also, you're running it at 30mA. R= 30, v = 5 - 4.1. Therefore with R = 30. I = V/R = 0.9/30 = 30mA.
That is a limit for most standard 3/5mm LED's. I'm sure you're not planning on running this permenantly on for years, but that much current will dramatically lower its operating life. No problem for the odd project, but if its an 'on' indicator for something which is powered up regularly, the hours add up!
You have 30 ohms as the series current limiting resistor, which isn't a standard value (33 is),I would raise that to say, 47ohms. Easier to get hold of, and this will lower your current to 19mA. I generally run LED's at 10mA, perhaps higher if they need to be brighter. So you could get away with 100ohms for the series resistor.
LOL you edited the post at the same time as me! gawd, so my comments are now out of date