Skip to content

d/patches: disable audiopd daemon on QRB2210 - #97

Open
Christopher Obbard (obbardc) wants to merge 1 commit into
qli/debian/latestfrom
wip/obbardc/qrb2210-disable-audiopd
Open

Christopher Obbard (obbardc) wants to merge 1 commit into
qli/debian/latestfrom
wip/obbardc/qrb2210-disable-audiopd

Conversation

@obbardc

Copy link
Copy Markdown

QRB2210 (RB1 and Arduino Uno Q) exposes /dev/fastrpc-adsp, so the unit's existing ConditionPathExists= checks pass and adsprpcd is started, but on these Agatti platforms the device tree does not protect the memory region used by fastrpc. Starting the audiopd daemon reboots the board which makes the machine unusable rather than simply leaving the unit failed.

The kernel side is fixed by the device tree patch linked below but it may not reach released kernels for some time. Until it does, guard the unit with a ConditionFirmware= match on the device tree compatible so it is skipped on affected machines:
https://lore.kernel.org/all/20260908-agatti-audio-v4-1-b3db91d3fdec@oss.qualcomm.com/

QRB2210 (RB1 and Arduino Uno Q) exposes /dev/fastrpc-adsp, so the unit's
existing ConditionPathExists= checks pass and adsprpcd is started, but on
these Agatti platforms the device tree does not protect the memory region
used by fastrpc. Starting the audiopd daemon reboots the board which makes
the machine unusable rather than simply leaving the unit failed.

The kernel side is fixed by the device tree patch linked below but it may
not reach released kernels for some time. Until it does, guard the
unit with a ConditionFirmware= match on the device tree compatible so it
is skipped on affected machines:
https://lore.kernel.org/all/20260908-agatti-audio-v4-1-b3db91d3fdec@oss.qualcomm.com/

Signed-off-by: Christopher Obbard <chris.obbard@oss.qualcomm.com>
@obbardc

Copy link
Copy Markdown
Author

Loïc Minier (@lool) should this be merged into debian/latest first ? Then into qli/* ? Not sure what the plan should be regarding this PR. I am happy to do the mechanical bits though.

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