All of lore.kernel.org
 help / color / mirror / Atom feed
From: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
To: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Cc: Kevin Hilman <khilman@kernel.org>, Kukjin Kim <kgene@kernel.org>,
	Vivek Gautam <gautamvivek1987@gmail.com>,
	Inki Dae <inki.dae@samsung.com>,
	Andrzej Hajda <a.hajda@samsung.com>,
	"linux-samsung-soc@vger.kernel.org"
	<linux-samsung-soc@vger.kernel.org>,
	Paolo Pisati <p.pisati@gmail.com>,
	Vivek Gautam <gautam.vivek@samsung.com>,
	Tushar Behera <tushar.b@samsung.com>,
	Doug Anderson <dianders@chromium.org>,
	Ajay Kumar <ajaykumar.rs@samsung.com>
Subject: Re: Peach Pi/Pit boot failures in linux-next  (was Re: [RFC PATCH 1/1] drm/exynos: Move platform drivers registration to module init)
Date: Mon, 24 Nov 2014 14:16:29 +0100	[thread overview]
Message-ID: <54732FAD.6070608@collabora.co.uk> (raw)
In-Reply-To: <1416827525.5023.10.camel@AMDC1943>

[adding Tushar Behera and Doug Anderson to cc list]

Hello,

On 11/24/2014 12:12 PM, Krzysztof Kozlowski wrote:
> On pon, 2014-11-24 at 12:07 +0100, Javier Martinez Canillas wrote:
>> Hello Krzysztof,
>> 
>> > It seems that mau_epll has to be enabled... or something is wrong with
>> > clock hierarchy.
>> >
>> 
>> Another strange thing is that the problem does not happen for some people
>> using the same board, kernel and config options. For example Vivek and Ajay
>> report that they can't reproduce the issue on a Peach Pi using next-20141121
>> and exynos_defconfig without using clk_ignore_unused.
> 
> Maybe they have different bootloader which messes here by enabling some
> clock?
> 
> Anyway it is reproducible on at least some Arndale Octa (Kevin's and
> mine) and Peach Pi boards (yours).
> 

This issue started to look extremely familiar to me so I searched in
my mail inbox and found that the same problem was previously reported
by Kevin a couple of months ago [0] and Tushar provided a fix [1].

I tested linux-next + [1] and that indeed fixes the hang on Peach.

To save you a click, the problem as explained by Tushar is that the
AUDSS mux has two parents: XXTI crystal and MAU_EPLL clock. But when
the output of AUDSS mux is gated, no operations can be made on the
clocks provided by MAU block. For some reason the kernel just oops
so it seems to be a H/W errata?

Mike was not fond about the solution proposed in [1] but something
along those lines would be needed maybe Tushar can comment on that.

Vivek and Ajay,

As explained in [0], you are not facing this issue because your RW
U-Boot seems to predate when audio support was enabled by default.

Can you try executing "sound init" in the U-Boot prompt and see if
that triggers the hang for you?

Best regards,
Javier

[0]: http://lists.infradead.org/pipermail/linux-arm-kernel/2014-June/262259.html
[1]: http://www.spinics.net/lists/arm-kernel/msg337970.html

  reply	other threads:[~2014-11-24 13:16 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-18 13:53 [RFC PATCH 1/1] drm/exynos: Move platform drivers registration to module init Javier Martinez Canillas
2014-11-18 18:41 ` Kevin Hilman
2014-11-18 20:28   ` Gustavo Padovan
2014-11-18 22:46     ` Kevin Hilman
2014-11-19 10:09       ` Javier Martinez Canillas
2014-11-19 16:52   ` Javier Martinez Canillas
2014-11-19 19:52     ` Kevin Hilman
2014-11-19 22:29       ` Kevin Hilman
2014-11-20  7:06     ` Vivek Gautam
2014-11-20  7:51       ` Vivek Gautam
2014-11-20  8:45         ` Javier Martinez Canillas
2014-11-20  9:52           ` Vivek Gautam
2014-11-20 14:24             ` Pankaj Dubey
2014-11-20 15:57             ` Paolo Pisati
2014-11-20 16:44               ` Javier Martinez Canillas
2014-11-20 16:41       ` Kevin Hilman
2014-11-20 17:47         ` Javier Martinez Canillas
2014-11-20 18:22           ` Kevin Hilman
2014-11-20 23:49             ` Paolo Pisati
2014-11-21 11:33               ` Andreas Färber
2014-11-21 17:32                 ` Ajay kumar
2014-11-21 20:57                   ` Javier Martinez Canillas
2014-11-24 10:05                     ` Javier Martinez Canillas
2014-11-24 10:36                       ` Vivek Gautam
2014-11-24 15:05                       ` Andreas Färber
2014-11-25  5:35                         ` Ajay kumar
2014-11-21 13:03             ` Peach Pi/Pit boot failures in linux-next (was Re: [RFC PATCH 1/1] drm/exynos: Move platform drivers registration to module init) Javier Martinez Canillas
2014-11-21 16:38               ` Kevin Hilman
2014-11-21 20:49                 ` Javier Martinez Canillas
2014-11-22 10:21                   ` Krzysztof Kozlowski
2014-11-24  9:38                     ` Javier Martinez Canillas
2014-11-24  9:42                       ` Javier Martinez Canillas
2014-11-24 10:13                       ` Krzysztof Kozlowski
2014-11-24 11:07                         ` Javier Martinez Canillas
2014-11-24 11:12                           ` Krzysztof Kozlowski
2014-11-24 13:16                             ` Javier Martinez Canillas [this message]
2014-11-24 13:28                               ` Vivek Gautam
2014-11-24 13:53                               ` Krzysztof Kozlowski
2014-11-24  9:51                   ` Krzysztof Kozlowski
2014-11-20 14:07 ` [RFC PATCH 1/1] drm/exynos: Move platform drivers registration to module init Inki Dae
2014-11-20 14:28   ` Javier Martinez Canillas
2014-11-20 15:06     ` Inki Dae
2014-11-20 15:26       ` Javier Martinez Canillas
2014-11-20 17:01         ` Inki Dae
2014-11-21 11:19           ` Javier Martinez Canillas

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=54732FAD.6070608@collabora.co.uk \
    --to=javier.martinez@collabora.co.uk \
    --cc=a.hajda@samsung.com \
    --cc=ajaykumar.rs@samsung.com \
    --cc=dianders@chromium.org \
    --cc=gautam.vivek@samsung.com \
    --cc=gautamvivek1987@gmail.com \
    --cc=inki.dae@samsung.com \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene@kernel.org \
    --cc=khilman@kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=p.pisati@gmail.com \
    --cc=tushar.b@samsung.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.