Skip to content

Power: idle dim and suspend readiness - #34

Open
brkosebay wants to merge 11 commits into
OrchardLinux:mainfrom
brkosebay:j71t/power
Open

brkosebay wants to merge 11 commits into
OrchardLinux:mainfrom
brkosebay:j71t/power

Conversation

@brkosebay

@brkosebay brkosebay commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Part 2 of 14 of the iPad 5 (J71t) series in this repository; merge after #33.

Problem

The tablet had no dim before idle lock, and suspend was offered even when the kernel could not complete it.

Change

Dim before the idle lock, disable hibernation, and gate every suspend (Power, idle, and the Smart Cover lock that main now does) on ipadlinux-suspend-ready, which answers from the kernel's own admission query, /sys/power/suspend_ready, on every board: no feature table, no board list, and a kernel without the query refuses. Sleep follows the kernel's suspend_ready; on J71t the G5P driver refuses until GPU resume works (OrchardLinux/orchard#120). The query comes with OrchardLinux/orchard#117; until a kernel has it, every sleep request is refused. Main's Smart Cover lock and wake are kept as they are; this branch's earlier cover-close requests are dropped for them. Documented in docs/power-and-lock.md.

Testing

  • Hardware: iPad 5 Wi-Fi (J71t). Hands-on testing on 2026-11-11 with an integrated build of this series, made before the final failure-path fixes, covered lock and idle behaviour.
  • 2026-11-12: no further hardware test of this change beyond the above.
  • Host: the Linux test suite passes at this branch's head (with Orchard at the top of its series).

Dependencies

Depends on #33.

This branch is stacked on the PRs it depends on and carries their commits; the change of this PR is its last commit, "Power: idle dim and suspend readiness".

This update keeps the branch's earlier history: it merges the current base into the branch and adds the reworked change as a new commit on top.

🤖 Generated with Claude Code

Bring this branch onto current main. The merge keeps the other side's
tree; the next commit carries the reworked version of this branch's
change.
Bring this branch onto current main and the series it depends on. The
merge keeps the other side's tree; the next commit carries the reworked
version of this branch's change.
Automatic rotation could not be overridden: there was no manual rotation
or lock in the menu.

Add omarchy-rotate with rotate and lock commands, menu entries for them,
and the release root wiring.

Assisted-by: LLM
The tablet had no lock on cover close, no dim before idle lock, and
suspend was offered even when the kernel could not complete it.

Lock on cover close, dim before the idle lock, disable hibernation, and
gate suspend on ipadlinux-suspend-ready, the kernel's readiness report.
Documented in docs/power-and-lock.md.

Assisted-by: LLM
Bring this branch onto current main, which now has the Smart Cover lock
and wake in omarchy-powerd.
Bring this branch onto current main, which reads the Smart Cover in
omarchy-powerd: closing it is a lock, opening it a wake. That replaces
this branch's cover-close and cover-suspend socket requests and their
test. A sleep after a cover lock goes through the same suspend
readiness check as Power and idle, which the merged test now covers.
An earlier wording pass on this branch turned "owner" into
"maintainer" in lines that come from main: powerd is the one owner of
the screen, the lock request and sleep, and the shutdown menu test
quotes that. Restore main's wording in the state model doc, the
powerd comment and the test.

Assisted-by: LLM
Orchard main now gives every installed system the Mac current
identity (0x1014) on its own, and nothing reads
/etc/orchard/mac-usb-current any more, so the notes on opting in with
that flag describe a switch that no longer exists.

Assisted-by: LLM
ipadlinux-suspend-ready first asked Orchard's device-feature helper
whether the board's feature table marked suspend as working. Orchard
does not ship that table: feature states describe a board, they do not
switch behaviour. Without the helper every sleep request was refused.

Decide from the kernel's own admission query on every board instead.
/sys/power/suspend_ready answers "ready" or names the first driver or
precheck that would refuse; a kernel without the query cannot show
that a suspend would be admitted, so sleep stays refused there, as do
an unreadable or malformed answer. The helper no longer reads the
board identity. Powerd, the session menu and systemd's direct suspend
guard keep calling the same helper.

Assisted-by: LLM
@brkosebay brkosebay changed the title Power, idle dim and native logout Power: idle dim and suspend readiness Nov 13, 2026
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.

1 participant