triffid / FiveD_on_Arduino

Rewrite of reprap mendel firmware
http://forums.reprap.org/read.php?147,33082
GNU General Public License v2.0
30 stars 12 forks source link

Sample config file for GEN6 boards #26

Closed sw closed 13 years ago

sw commented 13 years ago

See http://www.reprap.org/wiki/Generation_6_Electronics

I had to add handling of E_ENABLE_PIN to mendel.c. This is also used by the RAMPS config file but was not implemented.

Also I fixed an error (?) in the commented-out section of the GEN3 configuration file: For heaters, pin and port should match, right?

triffid commented 13 years ago

hi, some confusion in the heater definition- on 644 chip, PD6 uses OCR2B timer, OCR0A uses PB3, but your gen6 config tries to match PD6 with OCR0A, as if it's a 168 or 328. Which is correct?

Thanks for fixing that issue in gen3 config, technically should be a separate commit- nvm I'll sort that out for you :) Looks like they all needed fixing, ramps config was using portD and pinH !

Also, STEPS_PERMM* values expect user to take microstepping into account, please work out final value and plug that in instead of 320.0*8.0

sw commented 13 years ago

You're right. It should be DEFINE_HEATER(extruder, PORTD, PIND6, OCR2B)

For the steps per mm, I wrote 320*8 as an example with 320 full steps per mm, as in the other configuration file. What do you mean exactly?

Regards, Stephan

triffid commented 13 years ago

if 320 full steps per mm is correct * 8 microsteps per step, then put plain 2560.0 rather than 320.0_8 (which results in an integer) or 320.0_8.0

sw commented 13 years ago

Fixed the PWM register.

The steps per mm should be ok on the other hand. Unless I'm much mistaken, multiplying a float with an int causes the int to be cast to float.

Of course 320.0 is just an example. The actual value depends not on the electronics but the mechanical structure. I put in 320.0*8 to remind people to multiply whatever value they get by eight.

triffid commented 13 years ago

thanks, pushed :)