Skip to content

Conversation

lromeraj
Copy link

@lromeraj lromeraj commented Aug 18, 2025

This PR introduces several enhancements and fixes to the ATWINC1500 driver to improve functionality, reliability, and security. The main changes planned include (WIP):

  • Add basic OTA support for easier firmware updates.
  • Implement native networking support.
  • Improve error handling and control.
  • Fix race conditions affecting stability.
  • Add AP security options (WPA/WEP).
  • Support for secure sockets (offload).
  • Improve TCP sockets performance while receiving data.
  • Support for power-related configuration.
  • Resolve unsupported M2M_WIFI_CH_ALL errors in AP mode.
  • Automatically provide DHCP server address.

These changes aim to provide a more robust, secure, and feature-complete ATWINC1500 driver.
The hal_atmel repository must be updated to revision zephyrproject-rtos/hal_atmel#51 to ensure the HAL layer is properly updated.

Copy link

github-actions bot commented Aug 18, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_atmel zephyrproject-rtos/hal_atmel@065e57c (master) ❌ Impostor SHA: zephyrproject-rtos/hal_atmel@0465b7a zephyrproject-rtos/hal_atmel@065e57c5..0465b7ae

DNM label due to: 1 impostor SHA

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_atmel DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Aug 18, 2025
@nandojve nandojve self-assigned this Aug 18, 2025
Copy link

@nandojve
Copy link
Member

Hi @lromeraj ,

Could you rebase?

@lromeraj lromeraj force-pushed the wifi-atwinc1500-general-enhancements branch from caa4e9a to f3995a6 Compare September 18, 2025 18:32
@lromeraj
Copy link
Author

@nandojve Done

Copy link

@nandojve
Copy link
Member

nandojve commented Sep 19, 2025

Hi @lromeraj ,

I have an ATWINC1500-XPRO, what are the steps to me test these changes ?
The Adafruit WINC1500 Wifi Shield steps are enough?

Let me know what is your scenario then we can plan something to make this more accessible, like atmel_rf2xx shield.

BTW, I have a WINC1000-SD, do you think it can use same driver ?

@lromeraj
Copy link
Author

Hi @nandojve,

I have an ATWINC1500-XPRO, what are the steps for me to test these changes?
Are the Adafruit WINC1500 WiFi Shield steps enough?

Yes, that should be enough. However, it can’t be tested yet because I still need to push all the source code. It requires some modifications since I’ve implemented it as a separate driver in a private repository.

BTW, I have a WINC1000-SD, do you think it can use the same driver?

I think you mean WILC1000-SD, right? It might work, but I wouldn’t recommend using it in that case. Microchip provides a dedicated library for the ATWILC* models, which should be used instead. Supporting it would also require a separate PR for the Zephyr project as well as for the HAL to include those sources.

@nandojve
Copy link
Member

Hi @lromeraj ,

I have an ATWINC1500-XPRO, what are the steps for me to test these changes?
Are the Adafruit WINC1500 WiFi Shield steps enough?

Yes, that should be enough. However, it can’t be tested yet because I still need to push all the source code. It requires some modifications since I’ve implemented it as a separate driver in a private repository.

In this case, we need that part too. It is important that we can test these changes.

BTW, I have a WINC1000-SD, do you think it can use the same driver?

I think you mean WILC1000-SD, right? It might work, but I wouldn’t recommend using it in that case. Microchip provides a dedicated library for the ATWILC* models, which should be used instead. Supporting it would also require a separate PR for the Zephyr project as well as for the HAL to include those sources.

Ok, good to know. Let's focus only in the ATWINC1500-XPRO use case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_atmel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants