From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtpout-03.galae.net (smtpout-03.galae.net [185.246.85.4]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C7BD4AD51 for ; Fri, 12 Dec 2025 17:37:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.246.85.4 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765561079; cv=none; b=IiRzFfbfYoheEG5bzotxRuIoBSVsfl9soP2r1ijsrhQi4QvX0s+pQTGmFWptyZwLsDeXjI1gveQ44O8YNbFXkDyImCHv3U7kIODlPRwLZhyXa6so6ZrrQ5NBzDuoEbcBqhe6ibOOKw27eamgWvNAcFoTIPfK6P/+L+lSe01GKks= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765561079; c=relaxed/simple; bh=1cOAUg5SIveyydcSb1vsJ90D70W9bb4EfL31zmWJsY0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=MsRJMInLMG/IvOq2PfOaDODPEawLaEkasguNwKsP9DpHrjcRUGiaUhtGOgzViR8gtJ/N47RVRlhYr20ztApqQw4n6iQOxl/AHpi8ImDDg5alCoO2ib7/l4SOv5R9zl5aYPWQzfl3ASZZJc5uA4qdMHBTnC2D0u79+zCXQDs0JXE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=ONA4zBLS; arc=none smtp.client-ip=185.246.85.4 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="ONA4zBLS" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-03.galae.net (Postfix) with ESMTPS id 6F8B44E41BB4; Fri, 12 Dec 2025 17:37:48 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 3824A606DF; Fri, 12 Dec 2025 17:37:48 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id CBEF0103C8DEA; Fri, 12 Dec 2025 18:37:44 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1765561067; h=from:subject:date:message-id:to:cc:mime-version:content-type: in-reply-to:references; bh=+2hXAzMg48p6R/oXqTisskWAXxrhQhyyLNcBJNRfGbw=; b=ONA4zBLST0UPgrjhOK1FkmRyKgcNYBVB39ZgJDyVXUFHYUmaL4k7v0LSjjqDeXH+0DhZlN NgVJTCSVmvnygDvT4Iq4aYa520G0xW5iggHttbxBT+X7X4is3qhZNUas/YNB+UUg0vgymA Rt8GcKYWANQRWjlafDefvLVoTwMpcTJ1b4jEWnK01qw827ThvCvEI9/7XxHkQvTIREdF1g pTQgyI0VXbEOXN79EC8cawAE32gEIcQDlErxtkuPC62CNoWMomBXkvvNilM38f2vpicnIA byBPfFNaqPVuUp3bkW+SLcjLpC6ZsYu0g4tn32qSZd8VgvQfrvaeT1WJL3Yklg== Date: Fri, 12 Dec 2025 18:37:44 +0100 From: Alexandre Belloni To: "Rob Herring (Arm)" Cc: Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Nicolas Ferre , Claudiu Beznea , Russell King , linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 0/3] at91: Remove of_platform_default_populate() calls Message-ID: <202512121737447cdf2224@mail.local> References: <20251120-at91-probe-v2-0-8c8592f02e8e@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20251120-at91-probe-v2-0-8c8592f02e8e@kernel.org> X-Last-TLS-Session-Version: TLSv1.3 On 20/11/2025 15:37:49-0600, Rob Herring (Arm) wrote: > This is another attempt at removing of_platform_default_populate() calls > on at91 platforms. The last attempts were in 2018[1] and 2023[2]. The > problems reported are PM and NAND failures. Both of these depend on SRAM > pool which doesn't have any fw_devlink tracking. Also, the PM code is > not a driver, so fw_devlink wouldn't help. For the NAND driver, we can > simply defer probe. For the PM init, we can move it to the .late_init > hook. > > Rob > > [1] https://lore.kernel.org/all/20180712192222.32481-1-robh@kernel.org/ > [2] https://lore.kernel.org/all/3908fa6f-dc33-4aac-c21f-4a676a3a6b23@microchip.com/ > > Signed-off-by: Rob Herring (Arm) For the whole series: Acked-by: Alexandre Belloni Tested-by: Alexandre Belloni I tested on at91rm9200, at91sam9m10 and sama5d3 which is a representative sample of the older platforms. I tested suspend/resume successfully on at91sam9m10 and sama5d3. at91rm9200 is not waking up but this is not caused by the series. > --- > Rob Herring (1): > ARM: at91: Move PM init functions to .init_late hook > > Rob Herring (Arm) (2): > mtd: nand: atmel: Defer probe if SRAM is missing > ARM: at91: remove unnecessary of_platform_default_populate calls > > arch/arm/mach-at91/at91rm9200.c | 12 +----------- > arch/arm/mach-at91/at91sam9.c | 12 +----------- > arch/arm/mach-at91/sam9x60.c | 12 +----------- > arch/arm/mach-at91/sam9x7.c | 12 +----------- > arch/arm/mach-at91/sama5.c | 21 +++------------------ > arch/arm/mach-at91/sama7.c | 11 +---------- > drivers/mtd/nand/raw/atmel/nand-controller.c | 2 +- > 7 files changed, 9 insertions(+), 73 deletions(-) > --- > base-commit: 3a8660878839faadb4f1a6dd72c3179c1df56787 > change-id: 20251120-at91-probe-20fbce9a6d1b > > Best regards, > -- > Rob Herring (Arm) > -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com