ConfigValue: remove redundant as_pin and as_pwm methods

Pwm: use underlying pin; Switch: static Pwm

Stepper: remove redundant Pin pointers

Endstops: static Pins

TemperatureControl: static Pins

Extruder: static Pins

PauseButton: static Pins

Robot: static Pins

Merge pull request #98 from bgamari/fixes

Gcode: Again ensure no allocations occur

Gcode: Again ensure no allocations occur

For some reason `const char std::string::operator[]` still assumes that
things might be mutated. `const char& operator[]`, on the other hand,
appears to do the right thing. I'll admit that this is evidence that
`std::string` is just too subtle here. Perhaps a homebrew string type is
in order.

Reorganized members in Pin class to save 4 bytes.

Drops sizeof(Pin) down from 12 bytes per Pin to 8 bytes per Pin.

Merge pull request #94 from bgamari/fixes

Gcode: Ensure std::string operations don't incur allocations

Gcode: Ensure std::string operations don't incur allocations

_sbrk() check for overflow past MSP.

Make sure that a dynamic memory allocation won't exceed the current top
of stack.  It is still possible for stack to grow down into heap.

USB: ensure Connect pin is set up before disconnect delay

USB: work around OSX behaviour where if the device disconnects and quickly reconnects, it assumes it's the same device instead of checking

Module Events: remove on_stepper_wake_up event as it was unused

Module: don't sign all modules up for ON_CONFIG_RELOAD- they don't all use the event.

Module: use macros for event definitions to ensure consistency

USB: run USB stack from idle event instead of via interrupts

SDCard: implement busy flag

Merge branch 'edge' of github.com:arthurwolf/Smoothie into edge

added max_pwm to temperaturecontrol

Remove iprintf() call from spi.cpp

People were noticing that MRI was halting and waiting for GDB to attach
immediately at startup when configured for full debugging mode.  This
was because output was being sent to stdout and MRI was waiting to
redirect it to GDB's console.  I tracked this down to an
iprintf("\n") call in spi.cpp so I commented it out like the rest of
the iprintf() calls in this file.

Accept outside ENABLE_DEBUG_MONITOR settings.

You can now specify ENABLE_DEBUG_MONITOR setting from outside the
makefile and it will only be set to 0 if another value wasn't
provided on the make command line or environment variable.  This allows
it to be specified in builds kicked off from Eclipse.

adding info about junction_deviation

adding junction_deviation to config file

Merge branch 'edge' of github.com:arthurwolf/Smoothie into edge

removing max_jerk from Planner too

max_jerk was unused, removing

USB: silence warning when debug output is disabled

Alter USB descriptors and windows INF driver to have the same manufacturer- perhaps this will help with driver installation?

MRI Hooks: remember previous state, programmatically construct pin lists from TemperatureControl, StepperMotor and Switch

speed did not get reset between SOLO blocks in Extruder, fixing

Merge branch 'edge' of github.com:arthurwolf/Smoothie into edge

Extruder now reacts to G90 and G91

Set SDCard::_cs pin to be an output.

Smoothie didn't work with my 16GB SD card and it turns out that the
chip select GPIO pin wasn't being set to output mode.

adding current_block = NULL in player.cpp, johaness fix

Add OS X binaries for dfu-util and lpc21isp

I built these to use them on my MacBook Air with the Smoothie project
so I thought others might find it useful to have them as well.
They will be placed in the path by the default BuildShell script that
is created when install scripts are run.

Easier to configure MRI in makefile.

The default mode for MRI's usage in Smoothie is the same as before
where it only executes if a crash is encountered and isn't really
useable for things like single stepping, etc.

Now you can change the ENABLE_DEBUG_MONITOR make variable from 0 to 1
and it will switch MRI into a fuller debug experience where it takes
over UART0 and stops at init so that you have an opportunity to set
any breakpoints you might be interested in.

Merge pull request #91 from bgamari/edge

Ensure that C++11 checksum is evaluated at compile-time

Ensure that C++11 checksum is evaluated at compile-time

Wrapping the expression in an std::integral_constant<> seems to do the
right thing, even with no optimization.

Unfortunately, it seems that gcc 4.6 doesn't support user-defined string
literals, so we'll need to wait until we start building with 4.7 before
enabling this.

Make checksum macros work in non-optimized builds.

Previously the checksum macros would bloat the code because the inline math to
calculate the checksums were not constant folded in -O0 builds.  This commit
modifies the CHECKSUM() macro to just call get_checksum() instead in such
builds.

The fact that checksums are no longer constant expressions in non-optimized
builds means that they can't be used as case labels in switch statements. I
modified the affected switch statements to use if / else if statements instead.

Add bgamari's C++11 checksumming approach.

I haved added bgamari's cool C++11 user defined literals approach.  In simple
cases it generates the exact same code as the C preprocessor approach but if
multiple checkums are calculated in the same function, I see the constant
folding not happening for some which makes the code even larger than just
having the string in the code.  For this reason I have disabled it for now but
left it in so that it is easy to test out in the future and hopefully switch
to as its implementation is much shorter and clearer.

Maybe just an issue in GCC as these features are new or maybe there is an
optimizer setting that needs to be set.  I tried GCC 4.7 from
https://launchpad.net/gcc-arm-embedded/+milestone/4.7-2012-q4-major

Merge branch 'edge' into checksum_macro

https://gist.github.com/4671861 : using on_idle instead of on_main_loop caused everything to go way too deep

Calculate Fletcher Checksums at compile time from string.

You can now just call CHECKSUM macro like the following to have the checksum
calculated for you at compile time:
 #define enable_checksum CHECKSUM("enable")

This uses a set of recursive macros from src/libs/checksumm.h to inline the
checksum calculation and then the compiler uses constant folding in optimized
builds (-O1 or higher) to pre-calculate the values.

NOTE: The macros in checksumm.h currently only support a maximum string length
      of 32 characters.  If you exceed this, the macros will return a fixed value
      of 0xFFFF which won't match your string.  You should keep your
      checksummed strings below this length.

I took a copy of main.bin from before I starting making these changes and then
kept running builds and verifying that the new main.bin was exactly the same
as the original.  This means that it created the same checksums and didn't grow
the binary size at all.

the laser module now will not take the p21 pin, unless enabled. that was just silly. fixed

adding support for (comments)

Demangle C++ names in main.disasm.

Thanks to wolfmanjm on IRC for pointing out that I wasn't already doing this.

Merge pull request #87 from bgamari/edge

StreamOutput: Mark printf with __attribute__((format))

StreamOutput: Mark printf with __attribute__((format))

This allows the compiler to throw useful warnings.

StreamOutput: printf calls virtual puts so we can reduce code reuse elsewhere

StreamOutputPool: ditch printf, implement puts which distributes to each Stream's puts

USBSerial: marshall attach/detach operations to idle event

Catch dynamic de/allocations in ISRs.

removing get_value from laser too

removing a get_value from switch

added make dfu, need to merge with existing upload

Merge pull request #86 from bgamari/edge

Sundry fixes

Gcode: Use const_cast to silence warning

Gcode: Use operator[] instead of string::at to avoid allocation

Not entire sure why string::at allocates, but we don't need bounds
checking regardless. Perhaps this is due to const-ness?

Kernel: Use vector<array<Module*>> instead of flat array

As it turns out ON_CONFIG_RELOAD filled the previous array size of 32
entries. Using vector<array<...>> here does incur a memory indirection,
which Arthur rightly points out will have a performance penalty in a hot
path. That being said, we'll use this for now. Eventually it might make
sense to move to array<array<...>> with a sufficiently large size.

use MRI hooks to disable heaters in debug mode

Add __mriPlatform_EnteringDebuggerHook() and __mriPlatform_LeavingDebuggerHook() to MRI.

Upgrades to version 0.5 of MRI.

A debuggee can implement these "C" hook routines to be called when MRI enters and
leaves a debug exception to do cleanup such as turn off heaters so that they
don't remain on while sitting in the debugger.

NOTE: You can assume very little about the state of your program from these
      routines.  I recommend that they be written in low level C code.  It
      isn't even safe to assume that C++ global constructors have been run
      before these routines are called.

Example which just toggles a LED on while in debug exception:
extern "C" void __mriPlatform_EnteringDebuggerHook(void)
{
    LPC_GPIO1->FIODIR |= 1 << 18;
    LPC_GPIO1->FIOPIN |= 1 << 18;
}

extern "C" void __mriPlatform_LeavingDebuggerHook(void)
{
    LPC_GPIO1->FIODIR |= 1 << 18;
    LPC_GPIO1->FIOPIN &= ~(1 << 18);
}

Merge bgamari/median into edge

TemperatureControl: Use median smoothing instead of mean

added make dfu

Secret Sauce for windows to activate the driver it comes installed with

TemperatureControl: PID resets I to 0 if output exceeds bounds to reduce anti-windup effect

SlowTicker: implement G4 pause

Gcode: get_int method to shortcut string->float->int conversion

utils: is_alpha, is_digit, is_numeric, is_alphanum, is_whitespace utility functions

Extruder: make sure we have a block to release, avoid segfaults

USB DFU: keep usb connected while we respond to reset request.

we also become a kernel module so we can receive the on idle event. This would be an ideal place to do shutdown tasks, eg save position somewhere

Robot: remove unused variable

version change

Merge pull request #83 from bgamari/fixes

Some of these are small changes extracted from @triffid's std::string -> char* conversion which are worth on their own. Each patch is small, self-contained, and hopefully pretty non-controversial.

SerialConsole: reserve a reasonably sized buffer

utils: Make string utils const where possible

This will help prevent unnecessary allocations

Endstops: Fix whitespace

CircBuffer: Add peek()

Kernel: Use bzero instead of loop

Robot: add M220 speed override percentage, ala Marlin et al.

M220 S100 = normal speed, M220 S200 = double speed, M220 S50 = half speed, etc.

Axis limits are still obeyed to avoid skipped steps with ludicrous numbers, and a 1% minimum is enforced in case of accidental mis-keying and also to prevent the block queue locking up.

There is no maximum ;)

TemperatureControl: increase default I_MAX to PWM_MAX so marginally strong heaters have a chance to achieve temperature

TemperatureControl: honor readings_per_second configuration option

Merge branch 'feature/on_second_tick_event' into edge

TemperatureControl: report temperatures every second while waiting for warmup

move on_second_tick callout to on_idle event, also provide a flag to detect interrupt context

Events: add ON_SECOND_TICK event that fires once per second. Also finish moving event stuff to Module.{h,cpp} so it's all in one place

USB: ensure interrupts are disabled while getting LPC serial number

Merge branch 'fix/gcode_freed_while_in_queue' into edge

allocate Gcode objects on the heap, don't clean them up until the related block becomes unused

makefile: flash target to use defined console, and correct crystal frequency

looks like clear_vector_double might be broken, trying clear_vector instead

bare G92 by itself should behave now

Reported-by: Ben Gamari <bgamari.foss@gmail.com>

Merge branch 'feature/extra-usbserial-interface' into edge

another USBSerial interface

always add MSD descriptors, don't wait for connect

Merge branch 'hotfix/db35a1-typo' into edge

fix typo - https://github.com/arthurwolf/Smoothie/commit/db35a1525f75c50fb64f9fa0947dd0d421a2ca12#commitcomment-2365308

Reported-by: bwhoward