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

Accidentally left a hard-coded position in.

Added position reset at homing end; Added min and max axis positions.

Merge pull request #108 from logxen/edge

commented the second usbserial to simplify life for new users

commented the second usbserial to simplify life for new users

Merge pull request #105 from wolfmanjm/feature/playprogress

added new simpleshell commands progress, abort and help

added new simplshell commands progress, abort and help

Merge branch 'feature/USBSerial-works-outside-ISR-context' into edge

USBSerial: remove redundant commented code

SimpleShell: cat: use puts instead of printf("%s") for efficiency

USBSerial: call USBHW::usbisr() to drain queue when necessary

USBSerial: enough room in buffer for 2 full USB packets, plus a little extra to maintain alignment

CircBuffer: use actual size specified, report free space correctly

CircBuffer: move USBSerial buffers into USB ram to conserve main ram

Merge pull request #104 from bgamari/edge

Gcode: Ensure queued is initialized

Gcode: Ensure queued is initialized

This was a silly and obvious oversight from
6a3328dfe514e66a98c0cf5adc7252d333567e83. Thanks to wolfmanjm for
spotting it.

Merge pull request #102 from bgamari/fixes

Player: Reduce block queue depth to 16

Player: Reduce block queue depth to 16

It seems we don't have enough memory to keep this many blocks around;
the heap collides with the stack on LPC1769.

Merge pull request #92 from wolfmanjm/move_digipot

move digipot to CurrentCOntrol Module

Merge pull request #96 from wolfmanjm/upstreamedge

add closedir to simpleshell commands that open directories

Fixes issue #101

Simplified naming convention of sample config file and re-organized into
directory structure where more than one sample can live with each sample having
the correct final name of "config".

I have included a sample README.TXT file in the ConfigSamples directory that
others more knowledgable about this area may want to update.

Switch gcc4mbed.c license to Apache v2

I never meant for this to have a LGPL license on it.  I must have
copied that from MRI code base at some point.  I should have been
Apache v2.

SimpleShell: use strtol to check for valid line limit instead of int(atof(...))

Gcode: simplify get_value with strtod

Merge pull request #100 from bgamari/fixes

Various things encountered during search for source of heap corruption

StreamOutput: Kill dead code

StreamOutput: Buffer allocated by malloc being freed with delete

Thanks to adamgr for noticing this.

Player: Pop Gcode before deleting

SimpleShell: Reserve buffer before appending

Gcode: Make command const

This also kills allocations from indexing

GcodeDisplay: Use iterator instead of indexing

This avoids allocations due to _M_leak since possible_command is
mutable.

I giveth 2k back to the heap.

With commit d79d4eff87ffb5c88f19a32bf23d26ba0e52225a I initialized
_sbrk()'s heap pointer to the wrong linker symbol which ended up
taking away 2k from the heap.

Also removed redundant arm-none-eabi-size call in makefile.

Merge remote-tracking branch 'origin/edge' into edge

Switch back to ugly size output.

This works on Windows and doesn't assume *nix terminal with Perl installed.

Pin: use strtol to parse pindef strings in a simpler and more efficient fashion

ConfigValue: also accept 'y'/'yes' as a boolean

ConfigValue: use strtol/strtod instead of atoi/atof since they tell us directly if the string was a valid number, we don't have to do our own string parsing to work it out

Merge branch 'feature/static-Pins' into edge

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.

move digipot instance to CurrentControl module
make CurrentControl delete itself if not enabled
make Laser Module delete itself if not loaded

add closedir to simleshell commands that open directories

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