Switching power supply

Very simple one, just a square wave and an op amp. In reality, you are not controlling whether the signal activates the transistor or not,but the duty cycle of that signal. I really appreciate comments,so go ahead! :)