Sub programing in turning

[m_c]

You need to get rid of the G90 in the subroutine.
As it stands, you switch to incremental (G91) and move X, but then you switch back to absolute (G90) and ultimately move the X to an absolute position. So rather than every iteration of your sub moving incrementally in the X, you always end up moving to an absolute X position which means the next iteration will simply repeat the exact same movements.

[Kai]

Yes I see what you mean now. I was trying to use a sub program to avoid a shed load of code to get the bar turned down and the chamfer at the same time