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

Commit 947edcaa authored by qctecmdr's avatar qctecmdr Committed by Gerrit - the friendly Code Review server
Browse files

Merge "asoc: msm-compress: Replace goto with return in case of invalid value"

parents bd8ab508 9adf04ca

asoc/msm-compress-q6-v2.c

+10 −20
Original line number Original line Diff line number Diff line
@@ -3702,8 +3702,7 @@ static int msm_compr_dec_params_put(struct snd_kcontrol *kcontrol, 
	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

		pr_err("%s Received out of bounds fe_id %lu\n",

		pr_err("%s Received out of bounds fe_id %lu\n",

			__func__, fe_id);

			__func__, fe_id);

		rc = -EINVAL;

		return -EINVAL;

		goto end;

	}

	}





	cstream = pdata->cstream[fe_id];

	cstream = pdata->cstream[fe_id];
@@ -3711,14 +3710,12 @@ static int msm_compr_dec_params_put(struct snd_kcontrol *kcontrol, 




	if (!cstream || !dec_params) {

	if (!cstream || !dec_params) {

		pr_err("%s: stream or dec_params inactive\n", __func__);

		pr_err("%s: stream or dec_params inactive\n", __func__);

		rc = -EINVAL;

		return -EINVAL;

		goto end;

	}

	}

	prtd = cstream->runtime->private_data;

	prtd = cstream->runtime->private_data;

	if (!prtd) {

	if (!prtd) {

		pr_err("%s: cannot set dec_params\n", __func__);

		pr_err("%s: cannot set dec_params\n", __func__);

		rc = -EINVAL;

		return -EINVAL;

		goto end;

	}

	}





	mutex_lock(&pdata->lock);

	mutex_lock(&pdata->lock);
@@ -3975,22 +3972,19 @@ static int msm_compr_adsp_stream_cmd_put(struct snd_kcontrol *kcontrol, 
	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

		pr_err("%s Received invalid fe_id %lu\n",

		pr_err("%s Received invalid fe_id %lu\n",

			__func__, fe_id);

			__func__, fe_id);

		ret = -EINVAL;

		return -EINVAL;

		goto done;

	}

	}





	cstream = pdata->cstream[fe_id];

	cstream = pdata->cstream[fe_id];

	if (cstream == NULL) {

	if (cstream == NULL) {

		pr_err("%s cstream is null\n", __func__);

		pr_err("%s cstream is null\n", __func__);

		ret = -EINVAL;

		return -EINVAL;

		goto done;

	}

	}





	prtd = cstream->runtime->private_data;

	prtd = cstream->runtime->private_data;

	if (!prtd) {

	if (!prtd) {

		pr_err("%s: prtd is null\n", __func__);

		pr_err("%s: prtd is null\n", __func__);

		ret = -EINVAL;

		return -EINVAL;

		goto done;

	}

	}





	mutex_lock(&pdata->lock);

	mutex_lock(&pdata->lock);
@@ -4041,22 +4035,19 @@ static int msm_compr_ion_fd_map_put(struct snd_kcontrol *kcontrol, 
	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

		pr_err("%s Received out of bounds invalid fe_id %lu\n",

		pr_err("%s Received out of bounds invalid fe_id %lu\n",

			__func__, fe_id);

			__func__, fe_id);

		ret = -EINVAL;

		return -EINVAL;

		goto done;

	}

	}





	cstream = pdata->cstream[fe_id];

	cstream = pdata->cstream[fe_id];

	if (cstream == NULL) {

	if (cstream == NULL) {

		pr_err("%s cstream is null\n", __func__);

		pr_err("%s cstream is null\n", __func__);

		ret = -EINVAL;

		return -EINVAL;

		goto done;

	}

	}





	prtd = cstream->runtime->private_data;

	prtd = cstream->runtime->private_data;

	if (!prtd) {

	if (!prtd) {

		pr_err("%s: prtd is null\n", __func__);

		pr_err("%s: prtd is null\n", __func__);

		ret = -EINVAL;

		return -EINVAL;

		goto done;

	}

	}





	mutex_lock(&pdata->lock);

	mutex_lock(&pdata->lock);
@@ -4090,8 +4081,7 @@ static int msm_compr_rtic_event_ack_put(struct snd_kcontrol *kcontrol, 
	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

	if (fe_id >= MSM_FRONTEND_DAI_MAX) {

		pr_err("%s Received invalid fe_id %lu\n",

		pr_err("%s Received invalid fe_id %lu\n",

			__func__, fe_id);

			__func__, fe_id);

		ret = -EINVAL;

		return -EINVAL;

		goto done;

	}

	}





	mutex_lock(&pdata->lock);

	mutex_lock(&pdata->lock);