[–] [S] 2 points 3 years ago (1 child)

| (! 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

  • source
  • parent
  • context
  • [–] [S] 1 point 3 years 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

  • source
  • parent
  • context
  •  

    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