-
-
Notifications
You must be signed in to change notification settings - Fork 8.2k
Add some definition files for waveshare board. #9085
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
base: master
Are you sure you want to change the base?
Conversation
Two files not related to the new RP2040 boards sneaked into that commit. These are from a different already merged PR. So please remove them from your PR. |
|
4689d1e
to
ef518cb
Compare
I try to understand the situation wrt Waveshare Zero (have a couple of them). This board is mentioned in this ticket as well as in #10314 and #9321. Yet, there is no download option for Waveshare boards on the download page which may be related to the status OPEN. |
Although I don't quite understand, for some reason, it hasn't been merged yet, so I've been waiting.
|
Thank you for your quick response. As you appear to be the driving force behind this pull request, will you then try to break the "waiting for some unknown" situation ? |
Maybe MicroPython doesn't consider this pull request to be important, as it doesn't concern fixing any critical bugs or urgent issues. It appears that MicroPython merges pull requests in a sequential manner, and other reasons for non-merging might need to be addressed by MicroPython management. I am an embedded engineer and this is my profession.I like it.so I call myself EngineerWill.
|
This would be really great, I've had issues getting the waveshare RP2040-Zero boards to work since none of the firmware from the micropython website seem to work. Took me a while to find working firmware (I initially thought the boards were just broken), from the manufacturer website, which, however, seems pretty outdated and also needs unzipping for some reason https://www.waveshare.com/w/upload/b/b4/Rp2-pico-w-20230219-unstable-v1.19.1.uf2.zip Is there perhaps already something from the micropython website I can use? Closest I found was https://micropython.org/download/rp2-pico-w//resources/firmware/rp2-pico-w-20230426-v1.20.0.uf2 , which fails with:
|
Am puzzled by what you report on your experience with MP on Waveshare Zero. I am using the regular Pico firmware on the Waveshare board (now using: MicroPython v1.20.0-24-g867e4dd3d on 2023-04-27; Raspberry Pi Pico with RP2040) and that works (however, GPIO29 does not and I have not troubled to solder connections at the bottom-side as I can run my project with the pins availabe on the top side). So I cannot state I know for sure everything works. |
Hello, according to the log you provided, it seems that you provided the RP2040-Zero factory program which is only used to test the RP2040-Zero hardware and needs to be used with the corresponding testing tool. However, based on your description, it seems that you were unable to successfully flash the mpy firmware. Please note that RP2040-Zero is compatible with pico firmware, and you can obtain the corresponding firmware file on the Raspberry Pi official website.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #9085 +/- ##
==========================================
- Coverage 98.39% 98.37% -0.03%
==========================================
Files 158 156 -2
Lines 20965 20424 -541
==========================================
- Hits 20629 20092 -537
+ Misses 336 332 -4 ☔ View full report in Codecov by Sentry. |
19b0858
to
0de9fa0
Compare
// url : https://www.waveshare.com/product/rp2040-lcd-0.96.htm | ||
// wiki : https://www.waveshare.com/wiki/RP2040-LCD-0.96 | ||
// Board and hardware specific configuration | ||
#define MICROPY_HW_BOARD_NAME "Waveshar RP2040-LCD-0.96" |
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.
typo in waveshare?
"USB-C" | ||
], | ||
"images": [ | ||
"rp2040-lcd-0.96-1.jpg" |
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 image is not provided in this PR?
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.
Board images are intended to be submitted as a separate PR on the micropython-media repo, they're not included in this main repo to keep the size down.
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.
image is provided in this PR(micropython/micropython-media#62)
"rp2040-plus-1.jpg" | ||
], | ||
"mcu": "rp2040", | ||
"product": "Waveshare RP2040=Plus", |
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.
typo in product name?
// url : https://www.waveshare.com/product/rp2040-plus.htm | ||
// wiki : https://www.waveshare.com/wiki/RP2040-Plus | ||
// Board and hardware specific configuration | ||
#define MICROPY_HW_BOARD_NAME "Waveshar RP2040-Plus" |
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.
typo in waveshare again?
#define MICROPY_HW_USB_VID (0x2E8A) | ||
#define MICROPY_HW_USB_PID (0x1020) | ||
|
||
// user led gpio25 |
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.
should this go in a pins.csv? there's no point in just having a dangling comment
// url : https://www.waveshare.com/product/rp2040-plus.htm | ||
// wiki : https://www.waveshare.com/wiki/RP2040-Plus | ||
// Board and hardware specific configuration | ||
#define MICROPY_HW_BOARD_NAME "Waveshar RP2040-Plus" |
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.
same typo again
#define MICROPY_HW_USB_VID (0x2E8A) | ||
#define MICROPY_HW_USB_PID (0x1020) | ||
|
||
// user led gpio25 |
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.
same comment on dangling comment vs pins.csv
// url : https://www.waveshare.com/product/rp2040-zero.htm | ||
// wiki : http://www.waveshare.com/wiki/RP2040-Zero | ||
// Board and hardware specific configuration | ||
#define MICROPY_HW_BOARD_NAME "Waveshar RP2040-Zero" |
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.
same...
56d4194
to
20f3bb6
Compare
Signed-off-by: EngWill <646689853@qq.com>
20f3bb6
to
3946cf5
Compare
Some recent discussions with WaveShare support pointed me at this merge request, is there a reason it has not been done? My (mis-)adventures with changing the XTALs on board for more precision parts will required a custom build, should the XTAL freq change. I have proved this on Ri-P boards, but the WaveShare boards are tempting me with a good price and include variant with an on-board LCD. |
Ping. Is there a reason this PR is not yet implemented? |
I don’t know why this PR hasn’t been processed yet |
Actually, the two "plus" boards should now be implemented as two variants on the one board profile. To be fair the variant system didn't exist when you first put this merge request together. An example of variant definitions can be seen in https://github.com/micropython/micropython/pull/12281/files |
I referred to the definition of Do I need to modify it? If it needs to be modified, can you provide a reference case, because I think other rp2040 boards still have this problem.(like 'PIMORONI_PICOLIPO') |
Ah interesting, yeah that board is an apt comparison. I'm not sure how the maintainers feel about the variants as a must, I thought existing cases like the pimoroni ones had been "cleaned up" already. |
Code size report:
|
Add some Waveshare board descriptions,like: