Switching tone on and off at 120 bpm not working
50000000/2 requires 25 bits, not 15 (log base 2 of 25000000 is 24.575, the ceil of that is 25).  In the second module, counter gets set to 30783 instead of 25000000 so the 'BPM' is ~97456 instead of 120.