dragino / TrackerD

For Dragino TrackerD, which support GPS+WiFi+BLE+TEMP+HUM+ motion
20 stars 11 forks source link

Enable/Disable LED (LON) is always set to ON state #11

Open Martin-100 opened 1 year ago

Martin-100 commented 1 year ago

Common.cpp -> void SYS::config_Read(void) ->contains on line 618-622:

//lon lon = DATA.readUChar(addr1);
addr1 += sizeof(unsigned char);
if(lon == 0) lon = 1;

So writing a 0 to disable the leds results in turning them back on again?

mkwillis commented 1 year ago

Can this issue be fixed by editing the code?.... Its a highly annoying LED

namejinnameyuan commented 1 year ago

Yes, you can compile the code to change it. I changed it in v1.4.4. Before, 1 was on and 2 was off.

P-etr commented 1 year ago

Hi, my tracker has FW 1.4.4. and AT+LON keeps returning to 1. So I can't turn off the LED. Is it possible to somehow turn off the LED now? Will this be fixed in future versions?

mkwillis commented 1 year ago

Yes LED settings DO NOT stick... We need a fix pls

namejinnameyuan commented 1 year ago

Hi I will release a new version in the next few days, fix problems, and add features will be described

------------------ 原始邮件 ------------------ 发件人: "dragino/TrackerD" @.>; 发送时间: 2023年4月23日(星期天) 晚上8:04 @.>; @.**@.>; 主题: Re: [dragino/TrackerD] Enable/Disable LED (LON) is always set to ON state (Issue #11)

Hi, my tracker has FW 1.4.4. and AT+LON keeps returning to 1. So I can't turn off the LED. Is it possible to somehow turn off the LED now? Will this be fixed in future versions?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

namejinnameyuan commented 1 year ago

Hi I will release a new version in the next few days, fix problems, and add features will be described

------------------ 原始邮件 ------------------ 发件人: "dragino/TrackerD" @.>; 发送时间: 2023年4月23日(星期天) 晚上9:18 @.>; @.**@.>; 主题: Re: [dragino/TrackerD] Enable/Disable LED (LON) is always set to ON state (Issue #11)

Yes LED settings DO NOT stick... We need a fix pls

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>