* Refactor checksum to unique function
* Clear uart before reading data
* Add ack/nack check
* Implement output disable before requesting GSOF data
* Improve debug message to have line number
* Use debug in more code
* Stop delaying in configuration
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
* Instead of hard coding to COM2, allow users to set it
* The enum is confusing, so this needs a wiki entry
* Use the same port in requestBAUD
* If the user configures an invalid param, send an error
* Add values for the GSOF COM ports
* Fix bug in RS232 being port 3 instead of port 0
* Use set_default for the typical user value when the GSOF driver is run
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
* This removes magic numbers of hard coding the hardware port and output
rate
* This also fixes configuring the incorrect hardware port
* Now, COM2 (TTL) is configured for GSOF output
* The data rate remains the same as before
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
* Related to #23743, but I missed this since I didn't realize the buffer was not modified in-place
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
Fixes the issue of three unused variables, two of which were used in a
commented Debug() call.
To keep the convenient debug message (and the variable names for the
data bytes), this patch uncomment the debug call but wrap the variables
and the debug call around an ifdef for the local symbol
gsof_DEBUGGING. So by turning it on, the debug will already be in place.
The Debug() call was modified to actually compile and include the third
variable in the output.