From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH 2/2] soc: bcm: brcmstb: Add missing DDR MEMC compatible strings Date: Mon, 4 Jun 2018 14:15:46 -0700 Message-ID: <20180604211546.21349-1-f.fainelli@gmail.com> References: <20180511220242.837-1-f.fainelli@gmail.com> <20180511220242.837-3-f.fainelli@gmail.com> Return-path: In-Reply-To: <20180511220242.837-3-f.fainelli@gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: bcm-kernel-feedback-list@broadcom.com, linux-arm-kernel@lists.infradead.org Cc: Rob Herring , Mark Rutland , Brian Norris , Gregory Fong , Doug Berger , Justin Chen , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list List-Id: devicetree@vger.kernel.org On Fri, 11 May 2018 15:02:42 -0700, Florian Fainelli wrote: > We would not be matching the following chip/compatible strings > combinations, which would lead to not setting the warm boot flag > correctly, fix that: > > 7260A0/B0: brcm,brcmstb-memc-ddr-rev-b.2.1 > 7255A0: brcm,brcmstb-memc-ddr-rev-b.2.3 > 7278Bx: brcm,brcmstb-memc-ddr-rev-b.3.1 > > The B2.1 core (which is in 7260 A0 and B0) doesn't have the > SHIMPHY_ADDR_CNTL_0_DDR_PAD_CNTRL setup in the memsys init code, nor > does it have the warm boot flag re-definition on entry. Those changes > were for B2.2 and later MEMSYS cores. Fall back to the previous S2/S3 > entry method for these specific chips. > > Fixes: 0b741b8234c8 ("soc: bcm: brcmstb: Add support for S2/S3/S5 suspend states (ARM)") > Signed-off-by: Florian Fainelli > --- Applied to drivers/next, thanks! -- Florian