giga_r1: enable wifi #223
giga_r1: enable wifi #223
Conversation
| }; | ||
| }; | ||
|
|
||
| qspi_flash: &n25q128a1 { |
There was a problem hiding this comment.
I've added those partitions in #226 because we need them in every board with a file system, so I put them in a common dtsi. I also need an mbr partition at 0. Can we align on this? I can add the missing airoc_firmware partition for you.
There was a problem hiding this comment.
Sounds good, but note that Zephyr already has common QSPI partitioning upstream using qspi_flash, so we need to keep that naming to minimize changes. Will submit needed changes on your PR in a moment!
There was a problem hiding this comment.
I see that someone already added something very similar upstream, good! Note that it's missing the MBR partition though, we don't have to have it I guess, but not sure. Also, this requires updating the Zephyr fork, right? If that's not going to happen very soon, we could use the local common file for now until it gets updated.
There was a problem hiding this comment.
@KurtE, Zephyr will enter feature freeze for 4.3 this week, and we will definitely update soon. But please use PRs for actual code review, let's not have them turn into discussions. Feel free to open a separate topic for that 🙂
There was a problem hiding this comment.
I see that someone already added something very similar upstream, good! Note that it's missing the MBR partition though, we don't have to have it I guess, but not sure.
It was me 😅 I'll add it so it's more complete (I need to fix the wifi address there anyway).
Also, this requires updating the Zephyr fork, right? If that's not going to happen very soon, we could use the local common file for now until it gets updated.
We can surely import the commit, but it wasn't a clean cherry pick due to formatting changes in recent Zephyr. I only cared about the Giga for this PR, so I copied it in without thinking about your filesystem work.
There was a problem hiding this comment.
It was me 😅 I'll add it so it's more complete (I need to fix the wifi address there anyway).
We only need the MBR partition to create it if it gets erased for some reason. I just wanted to match the old cores behavior as much as possible.
We can surely import the commit, but it wasn't a clean cherry pick due to formatting changes in recent Zephyr. I only cared about the Giga for this PR, so I copied it in without thinking about your filesystem work.
That works for me. I think you also need to add/rename the qspi_flash label for all boards, right? I saw it's missing for some boards. This should work for at least all the boards we want a file system for: C33, Giga, Portenta-H7, etc...
Alternatively, we can just use this temporarily:
And when Zephyr is updated, I can just remove the duplicate partitions and include the upstream file instead. Note that the file also adds common fstab entries and disks, so it's going to be needed anyway.
|
This PR needs arduino/zephyr#4 to use |
|
@pillo79 Im' getting this error at boot: |
Working now, the issue was a broken firmware in the QSPI flash. |
|
@pillo79 can we include this commit zephyrproject-rtos/zephyr@bd01638 in our zephyr fork this is needed to make status() work |
Import the same versions of the WiFi firmware blobs as used in
the mbed core, for maximum compatibility with existing boards:
WLAN Firmware : wl0: Jan 30 2020 21:41:53 version 7.45.98.95 (r724303 CY) FWID 01-5afc8c1e
WLAN CLM : API: 12.2 Data: 9.10.39 Compiler: 1.29.4 ClmImport: 1.36.3 Creation: 2020-01-30 21:30:05
WHD VERSION : 3.3.2.25168 : v3.3.2 : GCC 12.2 : 2024-12-06 06:53:17 +0000
Signed-off-by: Luca Burelli <l.burelli@arduino.cc>
Enable all flags to get wifi working on the Giga R1 board.
Reduces flash image size by 400kB.
3 participants
Add Wi-Fi support to the Arduino Giga loader project, enabling net-related shell commands.