From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH] clk: samsung: Allow modular build of the Audio Subsystem CLKCON driver To: =?UTF-8?Q?Krzysztof_Koz=c5=82owski?= References: <1467987300-31450-1-git-send-email-s.nawrocki@samsung.com> <146799700614.73491.6564650478987470074@resonance> <578616AE.7010805@samsung.com> <578DF91D.8050800@samsung.com> Cc: Michael Turquette , linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, =?UTF-8?Q?Bart=c5=82omiej_=c5=bbo=c5=82nierkiewicz?= From: Sylwester Nawrocki Message-id: <579112F9.3030608@samsung.com> Date: Thu, 21 Jul 2016 20:22:49 +0200 MIME-version: 1.0 In-reply-to: <578DF91D.8050800@samsung.com> Content-type: text/plain; charset=utf-8 List-ID: On 07/19/2016 11:55 AM, Sylwester Nawrocki wrote: >> Unfortunately this breaks the audio on Odroid XU3 (simple-card). There >> > is no sound from: >> > $ aplay /usr/share/sounds/alsa/Front_Right.wav >> > and the command does not exit. >> > >> > Sylwester, >> > Are you going to look at this? > > Thanks for the bug report, I will check what's the problem here. > Odroid U3 also uses simple card and same codec and it all works there. It turned out that the difference was using ADMA on XU3 vs. PDMA on U3. AUDSS clock controller provides a clock for ADMA and converting the AUDSS clk driver to a loadable module causes deferring ADMA's controller initialization, which was not handled properly by the sound component drivers. I've sent a patch to fix the issue, added you at Cc. -- Thanks, Sylwester