search

albsod / pinetime-hypnos

Zephyr firmware for the nRF52832 PineTime smartwatch
Apache License 2.0
73 stars 21 forks source link

Update Zephyr #19

Closed endian-albin closed 4 years ago

endian-albin commented 4 years ago

The west manifest file, as inherited from najnesnaj, uses Nordic's zephyr fork at commit 3c75196149f876b4d40e490607d80591e60d467f from Jan 28 2020. I've tried to choose the latest, or even a somewhat newer commit in this source tree, but then hypnos won't compile because of an incorrect configuration of the touch sensor driver.

It's good to stay close to upstream in general, but I think we need this, to fix a bluetooth bug.

endian-albin commented 4 years ago

This is the exact error that I get.

/home/albin/projekt/exjobb/pinetime/drivers/sensor/cst816s/Kconfig:92) is assigned in a
configuration file, but is not directly user-configurable (has no prompt). It gets its value
indirectly from other symbols. See
http://docs.zephyrproject.org/latest/reference/kconfig/CONFIG_CST816S_TRIGGER.html and/or look up
CST816S_TRIGGER in the menuconfig/guiconfig interface. The Application Development Primer, Setting
Configuration Values, and Kconfig - Tips and Best Practices sections of the manual might be helpful
too.

When i tried to fix that particular error, then I got other, similar errors in completely different places.

endian-albin commented 4 years ago

I've had some progress in updating to the latest commit (c0e092a07d17820120f72b23a817dc895a5ca34e) in Nordics Zephyr fork. There are many API changes, patches that need to be applied and changes to be made in e.g. the nrf dependency, but it's doable.

Right now, I can build a working image, but with a lot of warnings. To be continued...

endian-albin commented 4 years ago

Now all the warnings are gone. I've pushed an update-zephyr branch that contains API changes.

endian-albin commented 4 years ago

I also had to make changes to the pinetime.dts to make it work.