anduril: needs a factory reset function

[Thomas Arthofer]

Currently, there is no way of resetting the light to factory defaults.

Idea: By holding down on boot, the light enters a "self destruct" sequence

[Thomas Arthofer (arthofer-thomas)]

By holding down the eSwitch on boot, the light will:

- Start a triangle ramp with increasing frequency
- Upon reaching the max, calibrate to ROOM_TEMPERATURE
- Write the defaults to EEPROM
- Go to off state

[Selene ToyKeeper (toykeeper)]

Oops, I didn't see this when I was on a bug-filing spree, and I made a duplicate. Copying the description of that bug here:

-----

People are sometimes afraid that they will mess up their settings and not be able to get back to sane values. Both Bistro and Narsil had factory reset functions, and it gave people sort of a safety net. I avoided including one for Anduril, because I feel its interface is simple enough it shouldn't need one, but maybe it should be included after all.

Part of the reason for this is to make it easier for manufacturers to calibrate their products at the factory. The attiny85's thermal sensor varies widely from unit to unit, so for thermal regulation it's really important to calibrate the sensor.

What I have in mind is something like what I use for the "self-destruct" function on my lightsaber:

  - Disconnect power. (loosen battery tube or something)
  - Hold the e-switch.
  - Connect power.
  - Continue holding the e-switch for a few seconds, while the light makes an escalating warning buzz.
  - If the user lets go of the e-switch: abort, fade the LED, and boot normally.
  - Otherwise, when the timeout is reached...
    - Do an automatic thermal calibration, based on the assumption that the light is currently at room temperature.
    - Save the config values (which are default because it hasn't loaded eeprom yet).
    - Make a bright flash to indicate the reset is complete, then sparkle back down to off.

Maybe it doesn't need to be as fancy as the lightsaber version, but still. This would make it easy for people to get sane values.

[Selene ToyKeeper (toykeeper)]

Assigning to me, since a patch is available and waiting on me to test/merge it.

[Thomas Arthofer (arthofer-thomas)]

Strange Idea:

Make ROOM_TEMPERATURE a value in flash instead of a define.
After the build, export the address of the const (and maybe some others) to another file (simmilar to a2l in vehicle software)

e.g.:

ROOM_TEMPERATURE - uint8 - 0xDEADBEEF
THERM_CEIL - uint8 - 0xDECAFBAD

Provide a script which takes that "fsm-a2l" file and allows to patch those values after the build.

Reason:
Programming station reads the temperature (environment or IR-temperature of the head), patches the hex-file and flashs the values.

[Selene ToyKeeper (toykeeper)]

I'm not sure anyone has the equipment or skill to implement that solution in a production environment, and it still doesn't resolve the issue of the attiny sensor being uncalibrated. Due to communication and translation difficulties, it is difficult to get manufacturers to use the correct fuse values, much less get them to patch a hex file.

However, what it could do is, at boot time, if the thermal calibration value isn't set, it could do a quick calibration. Normally, this would happen the first time a customer puts a battery in. It's like the manual factory reset, except it happens automatically on first boot.

[Thomas Arthofer (arthofer-thomas)]

This patch needs a small change. It currently passed the values ROOM_TEMP (20) and DEFAULT_THERM_CEIL (45) to the save function.

Sadly, the save function saves 30+passed value to the EEPROM.

In the current case, 75°C is set as the thermal ceiling if this is implemented that way.

[Selene ToyKeeper (toykeeper)]

That was odd. Looks like a bot of some sort.

[Selene ToyKeeper (toykeeper)]

BTW, I gave this a quick try last month, hoping to get a reset function in time for the FW3A... but things didn't go well. I went back to a solid and well-tested build for that, and haven't really gotten back to the reset function yet. But it should still happen.

[Selene ToyKeeper (toykeeper)]

I forgot to update this when I fixed it, but the new feature was released almost a month ago.