hiveeyes / terkin-datalogger

Datalogger for MicroPython and CPython.
https://terkin.org
GNU Affero General Public License v3.0
60 stars 28 forks source link

Add MCU_DEVICE to presets-example.mk #77

Open ClemensGruber opened 4 years ago

ClemensGruber commented 4 years ago

In case you update the PyCom firmware via the sandbox tool and make install-pycom-firmware you have to specify MCU_DEVICE

[ADVICE] Please adjust the "MCU_DEVICE" environment variable like export MCU_DEVICE=FiPy

Is there a reason why MCU_DEVICE is currently not in the presets-example.mk? E.g. conflicts in case there is a non-PyCom device connected or other circumstances?

If this is not the case please consider to merge this pull request. ;-)

ClemensGruber commented 4 years ago

Seems that adding MCU_DEVICE is not sufficient had still to call export MCU_DEVICE=FiPy

And even with this it is not working on WSL

root@XPS13-CGruber:/home/cgruber/hiveeyes/sources/hiveeyes-micropython-firmware# make install-pycom-firmware
[OK]      Found Pycom Firmware Updater at "/mnt/c/Program\ Files\ \(x86\)/Pycom/Pycom\ Firmware\ Update/pycom-fwtool-cli.exe"
make[1]: Entering directory '/home/cgruber/hiveeyes/sources/hiveeyes-micropython-firmware'
[CONFIRM] Install Pycom firmware "FiPy-1.20.2.rc6-0.10.1-vanilla-squirrel.tar.gz" on the device connected to "  COM10"make[2]: Entering directory '/home/cgruber/hiveeyes/sources/hiveeyes-micropython-firmware'
 [y/n] y
make[2]: Leaving directory '/home/cgruber/hiveeyes/sources/hiveeyes-micropython-firmware'
make[1]: Leaving directory '/home/cgruber/hiveeyes/sources/hiveeyes-micropython-firmware'
[INFO]    Downloading firmware from "https://packages.hiveeyes.org/hiveeyes/foss/pycom/vanilla/FiPy-1.20.2.rc6-0.10.1-vanilla-squirrel.tar.gz"
Will not apply HSTS. The HSTS database must be a regular and non-world-writable file.
ERROR: could not open HSTS store at '/root/.wget-hsts'. HSTS will be disabled.
File './dist-firmwares/FiPy-1.20.2.rc6-0.10.1-vanilla-squirrel.tar.gz' already there; not retrieving.

[INFO]    Installing firmware "FiPy-1.20.2.rc6-0.10.1-vanilla-squirrel.tar.gz"
/mnt/c/Program\ Files\ \(x86\)/Pycom/Pycom\ Firmware\ Update/pycom-fwtool-cli.exe --verbose --port "  COM10" flash --tar "dist-firmwares/FiPy-1.20.2.rc6-0.10.1-vanilla-squirrel.tar.gz"
Invalid serial port   COM10! Use list command to show valid ports.
amotl commented 4 years ago

--port " COM10"

Invalid serial port " COM10"! Use list command to show valid ports.

ClemensGruber commented 4 years ago

The number is correct, in presets.mk

MCU_PORT ?= /dev/ttyS10

and this is working e.g. with make repl

root@XPS13-CGruber:/home/cgruber/hiveeyes/sources/hiveeyes-micropython-firmware# make repl
Device port: usb => /dev/ttyS10
.venv3/bin/rshell --port /dev/ttyS10 --user micro --password python --buffer-size 2048 repl
Using buffer-size of 2048

even if I change MCU_PORT in presets.mk this is respected, but it seems to work not correctly with make install-pycom-firmware in combination with WSL.

I think we should forget it, we can use the Windows shell for this task or the GUI tool PyCom Firmware Updater. Just tried it out to have all things "at one place", but no problem to make it on different places.