Re: [linux-sunxi] [PATCH v2 3/6] ARM: sun4i: Convert to CCU

From: Kevin Hilman <khilman@baylibre.com>
To: Priit Laes <plaes@plaes.org>
Cc: Chen-Yu Tsai <wens@csie.org>,
	Maxime Ripard <maxime.ripard@free-electrons.com>,
	lkml <linux-kernel@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
	devicetree <devicetree@vger.kernel.org>,
	linux-clk@vger.kernel.org, linux-sunxi@googlegroups.com,
	Icenowy Zheng <icenowy@aosc.xyz>,
	Russell King <linux@armlinux.org.uk>,
	Mark Rutland <mark.rutland@arm.com>,
	Rob Herring <robh+dt@kernel.org>,
	Stephen Boyd <sboyd@codeaurora.org>,
	Michael Turquette <mturquette@baylibre.com>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Olof Johansson <olof@lixom.net>
Subject: Re: [linux-sunxi] [PATCH v2 3/6] ARM: sun4i: Convert to CCU
Date: Fri, 5 Jan 2018 08:10:54 -0800	[thread overview]
Message-ID: <CAOi56cXTrGg+mx4aJsrMTHcwAV4KLCUZDy9bjpAq1hi1gngKAg@mail.gmail.com> (raw)
In-Reply-To: <CAOi56cX0Uw=jiAhNSZ8p8_1LPbd7zePkS5kTnMf+ua0v98zfVg@mail.gmail.com>

On Wed, Dec 13, 2017 at 11:46 AM, Kevin Hilman <khilman@baylibre.com> wrote:
> On Wed, Dec 13, 2017 at 9:13 AM, Priit Laes <plaes@plaes.org> wrote:
>> On Wed, Dec 13, 2017 at 05:09:33PM +0000, Priit Laes wrote:
>>> On Tue, Dec 12, 2017 at 01:24:52PM -0800, Kevin Hilman wrote:
>>> > On Tue, Dec 12, 2017 at 9:26 AM, Priit Laes <plaes@plaes.org> wrote:
>>> > > On Mon, Dec 11, 2017 at 02:22:30PM -0800, Kevin Hilman wrote:
>>> > >> On Sun, Mar 26, 2017 at 10:20 AM, Priit Laes <plaes@plaes.org> wrote:
>>> > >> > Convert sun4i-a10.dtsi to new CCU driver.
>>> > >> >
>>> > >> > Signed-off-by: Priit Laes <plaes@plaes.org>
>>> > >>
>>> > >> I finally got around to bisecting a mainline boot failure on
>>> > >> sun4i-a10-cubieboard that's been happening for quite a while.  Based
>>> > >> on on kernelci.org, it showed up sometime during the v4.15 merge
>>> > >> window[1].  It bisected down to this commit (in mainline as commit
>>> > >> 41193869f2bdb585ce09bfdd16d9482aadd560ad).
>>> > >>
>>> > >> When it fails, there is no output on the serial console, so I don't
>>> > >> know exactly how it's failing, just that it no longer boots.
>>> > >
>>> > > We tried out latest 4.15 with various compilers and it works:
>>> > > - gcc version 7.1.1 20170622 (Red Hat Cross 7.1.1-3) (GCC) - A10 Gemei G9 tablet
>>> > > - gcc 7.2.0-debian - A10 Cubieboard
>>> >
>>> > And you can reproduce the bug with gcc5 or gcc6?
>>>
>>> Tried following commits on Gemei G9 (A10 tablet):
>>> * 4.15.0-rc3-00037-gd39a01eff9af - latest master
>>> * 4.14.0-rc1-00002-g41193869f2bd - the exact commit, causing the issue.
>>>
>>> With the same Linaro toolchain:
>>> (gcc version 5.3.1 20160412 (Linaro GCC 5.3-2016.05))
>>
>> And I also tried the same dtb and zImage from kernelci page [1] and it works with
>> that too...
>>
>> https://storage.kernelci.org/mainline/master/v4.15-rc3/arm/sunxi_defconfig/
>
> Can you share a full boot-log (including all the u-boot output etc.)
> so I can see exactly how the kernel is being loaded?    Especially the
> u-boot version?
>
> As $SUBJECT patch seems to be changing clocks around, perhaps this is
> an issue where some u-boot dependency is uncovered, and older versions
> of u-boot don't play well with this change.

Ping.

This is still failing in mainline, but passing int stable <= v4.14

Kevin