-
Notifications
You must be signed in to change notification settings - Fork 973
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
STM32F1 merge other (BluePill and Maple Mini) #61
Conversation
93d8db4
to
addd3ea
Compare
addd3ea
to
50d0ca3
Compare
50d0ca3
to
17fe6ab
Compare
40275c7
to
c7678b9
Compare
So I'm looking at this new branch and it doesn't seem like any of the USB issues I mentioned in #45 have been addressed. Is this something that is being looked at? |
BTW: The default upload method of "Maple bootloader" is probably a bad pick for the Other boards. Most people are using a bluepill according to the latest Poll on stm32duino. Even those who are using a maple mini typically update the bootloader to the latest one because it is smaller in flash size and works better in general. |
@RickKimball, it's ongoing, they are lot of work/review around USB and this need some time. As you can see, I've rebased regularly this PR. |
0216236
to
f85ebb5
Compare
Upload method Bootloader, Serial and BMP added. USBSerial class added because is required by bootloader. Signed-off-by: fpr <fabien.perroquin@wi6labs.com>
Signed-off-by: fpr <fabien.perroquin@wi6labs.com>
Signed-off-by: fpr <fabien.perroquin@wi6labs.com>
Signed-off-by: fpr <fabien.perroquin@wi6labs.com>
Signed-off-by: fpr <fabien.perroquin@wi6labs.com>
Signed-off-by: fpr <fabien.perroquin@wi6labs.com>
Signed-off-by: Frederic.Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic.Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic.Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic.Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic.Pillon <frederic.pillon@st.com>
f85ebb5
to
cf38a1f
Compare
Other.menu.upload_method.STLinkMethod=STLink | ||
Other.menu.upload_method.STLinkMethod.upload.protocol=STLink | ||
Other.menu.upload_method.STLinkMethod.upload.tool=stlink_upload | ||
Other.menu.upload_method.STLinkMethod.build.upload_flags=-DSERIAL_USB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move STLink method before DFUUploadMethod. STLink will be the default Upload method.
#define SS3 10 | ||
#define MOSI 6 | ||
#define MISO 5 | ||
#define SCK 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SPI configuration should be updated.
@cparata The official merge of Bluepill (and all the classic board with F103C8 / CB) is of vital importance given the commercial success of these boards, all the users of Roger's core have a Bluepill / Maplemini at home Official support for Bluepill has been mentioned for some time in the core homepage, but never merged. |
I have successfully tested this with maple mini + STLink and I would love to see this merged. |
I'm currently thinking to add BP and MM without USB support. |
… works on a maple mini (STLink upload, UART2).
… works on a maple mini (STLink upload, UART2).
I will provide the light version end of the week. |
Feel free to copy&paste from my temporary BP&MM rebase branch. I think I was too heavy handed with the USB stuff so I won't make a PR with it. |
Second part of the Arduino_Core_STM32F1 merge. #20 --> now covered by #213 except for USB part
Include above variant from #45
Need some review around USB (based on contributor feedback)
Need review upload menu (avoid maple one for BP)Covered by #213...