Re: [PATCH V2 0/2] Sound support for Exynos4412 Odroid X2, U3 board
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
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
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
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
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
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