All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sylwester Nawrocki <s.nawrocki@samsung.com>
To: "Krzysztof Kozłowski" <k.kozlowski.k+lkml@gmail.com>
Cc: "Michael Turquette" <mturquette@baylibre.com>,
	linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org,
	"Bartłomiej Żołnierkiewicz" <b.zolnierkie@samsung.com>
Subject: Re: [PATCH] clk: samsung: Allow modular build of the Audio Subsystem CLKCON driver
Date: Tue, 19 Jul 2016 11:55:41 +0200	[thread overview]
Message-ID: <578DF91D.8050800@samsung.com> (raw)
In-Reply-To: <CAJKOXPc3NqsLjEdbs3o1vwaKWCsxK5HxH9G1pRKpMKLD0QXQGw@mail.gmail.com>

On 07/19/2016 11:26 AM, Krzysztof Kozłowski wrote:
> On Wed, Jul 13, 2016 at 12:23 PM, Sylwester Nawrocki
> <s.nawrocki@samsung.com> wrote:
>> > On 07/08/2016 06:56 PM, Michael Turquette wrote:
>>> >> Quoting Sylwester Nawrocki (2016-07-08 07:15:00)
>>>>> >>> > Any clock dependencies can be properly handled with deferred probing
>>>>> >>> > so we can remove core_initcall and switch to a proper loadable platform
>>>>> >>> > driver module.
>>>>> >>> > This change has been tested on Exynos4412 Odroid U3 based board.
>>>>> >>> >
>>>>> >>> > Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
>>> >>
>>> >> Looks good to me.
>> >
>> > Thanks, I have added this patch to my tree, however at the moment
>> > I have only two patches queued there. Please feel free to apply
>> > this patch directly to clk-next.
> 
> 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.

--
Regards,
Sylwester

  reply	other threads:[~2016-07-19  9:55 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-08 14:15 [PATCH] clk: samsung: Allow modular build of the Audio Subsystem CLKCON driver Sylwester Nawrocki
2016-07-08 16:56 ` Michael Turquette
2016-07-08 16:56   ` Michael Turquette
2016-07-13 10:23   ` Sylwester Nawrocki
2016-07-13 22:00     ` Michael Turquette
2016-07-19  9:26     ` Krzysztof Kozłowski
2016-07-19  9:55       ` Sylwester Nawrocki [this message]
2016-07-21 18:22         ` Sylwester Nawrocki
2016-07-11  8:14 ` Krzysztof Kozlowski

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=578DF91D.8050800@samsung.com \
    --to=s.nawrocki@samsung.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=k.kozlowski.k+lkml@gmail.com \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=mturquette@baylibre.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 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.