Skip to content

audioreach-driver: Enable RPMsg packet driver#54

Closed
quic-annemari wants to merge 0 commit into
AudioReach:masterfrom
quic-annemari:master
Closed

audioreach-driver: Enable RPMsg packet driver#54
quic-annemari wants to merge 0 commit into
AudioReach:masterfrom
quic-annemari:master

Conversation

@quic-annemari
Copy link
Copy Markdown
Contributor

Add new packet driver file q6apm_audio_pkt_rpmsg, which uses RPMsg as communication between HLOS and DSP. Also adds a feature flag that will disable hyp_assign for non-qcom devices.

@quic-annemari quic-annemari requested review from a team March 5, 2026 22:32
@quic-annemari quic-annemari force-pushed the master branch 2 times, most recently from f7390ea to 5bded5e Compare March 5, 2026 23:47
Comment thread audioreach-driver/Makefile Outdated
audioreach_driver-objs += q6apm_audio_pkt.o
ccflags-y += -DQCOM_HYP_ASSIGN
else
audioreach_driver-objs += q6apm_audio_pkt_rpmsg.o
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.

Let's rename to audioreach_pkt_rpmsg

#include <linux/uaccess.h>
#include <linux/termios.h>
#include <linux/wait.h>
#include "audioreach.h"
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.

double check if it is really necessary to include this header file

#include <linux/termios.h>
#include <linux/wait.h>
#include "audioreach.h"
#include "q6prm_audioreach.h"
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.

double check if it is really necessary to include this header file

// Copyright (c) 2020, Linaro Limited
// Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.

#include <dt-bindings/soc/qcom,gpr.h>
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.

I don't think we need to include this header.

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