Skip to content

shell: configure Standard backlight intensity per board - #2077

Merged
jplexer merged 1 commit into
coredevices:mainfrom
jplexer:codex/firm-4392-getafix-battery-saver
Sep 16, 2026
Merged

jplexer merged 1 commit into
coredevices:mainfrom
jplexer:codex/firm-4392-getafix-battery-saver

Conversation

@jplexer

@jplexer jplexer commented Sep 15, 2026

Copy link
Copy Markdown
Member

Summary

Getafix's Standard backlight preset uses 50% brightness. Add CONFIG_BACKLIGHT_STANDARD_INTENSITY (range 1–100, default 50) and set it to 25 in Getafix's shared board defconfig. The Standard preset reads this configuration directly, so both Getafix revisions use 25% for fresh settings and when Standard is selected.

Standard remains the default mode with its Standard dynamic curve. Battery Saver retains its Dim curve, and other boards retain their existing defaults. Saved preferences still take precedence; existing watches storing 50% may report Advanced until Standard is selected again.

Fixes FIRM-4392

The host test build defines the new setting as 50%, matching the Kconfig default, since it does not load firmware Kconfig.

Validation

  • Generated Kconfig for Getafix DVT and DVT2: both resolve Standard intensity to 25%.
  • Generated Kconfig for Obelix PVT, Asterix, and QEMU Gabbro: all resolve to 50%.
  • Compiled the updated prefs.c using the newly generated Getafix DVT2 configuration header and the existing build's compiler settings.
  • Recompiled both host-test variants of prefs.c with the new setting; preferences tests passed using the rebuilt object. The local activity test link was blocked by incompatible cached objects from newer source.
  • Fresh CI test run started for the correction.
  • git diff --check and commit gitlint passed.
  • No full firmware build or hardware test. The earlier build dependency step was blocked by missing rsvg-convert.

AI-assisted implementation using GPT-6 Astra.

@jplexer
jplexer marked this pull request as ready for review September 15, 2026 20:46
@jplexer
jplexer requested a review from gmarull as a code owner September 15, 2026 20:46

@gmarull gmarull left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't we modify standard instead?

@jplexer
jplexer force-pushed the codex/firm-4392-getafix-battery-saver branch from 964a851 to 3076569 Compare September 16, 2026 09:41
@jplexer jplexer changed the title shell: default Getafix backlight to Battery Saver shell: set Getafix Standard backlight brightness to 25% Sep 16, 2026
@jplexer
jplexer requested a review from gmarull September 16, 2026 09:50
Comment thread src/fw/shell/normal/prefs.c Outdated
@jplexer jplexer changed the title shell: set Getafix Standard backlight brightness to 25% shell: configure Standard backlight intensity per board Sep 16, 2026
@jplexer
jplexer force-pushed the codex/firm-4392-getafix-battery-saver branch from 3076569 to d08e405 Compare September 16, 2026 10:52
Add BACKLIGHT_STANDARD_INTENSITY with a 1-100 range and a default of 50.
Set it to 25 in the shared Getafix board configuration and use it for
the Standard preset, which also supplies fresh-device brightness.

Keep Standard as the default mode and preserve saved preferences.
Battery Saver retains its Dim curve and other boards retain 50%.

Fixes FIRM-4392

Co-Authored-By: GPT-6 Astra <noreply@openai.com>
Signed-off-by: Joshua Jun <lets@throw.rocks>
@jplexer
jplexer force-pushed the codex/firm-4392-getafix-battery-saver branch from d08e405 to 9dbfafa Compare September 16, 2026 11:04
@jplexer
jplexer requested a review from gmarull September 16, 2026 11:27
@jplexer
jplexer merged commit 225910f into coredevices:main Sep 16, 2026
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants