Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Gyeongtaek Lee" <gt82.lee@samsung.com>
To: "'Kuninori Morimoto'" <kuninori.morimoto.gx@renesas.com>,
	<cpgs@samsung.com>
Cc: alsa-devel@alsa-project.org, khw0178.kim@samsung.com,
	'Takashi Iwai' <tiwai@suse.de>,
	lgirdwood@gmail.com,
	'Pierre-Louis Bossart' <pierre-louis.bossart@linux.intel.com>,
	kimty@samsung.com, donggyun.ko@samsung.com, hmseo@samsung.com,
	cpgs@samsung.com, s47.kang@samsung.com, pilsun.jang@samsung.com,
	tkjung@samsung.com
Subject: [PATCH v3 0/1] ASoC: dpcm: acquire dpcm_lock in dpcm_do_trigger()
Date: Thu, 3 Dec 2020 17:58:58 +0900	[thread overview]
Message-ID: <1891546521.01606986901738.JavaMail.epsvc@epcpadp3> (raw)
In-Reply-To: CGME20201203085858epcas2p1258c4c14b9398a15fb348ab92e306a25@epcas2p1.samsung.com

>Hi Gyeongtaek
>
>Thank you for your help
>
>> I tried to find out some change from the history.
>> However, I saw that dpcm_lock never be held in dpcm_be_dai_trigger() from the beginning.
>> So, I just fixed some miss from the commit comment and resend.
>> If you have another suggestion or request, just let me know.
>
>OK, thanks.
>I just thought it was related to recent (especially mine :) patch.
>
>BYW, your posted v2 still doesn't have alignment on log.
>Is it mailer issue ?
Sorry.
I didn't fully understand what alignment means.
I fixed the log by looking at some previous commits.

Thank you so much.

Gyeongtaek Lee
>
>> And I have a question.
>> This patch can?t be applied to stable branches directly by conflict.
>> I found out following commits are needed to apply this patch into the 5.4 stable.
>>  ASoC: soc-pcm: remove snd_soc_dpcm_be_get/set_state()
>>  ASoC: soc-pcm: add snd_soc_dpcm_can_be() and remove duplicate code
>> In this case, what should I do when stable maintainer requests resolving conflict to me?
>> Should I send those patches to the maintainer?
>> Or rewrite the patch on the latest of the each stable?
>
>Mark, please help him
>
>Thank you for your help !!
>
>Best regards
>---
>Kuninori Morimoto
>

Gyeongtaek Lee (1):
  ASoC: dpcm: acquire dpcm_lock in dpcm_do_trigger()

 sound/soc/soc-pcm.c | 62 ++++++++++++++++++++++++++++++++-------------
 1 file changed, 44 insertions(+), 18 deletions(-)


base-commit: fa02fcd94b0c8dff6cc65714510cf25ad194b90d
-- 
2.21.0




       reply	other threads:[~2020-12-03  9:16 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20201203085858epcas2p1258c4c14b9398a15fb348ab92e306a25@epcas2p1.samsung.com>
2020-12-03  8:58 ` Gyeongtaek Lee [this message]
     [not found] <CGME20201214083825epcas2p3c1b9e236ec1342195c46d91c6f29fb21@epcas2p3.samsung.com>
2020-12-14  8:38 ` [PATCH v3 0/1] ASoC: dpcm: acquire dpcm_lock in dpcm_do_trigger() Gyeongtaek Lee
     [not found] <CGME20201228112315epcas2p46266c89ffb53b307a1d751a590d06ad0@epcas2p4.samsung.com>
2020-12-28 11:23 ` Gyeongtaek Lee

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1891546521.01606986901738.JavaMail.epsvc@epcpadp3 \
    --to=gt82.lee@samsung.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=cpgs@samsung.com \
    --cc=donggyun.ko@samsung.com \
    --cc=hmseo@samsung.com \
    --cc=khw0178.kim@samsung.com \
    --cc=kimty@samsung.com \
    --cc=kuninori.morimoto.gx@renesas.com \
    --cc=lgirdwood@gmail.com \
    --cc=pierre-louis.bossart@linux.intel.com \
    --cc=pilsun.jang@samsung.com \
    --cc=s47.kang@samsung.com \
    --cc=tiwai@suse.de \
    --cc=tkjung@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox