No noTone needed, but delay is

Marlin/ultralcd.cpp

   if (freq > 0) {
     #if BEEPER > 0
       SET_OUTPUT(BEEPER);
-      tone(BEEPER, freq);
+      tone(BEEPER, freq, duration);
       delay(duration);
-      noTone(BEEPER);
     #elif defined(LCD_USE_I2C_BUZZER)
-      lcd.buzz(duration,freq);
+      lcd.buzz(duration, freq);
     #else
       delay(duration);
     #endif