You need to load a timer reload value.
Nah, the initial 0 is good, gives 256 counts Even though the port pin is swinging up and down (and I assume it is properly connected to light the LED), the LED will appear to be lit steady if the pin is toggling at too high of speed.Yes, change "counter" from char to int.

Erik

PS
counter = 0;
serves no purpose