From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.thorsis.com (mail.thorsis.com [217.92.40.78]) (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 0BF213FB7F4 for ; Wed, 29 Apr 2026 13:02:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.92.40.78 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777467764; cv=none; b=A4t7TMcDrl9/ucBTb6H+zyxSq7enoqUuCmFQD4i4Ncb/cgDbjIXVTiM+JcLEtZDs+hftybAxwb4CXRwTtorwdpsuRn9N8LONpWDINh2+XrLTmNW45S7Zc6OYGucLyK6KmroXdL2TzDFcHaFMgjk6D+4DB2HQvU3WjhQr51RaS7I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777467764; c=relaxed/simple; bh=969MOG2iH703ThHJK4rehtrZT5hFIPq8tqU7RbcIMxg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=MOZqxlliQ+HjYAjcr8A0Ww10xt6GfHL4cjuogrwM3R/Vk0MkW+0hot5SwJ80ko0DzkrxbY2EB2qqpaOzORFmBFXDUNSX62hDuCDGs67C4t6qmxwaZq71+JEwodDxrQd2beap22dR0KA3uVtgFXils/pRzIrxX4b83pHFbfviTgE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=thorsis.com; spf=pass smtp.mailfrom=thorsis.com; dkim=pass (2048-bit key) header.d=thorsis.com header.i=@thorsis.com header.b=X1aSrosw; arc=none smtp.client-ip=217.92.40.78 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=thorsis.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=thorsis.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=thorsis.com header.i=@thorsis.com header.b="X1aSrosw" Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 37C9C14841EA; Wed, 29 Apr 2026 15:02:23 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=thorsis.com; s=dkim; t=1777467747; h=from:subject:date:message-id:to:cc:mime-version:content-type: in-reply-to:references; bh=Rz5+8HKBObbUzWkRkJLrMIZCYQXi9v58Y1B9PohQInM=; b=X1aSroswucp7gpg/zRyamurUe94FEatr3CkLFyVcwVK8gdcpvPjF/sFtGyhB+90ndxnXHT 8HZei4yi9Rh3wWQ9uq1KfQ2bhhcAy05niuSE7x8/ArSfCN94bs38t28rLlnynXiwDdtb6X Wctigieia+1vIdAo/UueqJN8i1pYoA03yQ+toNriBSz0BQUKmVEoI8HJaRO/ZY5HTd7cX0 czQwW6LdQBbRtBWtTy9H+9WCsuX2n4p1ECaT7wzDWTUqmhzRGoABZPohs8QqWgZUd8l4yu 6i8kIy1i6aZgkp5Y6kZ0zI4LpMsdzNWqX53xcSWpuwCGF1y90Ib+awmsPeG5Tg== Date: Wed, 29 Apr 2026 15:02:22 +0200 From: Alexander Dahl To: Miquel Raynal Cc: "Rob Herring (Arm)" , Richard Weinberger , Vignesh Raghavendra , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Russell King , linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 3/3] ARM: at91: remove unnecessary of_platform_default_populate calls Message-ID: <20260429-blazing-thus-c1a4eb9b6b0c@thorsis.com> Mail-Followup-To: Miquel Raynal , "Rob Herring (Arm)" , Richard Weinberger , Vignesh Raghavendra , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Russell King , linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20260105-at91-probe-v3-0-594013ff2965@kernel.org> <20260105-at91-probe-v3-3-594013ff2965@kernel.org> <20260423-stoke-ocean-139dbe306256@thorsis.com> <20260424-worried-renewal-924d34ed945c@thorsis.com> <87tsswfnmy.fsf@bootlin.com> <20260429-lushly-greasily-c14c01c1aee8@thorsis.com> <87lde6cdc4.fsf@bootlin.com> 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: <87lde6cdc4.fsf@bootlin.com> User-Agent: Mutt/2.2.13 (2024-03-09) X-Last-TLS-Session-Version: TLSv1.3 Hello Miquel, Am Wed, Apr 29, 2026 at 11:42:19AM +0200 schrieb Miquel Raynal: > Hi Alexander, > > > [ 0.958298] bus: 'platform': add driver atmel-ebi > > [ 0.958451] platform 10000000.ebi: bus: 'platform': __driver_probe_device: matched device with driver atmel-ebi > > [ 0.958566] platform 10000000.ebi: error -EPROBE_DEFER: wait for supplier /ahb/apb/pinctrl@fffff400/ebi/ebi-data-lsb > > [ 0.958649] platform 10000000.ebi: Added to deferred list > > [ 0.959429] bus: 'platform': remove driver atmel-ebi > > [ 0.959540] driver: 'atmel-ebi': driver_release > > Interesting. > > If you look at the very last line of the driver, the EBI driver does not > use a standard "module_platform_driver" macro, it uses some kind of > run-once probe that is kept in an __init section: > https://elixir.bootlin.com/linux/v7.0.1/source/drivers/memory/atmel-ebi.c#L637 > > And this seems to be incompatible with probe deferrals: > https://elixir.bootlin.com/linux/v7.0.1/source/drivers/base/platform.c#L948 Indeed. o.O > I would suggest trying something alone these lines: > > --- a/drivers/memory/atmel-ebi.c > +++ b/drivers/memory/atmel-ebi.c > @@ -633,5 +633,6 @@ static struct platform_driver atmel_ebi_driver = { > .of_match_table = atmel_ebi_id_table, > .pm = &atmel_ebi_pm_ops, > }, > + .probe = atmel_ebi_probe, > }; > -builtin_platform_driver_probe(atmel_ebi_driver, atmel_ebi_probe); > +builtin_platform_driver(atmel_ebi_driver); > > Memory taken by the probe function will remain mapped. That seems to be > the drawback. This works. Sent a patch, see: https://lore.kernel.org/all/20260429125930.844790-1-ada@thorsis.com/ Regarding the drawback: This can't be much memory, can it? Greets Alex