Skip to content
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

Enhanced t-echo file system integrity check #3818

Merged
merged 11 commits into from
May 11, 2024

Conversation

lewisxhe
Copy link
Contributor

@lewisxhe lewisxhe commented May 7, 2024

The last fix submitted at #3775 can only solve the problem of some devices.
Today I found that some devices trigger assertions the moment they open a file and cannot perform formatting operations. This PR will disable LFS assertions, allowing the application layer to perform formatting.

LOG


//\ E S H T /\ S T / C

DEBUG | ??:??:?? 0  meshtastic.txt (0 Bytes)
DEBUG | ??:??:?? 0  config.proto (85 Bytes)
DEBUG | ??:??:?? 0  module.proto (100 Bytes)
DEBUG | ??:??:?? 0  channels.proto (57 Bytes)
DEBUG | ??:??:?? 0  db.proto (7555 Bytes)
DEBUG | ??:??:?? 0  db.proto.tmp (0 Bytes)
DEBUG | ??:??:?? 0 Using analog input 4 for battery level
INFO  | ??:??:?? 0 Scanning for i2c devices...
DEBUG | ??:??:?? 0 Scanning for i2c devices on port 1
DEBUG | ??:??:?? 0 I2C device found at address 0x51
INFO  | ??:??:?? 0 PCF8563 RTC found
DEBUG | ??:??:?? 0 I2C device found at address 0x77
DEBUG | ??:??:?? 0 Wire.available() = 1
INFO  | ??:??:?? 0 BME-280 sensor found at address 0x77
INFO  | ??:??:?? 0 2 I2C devices found
DEBUG | ??:??:?? 0 acc_info = 0
DEBUG | ??:??:?? 0 found i2c sensor meshtastic_TelemetrySensorType_BME280
INFO  | ??:??:?? 0 Meshtastic hwvendor=7, swver=2.3.8.102a20d2
DEBUG | ??:??:?? 0 Reset reason: 0x0
DEBUG | ??:??:?? 0 Setting random seed 3596778225
INFO  | ??:??:?? 0 Initializing NodeDB
INFO  | ??:??:?? 0 Loading /prefs/db.proto
ERROR | ??:??:?? 0 assert failed C:\Users\Lewis\.platformio\packages\framework-arduinoadafruitnrf52\libraries\Adafruit_LittleFS\src\littlefs\lfs.c: 1144, lfs_ctz_find, test=head

variants/t-echo/platformio.ini Outdated Show resolved Hide resolved
src/FSCommon.cpp Show resolved Hide resolved
@thebentern thebentern requested a review from caveman99 May 8, 2024 11:51
@caveman99 caveman99 merged commit 6ee995e into meshtastic:master May 11, 2024
70 checks passed
@meshtastic-bot
Copy link

This pull request has been mentioned on Meshtastic. There might be relevant details there:

https://meshtastic.discourse.group/t/t-echo-instability-seems-resolved-with-firmware-2-3-10/13033/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants