GitLab

Thus effectively double PWM frequency to about 4 kHz.

With 1/256 PWM this is way too fast for the AMC7135. :(

It doesn't make lot of sense as it is not much lower than the normal
low, but requires 14 bytes of flash. It also conflicted with the strobe
mode. A similar mode can be achieved by setting "start on low".

Looks and feels great, but requires 20 bytes of flash!

We can go down to 1 with our slow PWM.

This is simpler than using the PWM overflow handler, prepares for power
saving and allows us to use different PWM frequencies without other
modifications.

It is now also used for moonlight mode.

Currently only 1s and 2s interval

Extends the strobe mode. If options.moonlight is set and options.strobe
is not, then the light will turn on alway in moonlight and after a
single tap it will continue with the default mode.

At least with the development board there is no visibly huge difference
in output, so this should be considered WIP.

This saves a lot of flash.

The previous level was too bright in dark environment. The new level is
currently shared with other uses and thus saves two bytes of flash.

This saves 12 bytes of flash.

Quickly measured with bench power supply.

This saves 8 byte.

If we assume that the config case in the switch statement is reached
before the user can tap a second time [1] there is no need to handle
overflow of fast_presses in a separate if statement. Instead it is
enough to set fast_presses to the current value minus one so that when a
possible next press of the power switch increments the counter again
this results in the same case.

This saves 6 bytes of flash.

[1] Which is unlikely because the execution time until this point is
much shorter than human repetitive finger movement, so only button
bouncing can be an issue and that would require other measures

If users want a strobe mode, they usually want it for defence purposes.
Thus it should be possible to enter it as fast as possible. If strobe is
enabled (first option if STROBE is compiled) the flashlight always
starts in a strobe mode. Then a single press of the power switch enters
the usual UI.

TIMSK0 is outside of lower 32 I/O registers and thus cannot be set via
sbi. This saves 4 bytes of flash.

Set a flag in the interrupt handler instead of checking the current tick
value.

Requires additional 14 bytes of flash...

Now we use registers in the function call area. But we don't use any
functions with many arguments, so this is not a problem and saves 56
bytes.

Costs 8 bytes of flash.

Check fast_presses and state only once. Allows more deterministic
transitions, but introduces a goto and increases flash size by 14 bytes.

Move common actions out of mode specific handler. Removes redundant
code, but might not provide the same level of flexibility as before and
might reduce readability.

Saves 2 bytes of flash.

Just to make sure. We have enough free RAM at the moment.

toggle_options() preserves the value of options if it returns. The
compiler did not optimize the intermediate variable because it doesn't
know that fact. This saves 4 bytes.

Also make some tiny improvements to the Makefile related to fuses.