-
Notifications
You must be signed in to change notification settings - Fork 11
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
track some #79
Comments
@jlpoltrack just to confirm, you don't see any reasons why FLRC shouldn't be included in main, right? I.e. works for you? |
No concerns from me. |
great. So we'll make it permanent :) THX!! |
@olliw42 @rotorman I went through the diff and I see this line added twice: value="com.st.stm32cube.ide.mcu.gnu.managedbuild.tool.c.compiler.option.optimization.level.value.os" valueType="enumerated" But don't really understand what this does. For example, would one only need to add these lines to any .cproject file to enable debug? Thanks |
I think the procedure is NOT to directly fumble in this file, but to use the IDE's dialogs to enter the information. This might not be accurate, but like steps in chapters III & IV & A1 here https://github.com/olliw42/mLRS/blob/main/mLRS/CREATE_TARGET_INSTRUCTIONS.md#iii-create-target-ide-config, just for Debug |
I guess what one needs to do is step 10. click Debug if usb is needed step A1.2 follow exactly you guys may want to check EDIT: did check and confirmed to be correct, see below |
Would it make more sense to add the debug steps to that page itself? Then we could just link to it from the Docs / Custom Targets Page? |
well, it would make the install more complicated than it is already |
Indeed, I would also suggest to use STM32CubeIDE dialogs for this. Basically, presently the release configuration has all the necessary include paths correctly set, whereas some have not yet been added into debug configuration. The task to get debug going is matching/adding the missing parts (e.g. by just comparing the dialogs and copy/pasting the missing ones). |
I just tested it for a rx and a tx target, and can confirm that, when a Relase target was created, the steps setp 10. click Debug and if usb is needed step A1.2 follow exactly make it to compile. If it is working I can't tell, but it compiles succesfully. There are few vtable erros currently, which I will correct ASAP. |
@olliw42 Would image like this be sufficient for initial flashing diagram? I thought about adding in ST-Link hardware but there are many variations and I think wouldn't want to recommend the cheap ones. |
I would leave that totally you. I guess I would add a cheapo STLink just for the sake of teh picture looking more complete, but I think a diagram like yours is equally useful. Pl decide according to your taste :) |
all done I guess :) so closing |
list may grow/shrink
some are just ideas
out of sync:
none
ideas:
further:
The text was updated successfully, but these errors were encountered: