-
-
Notifications
You must be signed in to change notification settings - Fork 768
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
Initial merge of Wi-Fi support code for ctxLink. #1927
base: main
Are you sure you want to change the base?
Conversation
ed23bcf
to
66d60b2
Compare
66d60b2
to
b7da541
Compare
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.
Done an initial review, there are still several more files we need to look at but this gives some easy first items to deal with.
18e79dd
to
5a7a3d1
Compare
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.
With this review, we have only the WiFi server implementation to review and we think we've covered most of the rest. Some of the comments likely apply across multiple files, so if you see, eg, u_int32_t
in another file to the one commented in, then our uint32_t
comment applies to it as well.
bool is_swo_trace_client_connected(void); | ||
void send_swo_trace_data(uint8_t *lpBuffer, uint8_t length); | ||
|
||
void wifi_gdb_putchar(unsigned char ch, int flush); |
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.
We believe that flush
should be bool
here - you'll need to #include <stdbool.h>
at the top of the file for that but it should otherwise be good to go.
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.
This function is based up the common/stm32/gdb_if.c file, which as:
void gdb_if_putchar(const char c, const int flush)
613870d
to
6bfc7f9
Compare
ba7c432
to
761cb41
Compare
This is in preparation for updating the GDB I/O functions to handle both USB and WiFi connections.
Other compilation issues address in this commit.
Removed the winc1500 library sources, kept the API stub file in the ctxlink directory. It was not part of the winc1500 library.
Also fix local variable "count" to match sscanf return type.
…attribute. Also, simplify the access function.
Also, add define for RX/TX buffer sizes for clarity
…ariable declaration to file beginning.
Detailed description
This PR brings an early version of Wi-Fi support for ctxLink. Initially, it attempts to update the souce code to fit more closely with the style of BMF source code as definied by Clang-Tidy.
The built firmware, while offering initiall Wi-Fi connectivity and operation, is not full featured. The intent of this PR is to allow outstanding features in smaller PRs, thus reducing the burden on the author and the reviewer.
Your checklist for this pull request
Closing issues
None