Skip to content

audioreach-driver: Enable RPMsg packet driver#54

Open
quic-annemari wants to merge 1 commit intoAudioReach:masterfrom
quic-annemari:master
Open

audioreach-driver: Enable RPMsg packet driver#54
quic-annemari wants to merge 1 commit intoAudioReach:masterfrom
quic-annemari:master

Conversation

@quic-annemari
Copy link
Copy Markdown

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.

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.

Signed-off-by: Annemarie Porter <annemari@qti.qualcomm.com>
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