Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 04200bc9 authored by Ashish Jain's avatar Ashish Jain
Browse files

ASoC: bg: Register codec only once ADSP is up 



Sound card should not get registered till ADSP is up, hence
defer codec registration till the ADSP is up. Add a
flag in packetizer to indicate init complete.

Change-Id: I846049b586698ea6b29a651a9c2c65311a478723
Signed-off-by: default avatarAshish Jain <ashishj@codeaurora.org>
parent f38ea00b

drivers/soc/qcom/qdsp6v2/apr_v2.c

+2 −1
Original line number Diff line number Diff line 
/*

 * Copyright (c) 2012-2016 The Linux Foundation. All rights reserved.

 * Copyright (c) 2012-2016, 2018 The Linux Foundation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License version 2 and
@@ -25,6 +25,7 @@ enum apr_subsys_state apr_get_subsys_state(void) 
{

	return apr_get_q6_state();

}

EXPORT_SYMBOL(apr_get_subsys_state);



void apr_set_subsys_state(void)

{

drivers/soc/qcom/qdsp6v2/apr_v3.c

+2 −1
Original line number Diff line number Diff line 
/*

 * Copyright (c) 2013-2016, The Linux Foundation. All rights reserved.

 * Copyright (c) 2013-2016, 2018 The Linux Foundation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License version 2 and
@@ -25,6 +25,7 @@ enum apr_subsys_state apr_get_subsys_state(void) 
{

	return apr_get_modem_state();

}

EXPORT_SYMBOL(apr_get_subsys_state);



void apr_set_subsys_state(void)

{

sound/soc/codecs/bg_codec.c

+10 −1
Original line number Diff line number Diff line 
/*

 * Copyright (c) 2017, The Linux Foundation. All rights reserved.

 * Copyright (c) 2017-2018, The Linux Foundation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License version 2 and
@@ -29,6 +29,7 @@ 
#include <sound/soc.h>

#include <sound/soc-dapm.h>

#include <sound/tlv.h>

#include <linux/qdsp6v2/apr.h>

#include <sound/info.h>

#include <soc/qcom/bg_glink.h>

#include "pktzr.h"
@@ -739,6 +740,14 @@ static int bg_cdc_probe(struct platform_device *pdev) 
{

	struct bg_cdc_priv *bg_cdc;

	int ret = 0;

	int adsp_state;



	adsp_state = apr_get_subsys_state();

	if (adsp_state != APR_SUBSYS_LOADED) {

		pr_err("%s:Adsp is not loaded yet %d\n",

			__func__, adsp_state);

		return -EPROBE_DEFER;

	}



	bg_cdc = kzalloc(sizeof(struct bg_cdc_priv),

			    GFP_KERNEL);

sound/soc/codecs/pktzr.c

+7 −7
Original line number Diff line number Diff line 
/*

 * Copyright (c) 2017 The Linux Foundation. All rights reserved.

 * Copyright (c) 2017-2018, The Linux Foundation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License version 2 and
@@ -65,6 +65,7 @@ struct pktzr_priv { 
	struct platform_device *pdev;

	pktzr_data_cmd_cb_fn data_cmd_cb;

	int num_channels;

	bool pktzr_init_complete;

};



static struct pktzr_priv *ppriv;
@@ -103,8 +104,8 @@ static int pktzr_send_pkt(void *payload, uint32_t size, void *rsp, 
	int rc = 0;



	pr_debug("%s: cmd=%d sync=%d size=%d\n", __func__, cmd, sync_cmd, size);

	if (!ppriv) {

		pr_err("packetizer not initialized");

	if (!ppriv || !ppriv->pktzr_init_complete) {

		pr_err_ratelimited("packetizer not initialized\n");

		return -EINVAL;

	}

	mutex_lock(&ppriv->pktzr_lock);
@@ -237,22 +238,21 @@ int pktzr_init(void *pdev, struct bg_glink_ch_cfg *ch_info, int num_channels, 
			pr_err("%s: Failed to open channel\n", __func__);

			goto err;

		}

		ppriv->token = i;

	}

	ppriv->num_channels = num_channels;

	ppriv->data_cmd_cb = func;

	ppriv->token = 0;

	ppriv->pdev = pdev;

	init_completion(&ppriv->thread_complete);

	mutex_init(&ppriv->pktzr_lock);

	INIT_LIST_HEAD(&ppriv->ch_list);

	ppriv->pktzr_init_complete = true;



done:

	return 0;

err:

	for (i = 0; i < ppriv->token; i++)

		bg_cdc_channel_close(ppriv->pdev, ppriv->ch_info[i]);

	ppriv->token = 0;

		bg_cdc_channel_close(pdev, ppriv->ch_info[i]);



	if (ppriv)

		kzfree(ppriv);

	ppriv = NULL;