Protiron

joined 1 year ago
[–] Protiron 2 points 1 year ago

Hi thank's for your help. I rollback two custom modification ans it's working again :

I will investigate which of this two changes cut my turbo off

[–] Protiron 2 points 1 year ago (1 children)

| (! channel_has_args(cfg.channel_mode)

I test that asap. I already try to comment it and it's work fine :

//if (! arg) {  // first frame only, to allow thermal regulation to work
    uint8_t tl = style_2c ? 150 : turbo_level;
    set_level_and_therm_target(tl);
//}

But I am surprised that the functionality appears in the diagram and that no one remarks that it does not work (3H without Multi channel or 4H with) https://lemmy.world/post/1038159

[–] Protiron 2 points 1 year ago

To test it I flash my flashlight ... :(

[–] Protiron 1 points 1 year ago (4 children)

Also in following the code style the actual statement would also need to include an IFDEF for multi-channel-config but maybe not necessary for you test locally?

So Mementary Turbo 3H work for you? If you don't own multi channel, MT should be 3H from ON. I think l423 be identical for simple or multi channel.

I can't find where arg should be equals to 0... steady_state always call with arg <> 0

[–] Protiron 2 points 1 year ago (6 children)

Thank you, I fix it. Does Momentary Turbo 3H/4H work four you?

18
submitted 1 year ago* (last edited 1 year ago) by Protiron to c/flashlight
 

Hi Anduril user's !

I noticed that the momentary turbo does not work for 4H from the "ON" state (3H for those who do not have tint ramping).

To fix it, I have to comment out line 423:

if (! arg) { // first frame only, to allow thermal regulation to work

[https://bazaar.launchpad.net/~toykeeper/flashlight-firmware/multi-channel/view/head:/ToyKeeper/spaghetti-monster/anduril/ramp-mode.c#L423

Do you have the same behavior?

Thank you and good day