Grundsätzlich ist alles richtig, aber...
Aus einem AVR sollte man nicht mehr als 20mA an einem Pin herausziehen.
Die max. Werte sind im Datenblatt unter Electrical Characteristics.
Leds schaltet man so nicht parallel.
Eine davon hat einen geringfügig anderen Wert, zieht etwas mehr Strom, wird etwas wärmer, zieht noch mehr Strom, usw., dann eine kaputt, die restlichen haben dann mehr Strom........
Also für jede Led bei Parallelschaltung einen eigenen Widerstand.