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

Updated path for FreeRTOS example - FreeRTOS-Plus-TCP #574

Merged
merged 1 commit into from
Apr 13, 2023

Conversation

matty0005
Copy link
Contributor

I know that perhaps a lot of additional work is needed to get networking working on this processor, so this prehaps isn't the most useful fix.

Nevertheless, FreeRTOS have put all the source files (for only the TCP stuff) under $(FREERTOS_HOME)/FreeRTOS-Plus/Source/FreeRTOS-Plus-TCP/source/ and not $(FREERTOS_HOME)/FreeRTOS-Plus/Source/FreeRTOS-Plus-TCP/ as it currently is in the makefile in the FreeRTOS example.

This PR just updates those routes.

Users would also need to include a FreeRTOSIPConfig.h file as well. I have left that out for now, but I can add a note about that in the readme for the example. I can also provide a default one too - just wanted to see everyone's thoughts first.

@matty0005 matty0005 changed the title Updated path for FreeRTOS-Plus-TCP Updated path for FreeRTOS example - FreeRTOS-Plus-TCP Apr 11, 2023
@emb4fun
Copy link
Collaborator

emb4fun commented Apr 11, 2023

Hello matty,

do you still have a MAC for the PHY or what kind of ethernet hw do you want to use?

Regards,
Michael

@matty0005
Copy link
Contributor Author

Hi Michael,

I'm currently in the process of creating the MAC (for an RMII PHY) as part of my undergraduate thesis - I'll probably release the design once I'm done.

Regards,
Matt

@stnolting stnolting self-assigned this Apr 13, 2023
@stnolting stnolting added the SW software-related label Apr 13, 2023
@stnolting
Copy link
Owner

This PR just updates those routes.

Thanks for updating!

Users would also need to include a FreeRTOSIPConfig.h file as well. I have left that out for now, but I can add a note about that in the readme for the example. I can also provide a default one too - just wanted to see everyone's thoughts first.

That would be great! 😉

@stnolting stnolting merged commit 88ef812 into stnolting:main Apr 13, 2023
@matty0005
Copy link
Contributor Author

Great, I'll create another PR later once I have things working for the FreeRTOSIPConfig.h and can add some notes where appropriate.

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

Successfully merging this pull request may close these issues.

None yet

3 participants