Report actual RPM and PWM instead of just target

The way `report_speed` was implemented, `M957` shows `target_rpm` as the
"Current RPM" and then calculates the PWM "Analog value" from there.

Because `turn_off` uses `update_pwm(0)` to turn off the output signal
without clobbering the `target_rpm` set by `M3`, if you `turn_off` the
spindle (via `M5`, a `HALT` state, etc), `M957` (`report_speed`)
incorrectly continues to report the old "Current RPM" and calculated
"Analog value" even thought the PWM signal has actually been cut off.

I turned this around so `report_speed` reads the real PWM ("Analog
value") being sent out `pwm_pin` and then calculates the real "Current
RPM" from there.  I also appended a new "Target RPM" field to the
`report_speed` output containing `target_rpm`.

Tested over USB serial with `M597`, `M3`, `M5`, `M112`, `M999`, `^X`,
and `$X`.

Merge pull request #1236 from cilynx/fix/spindle_off_on_halt

Fix/spindle off on halt

Add spindle.ignore_on_halt to config sample snippet

Consolidate common spindle functions

Doing the exact same event registration in every sub-module isn't very
DRY and could lead to inadvertent missed registrations in new
sub-modules.  Consolidating the registrations into `SpindleMaker`.

Stop spindle on HALT

When a machine enters a `HALT` state for any reason, the spindle should
stop for safety along with everything else.  This commit changes the
default behavior to `turn_off` the spindle `on_halt`.  I added an
optional `spindle.ignore_on_halt` config parameter that defaults to
`false` in case anyone needs the previous less-safe behavior.

I believe this addresses #1069.

I confirmed functionality on `PWMSpindleControl` via `M957` as well as a
physical scope while causing `HALT` via `M112` or `^X` over the serial
interface.

I found some weirdness in `AnalogSpindleControl` around `report_speed`,
but I'll address that in another PR.  I did confirm that `turn_off` is
being called `on_halt` as it should be and the PWM duty cycle is reduced
to 0 even though `M957` (`report_speed`) continues to report that the
spindle is moving.

Without having the equipment for the other side of the RS485 connection,
I can't do much to test `ModbusSpindleContol`, but
`HuanyangSpindleControl`, the only supported Modbus VFD right now,
implements `turn_off`, so I don't see any reason it shouldn't work fine
with this change.

update cnc firmware latest

move spindle.config

Merge pull request #1235 from cilynx/feature/spindle_max_pwm

Feature/spindle max pwm

Feature/spindle max pwm

Treadmill motor controllers are becoming increasingly popular in DIY
machines.  One such popular controller is the mc2100 which expects a
duty cycle of 0-85% for 0-100% speed.  To keep the PWM signal below the
controller maximum and thus prevent the controller from going into an
error state when higher speeds than physically possible are requested,
we need a config option for the ceiling on the PWM signal.

I tested this for backwards compatibility with my original configuration
without this new parameter included and it works as expected, using 100%
as the maximum duty cycle.

I tested setting `spindle.max_pwm` to 0.85 (above the hard-coded 0.5
starting point) and 0.25 (below the hard-coded 0.5 starting point) and
confirmed expected function for both via M957 output and viewing the
physical PWM output with a scope.

I added a sample `spindle.config` showing this new parameter in context
with the other spindle parameters I could find with a quick grep of the
code and search through the docs.

Merge pull request #1234 from cilynx/fix/get_pid_settings

Align get_pid_settings() with report_settings()

Merge pull request #1233 from cilynx/fix/spindle_on

Remove conflicting bool spindle_on

Align get_pid_settings() with report_settings()

In the handler for M958 in `SpindleControl`, a function,
`get_pid_settings()`, is called which does not appear to exist beyond
its virtual initialization also in `SpindleControl`.  Because this
function does not really exist, M958 does not return any PID data.

`PWMSpindleControl` contains a function `report_settings()`, which
prints the PID data one would expect from M958.  Replacing
`SpindleControl`'s references to the non-existent `get_pid_settings()`
with `report_setting()` causes M958 to return PID data as expected.

Remove conflicting bool spindle_on

Bool `spindle_on` is initialized in `SpindleControl` and is shared
between `SpindleControl` and whichever control method (PWM, Modbus,
Huanyang, or Analog) is configured.

Initializing `spindle_on` in `PWMSpindleControl` as well clobbers the
shared variable and leads to `SpindleControl` and `PWMSpingleControl`
having different values for `spindle_on` and thus the spindle is never
actually turned on when sending M3 commands.

This can be confirmed via M957 which will always show that the current
PWM is 0, no matter what the current and target RPM.

Removing this superfluous initializion causes `spindle_on` from
`SpindleControl` to be shared as expeced and the module then works as
can be confirmed with M957 showing appropriate PWM values responding to
current and target RPM.

Merge pull request #1230 from oddstr13/edge

Added COPYING file

Added COPYING file

update firmware.bin

update latest bins (same as normal bin)

update firmware.bin

Merge pull request #1226 from Zaaphod/Fix_XZ_Arcs

Fix Math for Arcs in XZ Plane - Issue  #1181

Fix Math for Arcs in XZ Plane - Issue  #1181

The math for calculating angular travel needs to be reversed only for XZ arcs, this is because the arc is now starting and ending in different quadrants than a similar arc in the other 2 planes.

Merge pull request #1223 from DavidJRobertson/make-debug-store-win

Fix `make debug-store` (and hence `make upload`) on Windows

Fix `make debug-store` (and hence `make upload`) on windows

Note that the cd commands are to avoid having to translate between / and \ as path separators

Fix laser power read when pwm is inverted

Merge remote-tracking branch 'upstream/edge' into upstreamedge

Merge pull request #891 from greyltc/ISR-SPI-fix

Max31855: stop using SPI from ISR

Merge pull request #4 from JeffreyRodriguez/greyltc_ISR-SPI-fix_mergeEdge

Greyltc isr spi fix - merge edge

Merged with current edge.

Merge pull request #1214 from DavidJRobertson/configurable-kill-poll-rate

Add configuration option kill_button_poll_frequency, see #1212.

Add configuration option kill_button_poll_frequency, see #1212.

Merge pull request #1211 from DavidJRobertson/fix/panel-probe-truncated

Fix truncation of command output from panel probe screen on large displays

Fix truncation of command output from panel probe screen on large displays.

Merge pull request #1204 from sdavi/Feature/ZProbe-Dwell

Dwell before probing

Updated float comparison

Dwell before probing

Adds a configurable delay before probing

fix a potential typo in processing segments

Merge pull request #1200 from wolfmanjm/fix/delta-grid-write-corruption

close and reopen file every 400 bytes to avoid corruption

Merge remote-tracking branch 'upstream/edge' into fix/delta-grid-write-corruption

Merge pull request #1201 from wolfmanjm/upstreamedge

update firmware-latest

update firmware-latest

Merge remote-tracking branch 'upstream/edge' into fix/delta-grid-write-corruption

Merge pull request #1199 from wolfmanjm/hack/interim-feed-hold

interim feed hold

also fix cart grid

close and reopen file every 400 bytes to avoid corruption

Merge remote-tracking branch 'upstream/edge' into hack/interim-feed-hold

Update config

Update config

Update config

Update config

Update config

Update config

add new_status_format for grbl1.1-like status format which is easier to parse and adds ABC axis to MCS, and F: and S: and L:

Merge branch 'upstreamedge' into hack/interim-feed-hold

Merge pull request #1195 from wolfmanjm/upstreamedge

Fix rectangular grid find bed, which was broken

fix error message

Fix rectangular grid find bed, which was broken

Merge pull request #1194 from wolfmanjm/upstreamedge

change prepare screen menu 'set home' to 'set X0Y0' to more properly …

Merge branch 'upstreamedge' into hack/interim-feed-hold

change prepare screen menu 'set home' to 'set X0Y0' to more properly reflect what it does and is supposed to do

Merge branch 'upstreamedge' into hack/interim-feed-hold

Conflicts:
FirmwareBin/firmware-latest.bin

Merge pull request #1193 from wolfmanjm/upstreamedge

add zprobe.max_z config option

update firmware-latest.bin

add zprobe.max_z config option (falls back to gamma_max if not defined)
fix delta homing for A only so Z does not move back when only homing A(BC)

Add ABC or E to the ? query

fix comment typo

allow feed hold to be enabled independently of grbl mode, so in grbl mode it is set in 3d printer mode it is not set by default

initial attempt to implement a feedhold.
  this acts like the planner queue is full, but will not pause immediately as the existing stuff in the planner queue will need to complete
  in grbl-mode ! will pause and ~ will resume

Merge pull request #1192 from wolfmanjm/upstreamedge

fix G28 A0 for deltas, should be able to home just the A (or B or C) …

fix G28 A0 for deltas, should be able to home just the A (or B or C) axis

Merge pull request #1191 from wolfmanjm/fix/delta-a-endstop

Fix/delta a endstop

update katest bin

Allow ABC axis to be homed on a delta

Merge pull request #1189 from wolfmanjm/upstreamedge

remove debug printf from zprobe

updat firmnware

remove debug printf

Merge pull request #1188 from wolfmanjm/upstreamedge

reduce stack usage for  zprobes when run from panel

Fix coordinated move

reduce stack usage for cart grid G32 and zprobe

Merge pull request #1187 from wolfmanjm/upstreamedge

fix momentary input switch to reflect input state not toggle it

fix momentary input switch to reflect input state not toggle it

Merge pull request #1186 from wolfmanjm/upstreamedge

Save state when jogging G0

So streaming over network dos not hog all of main loop we now issue one net command per main loop iteration
this allows another telnet connection and/or a USB serial connection to actually get a response

do not send > prompt for a non telnet client connection

Merge remote-tracking branch 'upstream/edge' into upstreamedge

add command 'switch name' to query the state of a named switch

Add an AutoPushPop and RAII style to push state and pop state on exit of scope
Use above for the test commands so we don;t leave G0 feedrate set incorrectly

Merge pull request #1184 from nicofouli/patch-1

Fix missing zprobe_offset

Fix missing zprobe_offset

I added missing zprobe_offset values in the find_bed function to avoid the probe looking for the bed outside of the print surface. Now puts correctly the probe at 0,0.

Merge pull request #1183 from wolfmanjm/upstreamedge

update firmware-latest.bin

update firmware-latest.bin

Merge pull request #1182 from wolfmanjm/upstreamedge

save state when doing z probe

when doing zprobes save state first then restore it

Merge remote-tracking branch 'upstream/edge' into upstreamedge

remove old extruder config settings #defines

Merge pull request #1179 from jkoelker/p9_fix

plan9: Update error string names

plan9: Update error string names

Several of the error names are int values defined elsewhere. Prefix them
with `P9_` to prevent `plan9.cpp:27:12: error: expected unqualified-id
 before numeric constant`.

Update abc-endstop.config

Merge pull request #1177 from wolfmanjm/upstreamedge

update the *-latest.bin binaries

update the latest bin versions

Merge remote-tracking branch 'upstream/edge' into upstreamedge

update firmware.bin