Skip to content
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

Update SPL packages #9

Open
maxgerhardt opened this issue Sep 16, 2021 · 3 comments
Open

Update SPL packages #9

maxgerhardt opened this issue Sep 16, 2021 · 3 comments

Comments

@maxgerhardt
Copy link
Member

As noticed in #8 (comment), we are one minotr version behind with the GD32F30x firmware library.

We should look for updates at, pull them into https://github.com/CommunityGD32Cores/gigadevice-firmware-and-docs for documentation and then pull them in here for an update.

We might have to do bugfixes again like cplusplus --> __cplusplus, re-definition of bool and I also remember one case in a CMSIS device header where the a macro should have been lower instead of uppercase.

All these fixes / patches should be documented too and maybe create a .patch file from them so that we can update packages more easily.

@hisaha
Copy link

hisaha commented May 6, 2022

I have tried update f10x to v2.2.2 with below steps,

It seems to work. Is this process correct ?

Thanks

@hisaha
Copy link

hisaha commented May 9, 2022

Thanks for your merging. I am surprised the progress is fast !

@maxgerhardt
Copy link
Member Author

maxgerhardt commented May 9, 2022

Ah yes sorry I wanted to notify for after merge, I merged your stuff since it looked all good.

I'm always keeping an eye on the modifications because we ourselfs did some mods for things that needed to be fixed but Dialog hasn't yet fixed. For example, they define their own boolean type with FALSE and TRUE, we exchanged that for #include <stdbool.h>. They continously misspell #ifdef cplusplus when it should be #ifdef __cplusplus, which breaks using C++ with SPL. But your merge did not modify any of this, so our fixes stayed (good).

At some point I have to think about refactoring those into .patch files or pinging GigaDevice devs again to fix the bugs upstream.

But, to answer your original question: Yes, that update process is perfectly valid. Basically we pull out the SPL files from the original download and put them in the folder structure we have here.

I will also have a look at updating / integrating the USB library at a later time. We already have PlatformIO builder scripts for other USB stacks and they're all similiar (2 basic types I think).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants