Marlin
Marlin copied to clipboard
✨Mellow Fly E3 Pro V3 (STM32F407VG)
Description
Based on the Fly-E3-v2 mainboard that was merged into Marlin, this is for the Fly-E3-Pro-V3
Requirements
A Fly-E3-Pro-V3
Benefits
Adds support for the Fly-E3-Pro-V3 board
Configurations
Related Issues
None
Since this is a duplicate PR, it'd be better to send PRs to https://github.com/MarlinFirmware/Marlin/pull/25894 / https://github.com/IoakeimSogiakas/Marlin/tree/FLY_RRF_MARLIN
i've sent a message to the author but not heard anything back. can we not just merge this one and close the other PR?
can we not just merge this one and close the other PR?
That’s not how it generally works. Users collaborate via a single PR/fork, not open multiple PRs.
PR https://github.com/MarlinFirmware/Marlin/pull/25894 closed, superseded with this on of this one
I guess that settles that.
original pr was not finished and seems to be abandoned. this is a step forward
@ellensp thanks
As far as I'm concerned this is ready to merge
anything else i need to do with this?
be patent, main developer is taking a break for an unknown period of time
@jaysuk can you update this to work with the latest bugfix-2.1.x and push new changes? If so we should be able to do a final review of this and get it in soon.
@sjasonsmith updated to the latest. only had to change the boards file
I also just pushed 55a109e to fix a few small things in pins_MELLOW_FLY_E3_PRO_V3.h
.
thanks, didn't even think to check those. Once this is merged I will move on to the next board. I still have the Fly-CDYv3 and Super8 boards to do in the F4 range. Then its on to the H7 ones
i checked all the pins against the pinouts of the modules etc. i maintain all the docs for the RRF port so i know the boards "fairly" well
PR has been brought up to date & I've pushed some more updates to pins_MELLOW_FLY_E3_PRO_V3.h
. See my question above about flash drive support.
I just discovered this note about the Mellow Fly E3 Pro V3's thermistor pull-ups:
[!IMPORTANT] The Fly-E3-Pro-v3 uses a 2k2 resistor on the thermistor inputs rather than 4k7. Please make sure you add R2200 to each M308 in your config
I ran into this while working on LDO Leviathan board support and it means that users will have select thermistor 1000 for any thermistor they want to use and will have to define all the custom thermistor 1000 parameters found in Configuration_adv.h
since Marlin assumes 4.7K as the default pull-up type for most thermistors.
I also saw the note about needing to add jumpers on Z2's output (I assume they are driven in series?), so adding a warning to the pins file would be a good idea.