Changes for RPI5 -- somewhat functional? #3
Conversation
There was a problem hiding this comment.
It feels like this should be achieved via overlay_map.dtb with the common bits being targeted at any platform and the chip specific UART separately.
I tested breaking out the fragments it seemed to work as well as the whole file but the compatible directive doesn't seem to exclude application.
My current workaround is using dtoverlay wlmio-pi5
There was a problem hiding this comment.
I am going to remove overlays entirely for this package. overlay_map.dtb is a hack. Instead we are going to ship a wlmio.txt to be included with include wlmio.txt in config.txt that will handle all of the platform differences in an easy way for users.
There is a symlink to /boot/firmware/overlays at /boot/overlays so I think we will rely on this for now to support bullseye.
| executable('wl_factoryreset', 'factoryreset.c', dependencies: [wlmio_dep], install: true) | ||
| executable('wl_fwupdate', 'fwupdate.c', dependencies: [canard_dep, libgpiod_dep], install: true) | ||
| executable('wl_infodump', 'infodump.c', dependencies: [wlmio_dep], install: true) | ||
| executable('wl_monitor', 'monitor.c', dependencies: [wlmio_dep], install: true) | ||
| executable('wl_regtool', 'regtool.c', dependencies: [wlmio_dep], install: true) | ||
| executable('wl_store', 'store.c', dependencies: [wlmio_dep], install: true) |
There was a problem hiding this comment.
These names specifically are problematic in their generic nature the better alternative might be changing the prefix to /usr/libexec/widgetlords/ or /opt/widgetlords/
There was a problem hiding this comment.
Have you seen any conflicts in the wild?
2 participants
Do not merge this pull, it is here to make discussion easier