Hallo,

vom Prinzip her richtig (siehe z.B. Assembler-Tutorial bei mikrocontroller.net)
Allerdings musst du deinen Takt nochmal durch 256 (bei 8 bit) teilen.
Somit wäre die Frequenz zu niedrig - mehr als 100 Herz solltens schon sein damit die LED bei niedrigem Tastverhältnis auch nicht bklinkt!

Für die Leistungstreiber kannst du auch n Transistorarray nehmen(Wenn der gegen GND schaltet kannst du die LED-Spannung erhöhen.
Ansonnsten könnte man noch n MosFET nehmen!

MFG Mixxer