* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-08-26 8:45 ` 박상수
0 siblings, 0 replies; 14+ messages in thread
From: 박상수 @ 2012-08-26 8:45 UTC (permalink / raw)
To: alsa-devel, linux-arm-kernel, linux-samsung-soc
Cc: lrg, broonie, sbkim73, ben-linux, Kukjin Kim
SND_SOC_DAIFMT_CONT option is now needed because of below commit:
ASoC: Allow DAI formats to be specified in the dai_link
Otherwise, s3c_pcm_set_fmt() returns -EINVAL.
Signed-off-by: Sangsu Park <sangsu4u.park@samsung.com>
---
sound/soc/samsung/smdk_wm8994pcm.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/sound/soc/samsung/smdk_wm8994pcm.c b/sound/soc/samsung/smdk_wm8994pcm.c
index 77ecba9..cb88cbc 100644
--- a/sound/soc/samsung/smdk_wm8994pcm.c
+++ b/sound/soc/samsung/smdk_wm8994pcm.c
@@ -77,6 +77,7 @@ static int smdk_wm8994_pcm_hw_params(struct snd_pcm_substream *substream,
/* Set the cpu DAI configuration */
ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_DSP_B
+ | SND_SOC_DAIFMT_CONT
| SND_SOC_DAIFMT_IB_NF
| SND_SOC_DAIFMT_CBS_CFS);
if (ret < 0)
--
1.7.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-08-26 8:45 ` 박상수
0 siblings, 0 replies; 14+ messages in thread
From: 박상수 @ 2012-08-26 8:45 UTC (permalink / raw)
To: linux-arm-kernel
SND_SOC_DAIFMT_CONT option is now needed because of below commit:
ASoC: Allow DAI formats to be specified in the dai_link
Otherwise, s3c_pcm_set_fmt() returns -EINVAL.
Signed-off-by: Sangsu Park <sangsu4u.park@samsung.com>
---
sound/soc/samsung/smdk_wm8994pcm.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/sound/soc/samsung/smdk_wm8994pcm.c b/sound/soc/samsung/smdk_wm8994pcm.c
index 77ecba9..cb88cbc 100644
--- a/sound/soc/samsung/smdk_wm8994pcm.c
+++ b/sound/soc/samsung/smdk_wm8994pcm.c
@@ -77,6 +77,7 @@ static int smdk_wm8994_pcm_hw_params(struct snd_pcm_substream *substream,
/* Set the cpu DAI configuration */
ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_DSP_B
+ | SND_SOC_DAIFMT_CONT
| SND_SOC_DAIFMT_IB_NF
| SND_SOC_DAIFMT_CBS_CFS);
if (ret < 0)
--
1.7.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
2012-08-26 8:45 ` 박상수
@ 2012-08-27 20:45 ` Mark Brown
-1 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2012-08-27 20:45 UTC (permalink / raw)
To: �ڻ���
Cc: alsa-devel, linux-arm-kernel, linux-samsung-soc, lrg, sbkim73,
ben-linux, Kukjin Kim
On Sun, Aug 26, 2012 at 05:45:54PM +0900, �ڻ��� wrote:
> SND_SOC_DAIFMT_CONT option is now needed because of below commit:
>
> ASoC: Allow DAI formats to be specified in the dai_link
Clearly it's not a direct consequence of this commit, that just
introduces a new optional facility which...
> /* Set the cpu DAI configuration */
> ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_DSP_B
> + | SND_SOC_DAIFMT_CONT
> | SND_SOC_DAIFMT_IB_NF
> | SND_SOC_DAIFMT_CBS_CFS);
...isn't being used by the driver.
Looking at the above it looks like the PCM driver is doing something
very odd - _CONT doesn't make a whole lot of sense on the slave side,
it's related to the clock generation so should have no effect at all on
the slave side. The driver code looks reasonably sensible though.
What specifically is going wrong here?
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-08-27 20:45 ` Mark Brown
0 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2012-08-27 20:45 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Aug 26, 2012 at 05:45:54PM +0900, ????? wrote:
> SND_SOC_DAIFMT_CONT option is now needed because of below commit:
>
> ASoC: Allow DAI formats to be specified in the dai_link
Clearly it's not a direct consequence of this commit, that just
introduces a new optional facility which...
> /* Set the cpu DAI configuration */
> ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_DSP_B
> + | SND_SOC_DAIFMT_CONT
> | SND_SOC_DAIFMT_IB_NF
> | SND_SOC_DAIFMT_CBS_CFS);
...isn't being used by the driver.
Looking at the above it looks like the PCM driver is doing something
very odd - _CONT doesn't make a whole lot of sense on the slave side,
it's related to the clock generation so should have no effect at all on
the slave side. The driver code looks reasonably sensible though.
What specifically is going wrong here?
^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
2012-08-27 20:45 ` Mark Brown
@ 2012-08-29 11:06 ` Sangsu Park
-1 siblings, 0 replies; 14+ messages in thread
From: Sangsu Park @ 2012-08-29 11:06 UTC (permalink / raw)
To: 'Mark Brown'
Cc: alsa-devel, linux-samsung-soc, sbkim73, 'Kukjin Kim',
ben-linux, lrg, linux-arm-kernel
On Sun, Aug 26, 2012 at 05:45:54PM +0900, Mark Brown wrote:
> On Sun, Aug 26, 2012 at 05:45:54PM +0900, ڻ wrote:
> > SND_SOC_DAIFMT_CONT option is now needed because of below commit:
> >
> > ASoC: Allow DAI formats to be specified in the dai_link
>
> Clearly it's not a direct consequence of this commit, that just introduces a new optional facility
> which...
OK, I'll change it.
>
> > /* Set the cpu DAI configuration */
> > ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_DSP_B
> > + | SND_SOC_DAIFMT_CONT
> > | SND_SOC_DAIFMT_IB_NF
> > | SND_SOC_DAIFMT_CBS_CFS);
>
> ...isn't being used by the driver.
>
> Looking at the above it looks like the PCM driver is doing something very odd - _CONT doesn't make
> a whole lot of sense on the slave side, it's related to the clock generation so should have no
> effect at all on the slave side. The driver code looks reasonably sensible though.
You mean cpu side is slave?
And I think that pcm driver can be changed to solve this issue.
>
> What specifically is going wrong here?
SND_SOC_DAIFMT_CONT is used sound/soc/Samsung/pcm.c (s3c_pcm_set_fmt()) like below.
switch (fmt & SND_SOC_DAIFMT_CLOCK_MASK) {
case SND_SOC_DAIFMT_CONT:
pcm->idleclk = 1;
break;
case SND_SOC_DAIFMT_GATED:
pcm->idleclk = 0;
break;
default:
dev_err(pcm->dev, "Invalid Clock gating request!\n");
ret = -EINVAL;
goto exit;
}
If fmt argument has no SND_SOC_DAIFMT_ option, s3c_pcm_set_fmt() return -EINVAL.
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-08-29 11:06 ` Sangsu Park
0 siblings, 0 replies; 14+ messages in thread
From: Sangsu Park @ 2012-08-29 11:06 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Aug 26, 2012 at 05:45:54PM +0900, Mark Brown wrote:
> On Sun, Aug 26, 2012 at 05:45:54PM +0900, ? wrote:
> > SND_SOC_DAIFMT_CONT option is now needed because of below commit:
> >
> > ASoC: Allow DAI formats to be specified in the dai_link
>
> Clearly it's not a direct consequence of this commit, that just introduces a new optional facility
> which...
OK, I'll change it.
>
> > /* Set the cpu DAI configuration */
> > ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_DSP_B
> > + | SND_SOC_DAIFMT_CONT
> > | SND_SOC_DAIFMT_IB_NF
> > | SND_SOC_DAIFMT_CBS_CFS);
>
> ...isn't being used by the driver.
>
> Looking at the above it looks like the PCM driver is doing something very odd - _CONT doesn't make
> a whole lot of sense on the slave side, it's related to the clock generation so should have no
> effect at all on the slave side. The driver code looks reasonably sensible though.
You mean cpu side is slave?
And I think that pcm driver can be changed to solve this issue.
>
> What specifically is going wrong here?
SND_SOC_DAIFMT_CONT is used sound/soc/Samsung/pcm.c (s3c_pcm_set_fmt()) like below.
switch (fmt & SND_SOC_DAIFMT_CLOCK_MASK) {
case SND_SOC_DAIFMT_CONT:
pcm->idleclk = 1;
break;
case SND_SOC_DAIFMT_GATED:
pcm->idleclk = 0;
break;
default:
dev_err(pcm->dev, "Invalid Clock gating request!\n");
ret = -EINVAL;
goto exit;
}
If fmt argument has no SND_SOC_DAIFMT_ option, s3c_pcm_set_fmt() return -EINVAL.
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
2012-08-29 11:06 ` Sangsu Park
@ 2012-08-30 17:43 ` Mark Brown
-1 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2012-08-30 17:43 UTC (permalink / raw)
To: Sangsu Park
Cc: alsa-devel, linux-samsung-soc, sbkim73, 'Kukjin Kim',
ben-linux, lrg, linux-arm-kernel
On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
Please check your mailer configuration, it looks like it's reformatting
all the text with much longer line widths.
> On Sun, Aug 26, 2012 at 05:45:54PM +0900, Mark Brown wrote:
> > a whole lot of sense on the slave side, it's related to the clock generation so should have no
> > effect at all on the slave side. The driver code looks reasonably sensible though.
> You mean cpu side is slave?
Yes.
> > What specifically is going wrong here?
> SND_SOC_DAIFMT_CONT is used sound/soc/Samsung/pcm.c (s3c_pcm_set_fmt()) like below.
> switch (fmt & SND_SOC_DAIFMT_CLOCK_MASK) {
> case SND_SOC_DAIFMT_CONT:
> pcm->idleclk = 1;
> break;
> case SND_SOC_DAIFMT_GATED:
> pcm->idleclk = 0;
> break;
> default:
> dev_err(pcm->dev, "Invalid Clock gating request!\n");
> ret = -EINVAL;
> goto exit;
> }
> If fmt argument has no SND_SOC_DAIFMT_ option, s3c_pcm_set_fmt() return -EINVAL.
So, clearly this is unrelated to the change you mentioned in your commit
message - it looks like this is just a plain issue in the machine
driver. That said the above code should be changed to at least ignore
the setting in slave mode (since the clock shouldn't be being driven and
it doesn't matter).
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-08-30 17:43 ` Mark Brown
0 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2012-08-30 17:43 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
Please check your mailer configuration, it looks like it's reformatting
all the text with much longer line widths.
> On Sun, Aug 26, 2012 at 05:45:54PM +0900, Mark Brown wrote:
> > a whole lot of sense on the slave side, it's related to the clock generation so should have no
> > effect at all on the slave side. The driver code looks reasonably sensible though.
> You mean cpu side is slave?
Yes.
> > What specifically is going wrong here?
> SND_SOC_DAIFMT_CONT is used sound/soc/Samsung/pcm.c (s3c_pcm_set_fmt()) like below.
> switch (fmt & SND_SOC_DAIFMT_CLOCK_MASK) {
> case SND_SOC_DAIFMT_CONT:
> pcm->idleclk = 1;
> break;
> case SND_SOC_DAIFMT_GATED:
> pcm->idleclk = 0;
> break;
> default:
> dev_err(pcm->dev, "Invalid Clock gating request!\n");
> ret = -EINVAL;
> goto exit;
> }
> If fmt argument has no SND_SOC_DAIFMT_ option, s3c_pcm_set_fmt() return -EINVAL.
So, clearly this is unrelated to the change you mentioned in your commit
message - it looks like this is just a plain issue in the machine
driver. That said the above code should be changed to at least ignore
the setting in slave mode (since the clock shouldn't be being driven and
it doesn't matter).
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
2012-08-30 17:43 ` Mark Brown
@ 2012-09-03 2:10 ` Sangsu Park
-1 siblings, 0 replies; 14+ messages in thread
From: Sangsu Park @ 2012-09-03 2:10 UTC (permalink / raw)
To: 'Mark Brown'
Cc: alsa-devel, linux-samsung-soc, sbkim73, 'Kukjin Kim',
ben-linux, lrg, linux-arm-kernel
On Sun, Aug 31, 2012 at 2:43 AM +0900, Mark Brown wrote:
> On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
>
> Please check your mailer configuration, it looks like it's reformatting
> all the text with much longer line widths.
I've changed line width configuration. Is it ok now?
>
> > On Sun, Aug 26, 2012 at 05:45:54PM +0900, Mark Brown wrote:
>
> > > a whole lot of sense on the slave side, it's related to the clock
> generation so should have no
> > > effect at all on the slave side. The driver code looks reasonably
> sensible though.
>
> > You mean cpu side is slave?
>
> Yes.
But, Samsung pcm is working only for cpu master.
>
>
> > > What specifically is going wrong here?
>
> > SND_SOC_DAIFMT_CONT is used sound/soc/Samsung/pcm.c (s3c_pcm_set_fmt()) like
> below.
>
> > switch (fmt & SND_SOC_DAIFMT_CLOCK_MASK) {
> > case SND_SOC_DAIFMT_CONT:
> > pcm->idleclk = 1;
> > break;
> > case SND_SOC_DAIFMT_GATED:
> > pcm->idleclk = 0;
> > break;
> > default:
> > dev_err(pcm->dev, "Invalid Clock gating request!\n");
> > ret = -EINVAL;
> > goto exit;
> > }
>
> > If fmt argument has no SND_SOC_DAIFMT_ option, s3c_pcm_set_fmt() return -
> EINVAL.
>
> So, clearly this is unrelated to the change you mentioned in your commit
> message - it looks like this is just a plain issue in the machine
> driver. That said the above code should be changed to at least ignore
> the setting in slave mode (since the clock shouldn't be being driven and
> it doesn't matter).
Do you think that changing pcm driver is right approach?
Then I'll fix pcm driver. (I think that pcm driver has some strange code.)
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-09-03 2:10 ` Sangsu Park
0 siblings, 0 replies; 14+ messages in thread
From: Sangsu Park @ 2012-09-03 2:10 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Aug 31, 2012 at 2:43 AM +0900, Mark Brown wrote:
> On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
>
> Please check your mailer configuration, it looks like it's reformatting
> all the text with much longer line widths.
I've changed line width configuration. Is it ok now?
>
> > On Sun, Aug 26, 2012 at 05:45:54PM +0900, Mark Brown wrote:
>
> > > a whole lot of sense on the slave side, it's related to the clock
> generation so should have no
> > > effect at all on the slave side. The driver code looks reasonably
> sensible though.
>
> > You mean cpu side is slave?
>
> Yes.
But, Samsung pcm is working only for cpu master.
>
>
> > > What specifically is going wrong here?
>
> > SND_SOC_DAIFMT_CONT is used sound/soc/Samsung/pcm.c (s3c_pcm_set_fmt()) like
> below.
>
> > switch (fmt & SND_SOC_DAIFMT_CLOCK_MASK) {
> > case SND_SOC_DAIFMT_CONT:
> > pcm->idleclk = 1;
> > break;
> > case SND_SOC_DAIFMT_GATED:
> > pcm->idleclk = 0;
> > break;
> > default:
> > dev_err(pcm->dev, "Invalid Clock gating request!\n");
> > ret = -EINVAL;
> > goto exit;
> > }
>
> > If fmt argument has no SND_SOC_DAIFMT_ option, s3c_pcm_set_fmt() return -
> EINVAL.
>
> So, clearly this is unrelated to the change you mentioned in your commit
> message - it looks like this is just a plain issue in the machine
> driver. That said the above code should be changed to at least ignore
> the setting in slave mode (since the clock shouldn't be being driven and
> it doesn't matter).
Do you think that changing pcm driver is right approach?
Then I'll fix pcm driver. (I think that pcm driver has some strange code.)
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
2012-09-03 2:10 ` Sangsu Park
@ 2012-09-06 0:21 ` Mark Brown
-1 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2012-09-06 0:21 UTC (permalink / raw)
To: Sangsu Park
Cc: alsa-devel, 'Kukjin Kim', sbkim73, linux-samsung-soc,
ben-linux, lrg, linux-arm-kernel
On Mon, Sep 03, 2012 at 11:10:03AM +0900, Sangsu Park wrote:
> On Sun, Aug 31, 2012 at 2:43 AM +0900, Mark Brown wrote:
> > On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
> > Please check your mailer configuration, it looks like it's reformatting
> > all the text with much longer line widths.
> I've changed line width configuration. Is it ok now?
Looks like it, thanks.
> Do you think that changing pcm driver is right approach?
> Then I'll fix pcm driver. (I think that pcm driver has some strange code.)
Well, we could do both. It'd certainly be more natural to make it have
a default given that this isn't something that normally needs to be
configured.
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-09-06 0:21 ` Mark Brown
0 siblings, 0 replies; 14+ messages in thread
From: Mark Brown @ 2012-09-06 0:21 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Sep 03, 2012 at 11:10:03AM +0900, Sangsu Park wrote:
> On Sun, Aug 31, 2012 at 2:43 AM +0900, Mark Brown wrote:
> > On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
> > Please check your mailer configuration, it looks like it's reformatting
> > all the text with much longer line widths.
> I've changed line width configuration. Is it ok now?
Looks like it, thanks.
> Do you think that changing pcm driver is right approach?
> Then I'll fix pcm driver. (I think that pcm driver has some strange code.)
Well, we could do both. It'd certainly be more natural to make it have
a default given that this isn't something that normally needs to be
configured.
^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
2012-09-06 0:21 ` Mark Brown
@ 2012-09-06 8:57 ` Sangsu Park
-1 siblings, 0 replies; 14+ messages in thread
From: Sangsu Park @ 2012-09-06 8:57 UTC (permalink / raw)
To: 'Mark Brown'
Cc: alsa-devel, linux-samsung-soc, sbkim73, 'Kukjin Kim',
ben-linux, lrg, linux-arm-kernel
On Thurs, Sep 06, 2012 at 9:22 AM +0900, Mark Brown wrote:
> On Mon, Sep 03, 2012 at 11:10:03AM +0900, Sangsu Park wrote:
> > On Sun, Aug 31, 2012 at 2:43 AM +0900, Mark Brown wrote:
> > > On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
>
> > > Please check your mailer configuration, it looks like it's reformatting
> > > all the text with much longer line widths.
>
> > I've changed line width configuration. Is it ok now?
>
> Looks like it, thanks.
>
> > Do you think that changing pcm driver is right approach?
> > Then I'll fix pcm driver. (I think that pcm driver has some strange code.)
>
> Well, we could do both. It'd certainly be more natural to make it have
> a default given that this isn't something that normally needs to be
> configured.
>
Then, I'll fix pcm driver to use default given
and resend that patch.
Thanks.
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt()
@ 2012-09-06 8:57 ` Sangsu Park
0 siblings, 0 replies; 14+ messages in thread
From: Sangsu Park @ 2012-09-06 8:57 UTC (permalink / raw)
To: linux-arm-kernel
On Thurs, Sep 06, 2012 at 9:22 AM +0900, Mark Brown wrote:
> On Mon, Sep 03, 2012 at 11:10:03AM +0900, Sangsu Park wrote:
> > On Sun, Aug 31, 2012 at 2:43 AM +0900, Mark Brown wrote:
> > > On Wed, Aug 29, 2012 at 08:06:32PM +0900, Sangsu Park wrote:
>
> > > Please check your mailer configuration, it looks like it's reformatting
> > > all the text with much longer line widths.
>
> > I've changed line width configuration. Is it ok now?
>
> Looks like it, thanks.
>
> > Do you think that changing pcm driver is right approach?
> > Then I'll fix pcm driver. (I think that pcm driver has some strange code.)
>
> Well, we could do both. It'd certainly be more natural to make it have
> a default given that this isn't something that normally needs to be
> configured.
>
Then, I'll fix pcm driver to use default given
and resend that patch.
Thanks.
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2012-09-06 8:57 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-26 8:45 [PATCH] ASoC: SAMSUNG: Add SND_SOC_DAIFMT_CONT option for snd_soc_set_fmt() 박상수
2012-08-26 8:45 ` 박상수
2012-08-27 20:45 ` Mark Brown
2012-08-27 20:45 ` Mark Brown
2012-08-29 11:06 ` Sangsu Park
2012-08-29 11:06 ` Sangsu Park
2012-08-30 17:43 ` Mark Brown
2012-08-30 17:43 ` Mark Brown
2012-09-03 2:10 ` Sangsu Park
2012-09-03 2:10 ` Sangsu Park
2012-09-06 0:21 ` Mark Brown
2012-09-06 0:21 ` Mark Brown
2012-09-06 8:57 ` Sangsu Park
2012-09-06 8:57 ` Sangsu Park
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.