From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1541993608.30437.3.camel@mtkswgap22> Subject: Re: [PATCH v1 3/3] arm64: dts: mt7622: Drop the general purpose timer node From: Ryder Lee To: Sasha Levin CC: Rob Herring , Matthias Brugger , Weijie Gao , , , , , Date: Mon, 12 Nov 2018 11:33:28 +0800 In-Reply-To: <20181112024556.GI2642@sasha-vm> References: <30f4a4110190a175307ac4dfebc98ffe96c44b90.1541643419.git.ryder.lee@mediatek.com> <17fde639c7d37f9cfabab230379913e652cf869a.1541643419.git.ryder.lee@mediatek.com> <20181112024556.GI2642@sasha-vm> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 List-ID: On Sun, 2018-11-11 at 21:45 -0500, Sasha Levin wrote: > On Mon, Nov 12, 2018 at 09:28:08AM +0800, Ryder Lee wrote: > >The crash http://termbin.com/zitb is caused by the timer register > >into system in early pahse during kernel boot, but the clock > >sources didn't get ready at that time. > > > >A better way is to switch to use CLK_OF_DECLARE() in driver for things > >that need them early, but this node is actually useless in MT7622. > >So we drop it. > > > >Fixes: 9cc7f0de9e67 ("arm64: dts: mt7622: add timer, CCI-400 and PMU nodes") > >Cc: stable@vger.kernel.org > >Signed-off-by: Ryder Lee > > To confirm: the patch this fixes was added in v4.20-rc1, which means > that this current patch doesn't apply to any of the stable trees. > > Did we miss something? > I didn't see any fixup for this? Ryder