Skip to content

qcom-rtss-mailbox-kmd: add recipe for RTSS mailbox kernel driver#2528

Open
snegi-qti wants to merge 1 commit into
qualcomm-linux:masterfrom
snegi-qti:rtss_kmd
Open

qcom-rtss-mailbox-kmd: add recipe for RTSS mailbox kernel driver#2528
snegi-qti wants to merge 1 commit into
qualcomm-linux:masterfrom
snegi-qti:rtss_kmd

Conversation

@snegi-qti

Copy link
Copy Markdown

The RTSS (Real Time Subsystem) subsystem communicates with the application processor over a mailbox IPC channel. Add a recipe for the RTSS mailbox DLKM, which provides the kernel-side mailbox driver.

@ricardosalveti

ricardosalveti commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Repo is not even public: https://github.com/qualcomm-linux/rtss-mailbox-kmd

@ricardosalveti ricardosalveti marked this pull request as draft June 18, 2026 16:23
@snegi-qti snegi-qti force-pushed the rtss_kmd branch 2 times, most recently from be5f40d to 90ca377 Compare June 18, 2026 16:26
@snegi-qti

Copy link
Copy Markdown
Author

Sure, we are very close on making the repo public - awaiting some approvals, meanwhile, we can keep this as draft.

@github-actions

Copy link
Copy Markdown

Test run workflow

Test jobs for commit 58a5593

nodistro
Pass: 9 | Fail: 0 | Total: 9
qcom-distro_linux-qcom-6.18
Pass: 206 | Fail: 1 | Total: 245
qcom-distro
Pass: 228 | Fail: 0 | Total: 274

@test-reporting-app

test-reporting-app Bot commented Jun 19, 2026

Copy link
Copy Markdown

Test Results

  105 files  +   79    634 suites  +608   5h 16m 59s ⏱️ + 4h 21m 49s
  125 tests +  112    125 ✅ +  116   0 💤 ± 0  0 ❌  - 4 
6 037 runs  +5 842  5 953 ✅ +5 762  84 💤 +84  0 ❌  - 4 

Results for commit ebfa959. ± Comparison against base commit f3eeb8b.

This pull request removes 2 and adds 114 tests. Note that renamed tests count towards both.
lava ‑ auto-login-action
lava ‑ minimal-boot
0_AudioRecord_Config01 ‑ AudioRecord_Config01
0_BT_FW_KMD_Service ‑ BT_FW_KMD_Service
0_GStreamer_Video_Encode_h264_480p ‑ GStreamer_Video_Encode_h264_480p
0_Libcamera_cam ‑ Libcamera_cam
0_Weston_Runtime_Preflight ‑ Weston_Runtime_Preflight
0_hotplug ‑ hotplug
10_AudioRecord ‑ AudioRecord
10_Kubernetes_Kernel_Config ‑ Kubernetes_Kernel_Config
11_DSP_AudioPD ‑ DSP_AudioPD
11_Ethernet ‑ Ethernet
…

♻️ This comment has been updated with latest results.

The RTSS subsystem communicates with the application processor over
a mailbox IPC channel. Add a recipe for the RTSS mailbox DLKM, which
provides the kernel-side mailbox driver.

Signed-off-by: Sankalp Negi <snegi@qti.qualcomm.com>
@github-actions

Copy link
Copy Markdown

Test run workflow

Test jobs for commit ebfa959

nodistro
Pass: 9 | Fail: 0 | Total: 9
qcom-distro_linux-qcom-6.18
Pass: 188 | Fail: 0 | Total: 222
qcom-distro
Pass: 234 | Fail: 0 | Total: 277

@snegi-qti

Copy link
Copy Markdown
Author

qli-2.0 GA Critical Fix

@lumag

lumag commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

qli-2.0 GA Critical Fix

It's not a Fix. It's a new features, submitted past the cut-off date.

@@ -0,0 +1,39 @@
# SPDX-License-Identifier: GPL-2.0-only
# Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Drop this copyright part, not required for the recipe.

# SPDX-License-Identifier: GPL-2.0-only
# Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.

inherit module

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Move this part after SRCREV.

# Module metadata
RPROVIDES:${PN} += "kernel-module-rtss-mailbox"

# Build configuration

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Drop this comment,.

SRC_URI = "git://github.com/qualcomm-linux/rtss-mailbox-kmd.git;branch=rtss-mailbox-kernel.le.0.0;protocol=https"
SRCREV = "74f4ec17e45a65e455c2bff33638b5c5a7c507e0"

# Module metadata

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Drop this comment.

MAKE_TARGETS = "modules"
MODULES_INSTALL_TARGET = "modules_install"

# Auto-load module on boot

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why?

LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/${LICENSE};md5=801f80980d171dd6425610833a22dbe6"

SRC_URI = "git://github.com/qualcomm-linux/rtss-mailbox-kmd.git;branch=rtss-mailbox-kernel.le.0.0;protocol=https"
SRCREV = "74f4ec17e45a65e455c2bff33638b5c5a7c507e0"

@ricardosalveti ricardosalveti Jun 22, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please create a tag/release and use that instead of just a hash here (and then map the recipe version properly, instead of using _git.

@ricardosalveti ricardosalveti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please improve your commit message, explain why this module is needed in first place, how it is going to be used, by which components, etc.

@github-actions

Copy link
Copy Markdown

Test run workflow

Test jobs for commit ebfa959

nodistro
Pass: 9 | Fail: 0 | Total: 9
qcom-distro_linux-qcom-6.18
Pass: 211 | Fail: 0 | Total: 250
qcom-distro
Pass: 252 | Fail: 0 | Total: 297

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.

3 participants