Skip to content

Add predefined AUTO_FAN_PINs to pins files#5072

Merged
thinkyhead merged 8 commits intoMarlinFirmware:RCBugFixfrom
thinkyhead:rc_preset_auto_fans
Oct 27, 2016
Merged

Add predefined AUTO_FAN_PINs to pins files#5072
thinkyhead merged 8 commits intoMarlinFirmware:RCBugFixfrom
thinkyhead:rc_preset_auto_fans

Conversation

@thinkyhead
Copy link
Member

@thinkyhead thinkyhead commented Oct 23, 2016

…well, some pins files.

As suggested by #5011 (comment)

  • Fix broken _GET_TIMER macro in fastio.h
  • To ease transition, rename EXTRUDER_n_AUTO_FAN_PIN to En_AUTO_FAN_PIN
  • Sanity check for the old setting
  • Add ORIG_En_AUTO_FAN_PIN for use by pins files as the suggested auto fan pins
  • Update commentary for the auto pins options in Configuration_adv.h
  • Sanity check: If the fan speed is under 255 the selected pins must be PWM-able
  • Supply default auto fan pins for some boards — based on example configurations
  • Use EXTRUDER_AUTO_FAN_SPEED only for PWM-able pins, else use 255
  • Constrain auto fan pins, limited by HOTENDS
  • For Cartesio fix some issues: Allow a single status LED. Allow keypad with CartesioUI.

@thinkyhead thinkyhead merged commit c34dbfb into MarlinFirmware:RCBugFix Oct 27, 2016
@thinkyhead thinkyhead deleted the rc_preset_auto_fans branch October 27, 2016 22:12
@thinkyhead thinkyhead mentioned this pull request Nov 1, 2016
thinkyhead added a commit that referenced this pull request Dec 1, 2016
Fix for the PR #5072 (Add predefined AUTO_FAN_PINs to pins files), etc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant