Re: [PATCH V2 0/2] Sound support for Exynos4412 Odroid X2, U3 board

2014-06-24 Thread Marek Szyprowski

Hello,

On 2014-06-24 10:35, Daniel Drake wrote:

On Mon, Jun 23, 2014 at 5:32 PM, Sylwester Nawrocki
 wrote:

I could reproduce such behaviour on the U3 board, but only with u-boot
which sets the MPLL clock frequency (fout_mpll) to 880 MHz, rather
than 800 MHz, which was the case in my original environment.
All fout_mpll child clocks have then different frequency values
in both cases.
It's a bit strange though, because frequencies of all the audio
subsystem clocks seem to be same anyway:

I'm using the standard uboot from hardkernel.


# cat /sys/kernel/debug/clk/clk_summary

This command makes the kernel totally hang, weird.


This is known issue with clocks vs. ISP power domain. We should really 
fix this in exynos4 clock driver.



# speaker-test -c 2 -t wav -l 2 -p 1024

This plays back fine.

Could it be a problem with the samsung-i2s driver, not correctly
flushing at the right times?
Or do you think the problem is more likely to be clock-related?


Hard to say right now, but at least we need to test a bit more our 
solutions with different initial clocks configuration.


Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH V2 0/2] Sound support for Exynos4412 Odroid X2, U3 board

2014-06-24 Thread Daniel Drake
On Mon, Jun 23, 2014 at 5:32 PM, Sylwester Nawrocki
 wrote:
> I could reproduce such behaviour on the U3 board, but only with u-boot
> which sets the MPLL clock frequency (fout_mpll) to 880 MHz, rather
> than 800 MHz, which was the case in my original environment.
> All fout_mpll child clocks have then different frequency values
> in both cases.
> It's a bit strange though, because frequencies of all the audio
> subsystem clocks seem to be same anyway:

I'm using the standard uboot from hardkernel.

> # cat /sys/kernel/debug/clk/clk_summary

This command makes the kernel totally hang, weird.

# speaker-test -c 2 -t wav -l 2 -p 1024

This plays back fine.

Could it be a problem with the samsung-i2s driver, not correctly
flushing at the right times?
Or do you think the problem is more likely to be clock-related?

Thanks
Daniel
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH V2 0/2] Sound support for Exynos4412 Odroid X2, U3 board

2014-06-23 Thread Sylwester Nawrocki
On 23/06/14 18:32, Sylwester Nawrocki wrote:
> On 23/06/14 11:40, Daniel Drake wrote:
>> > I tested ODROID-U2's 3.5mm analog headphone jack output with:
>> > 
>> > # speaker-test -c 2 -t wav -l 2
>> > 
>> > On my x86 laptop this command takes ~6 seconds and produces audible output:
>> > "Front left, front right, front left, front right"
>> > 
>> > When those words are reproduced over the speakers, there is a
>> > corresponding message printed to stdout which synchronizes nicely.
>> > 
>> > On ODROID-U2 the same command doesn't work quite right - execution
>> > takes only 3.5 seconds and the audible output is truncated:
>> > "Front left, front right, front"
>> > and the stdout messages do not really coincide with the audio being
>> > reproduced at that time.

Is it still wrong when you set the period manually to 1024, i.e.

# speaker-test -c 2 -t wav -l 2 -p 1024

?

--
Thanks,
Sylwester
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH V2 0/2] Sound support for Exynos4412 Odroid X2, U3 board

2014-06-23 Thread Sylwester Nawrocki
On 23/06/14 11:40, Daniel Drake wrote:
> I tested ODROID-U2's 3.5mm analog headphone jack output with:
> 
> # speaker-test -c 2 -t wav -l 2
> 
> On my x86 laptop this command takes ~6 seconds and produces audible output:
> "Front left, front right, front left, front right"
> 
> When those words are reproduced over the speakers, there is a
> corresponding message printed to stdout which synchronizes nicely.
> 
> On ODROID-U2 the same command doesn't work quite right - execution
> takes only 3.5 seconds and the audible output is truncated:
> "Front left, front right, front"
> and the stdout messages do not really coincide with the audio being
> reproduced at that time.
> 
> No pulseaudio or anything like that.
> 
> Can you reproduce this?

I could reproduce such behaviour on the U3 board, but only with u-boot
which sets the MPLL clock frequency (fout_mpll) to 880 MHz, rather
than 800 MHz, which was the case in my original environment.
All fout_mpll child clocks have then different frequency values
in both cases.
It's a bit strange though, because frequencies of all the audio
subsystem clocks seem to be same anyway:

# cat /sys/kernel/debug/clk/clk_summary
...
fout_epll   0   019200
   mout_audss   0   019200
  dout_rp   0   01920
 dout_aud_bus   0   01920
  mout_i2s0 0   019200
 dout_i2s0  0   019200
   sclk_epll0   019200
  audss 0   019200

So there might be some other difference I'm overlooking.
Will let you know as soon I find out anything else.

Regards,
Sylwester
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH V2 0/2] Sound support for Exynos4412 Odroid X2, U3 board

2014-06-23 Thread Daniel Drake
On Wed, Jun 18, 2014 at 5:22 PM, Sylwester Nawrocki
 wrote:
> This series adds basic sound support for the Odroid X2/U3 boards.
> It relies on specific Exynos Audio Subsystem clock parent and
> frequencies being pre-configured.
>
> My full testing git branch has been pushed to:
> git://linuxtv.org/snawrocki/samsung.git v3.16-rc1-odroid-sound-clk

Thanks for this! I tested these patches plus the ones in your branch
to integrate with the ODROID.
I tested ODROID-U2's 3.5mm analog headphone jack output with:

# speaker-test -c 2 -t wav -l 2

On my x86 laptop this command takes ~6 seconds and produces audible output:
"Front left, front right, front left, front right"

When those words are reproduced over the speakers, there is a
corresponding message printed to stdout which synchronizes nicely.

On ODROID-U2 the same command doesn't work quite right - execution
takes only 3.5 seconds and the audible output is truncated:
"Front left, front right, front"
and the stdout messages do not really coincide with the audio being
reproduced at that time.

No pulseaudio or anything like that.

Can you reproduce this?

Thanks
Daniel
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH V2 0/2] Sound support for Exynos4412 Odroid X2, U3 board

2014-06-18 Thread Sylwester Nawrocki
This series adds basic sound support for the Odroid X2/U3 boards.
It relies on specific Exynos Audio Subsystem clock parent and
frequencies being pre-configured.

My full testing git branch has been pushed to:
git://linuxtv.org/snawrocki/samsung.git v3.16-rc1-odroid-sound-clk

It can be browsed at:
http://git.linuxtv.org/cgit.cgi/snawrocki/samsung.git/log/?h=v3.16-rc1-odroid-sound-clk

I didn't use the simple-card since there is going to be added
one more CPU interface using the secondary I2S and IDMA.

Sylwester Nawrocki (2):
  ASoC: samsung: Add machine driver for Odroid X2/U3
  doc: dt bindings: Document Odroid X2/U3 audio subsystem bindings

 .../bindings/sound/samsung,odroidx2-max98090.txt   |   35 
 sound/soc/samsung/Kconfig  |8 +
 sound/soc/samsung/Makefile |2 +
 sound/soc/samsung/odroidx2_max98090.c  |  191 
 4 files changed, 236 insertions(+)
 create mode 100644 
Documentation/devicetree/bindings/sound/samsung,odroidx2-max98090.txt
 create mode 100644 sound/soc/samsung/odroidx2_max98090.c

--
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html