From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtpout-04.galae.net (smtpout-04.galae.net [185.171.202.116]) (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 F09D72DF6F4 for ; Mon, 27 Apr 2026 15:07:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.171.202.116 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777302428; cv=none; b=KK2tWdZ6DnAGNl7uMhqaWzvWA98oSivonkDgWs3h+EEum7H406rP6no4cdWaEM6QSf0wjSWbvWPQQhTxH30lsP90dWhIFzjD9M/5zW/xPmxkSjC4MR2MHOdvPYsl5i/g6M7/qWpFW2vxoMU6X5ffMuQSbnMUZ8SPAxDwe+Tra8g= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777302428; c=relaxed/simple; bh=aSBQLer43zFv4KL0AHMgdVvTArQUfNZa9rzJSyHzL7A=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=PgyHAuTLQKAzcB+eC1E9exH2oaG9Bp2HN3Qnhb0/y7+5UI0iMuIusMzEqMYVIZhQiTX8TUhZ3Fm104KG7SQCcjBt+UBK+tbCjAlSbQ5Z7rUezFAN04VzR6cR2HMpff7qIdNocbHU4eGSTl1dKDgDZWoUM1nmSNLIg46p4rpxpE4= 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=iPoYCp2Q; arc=none smtp.client-ip=185.171.202.116 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="iPoYCp2Q" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-04.galae.net (Postfix) with ESMTPS id 6D117C5CD4B; Mon, 27 Apr 2026 15:07:47 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 155A0600D1; Mon, 27 Apr 2026 15:07:04 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id CD5101072822D; Mon, 27 Apr 2026 17:06:59 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1777302423; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=gMNoBFW05PGN89Lx84+WtOyYzlPOjlTQR1HX96yjry0=; b=iPoYCp2Q3BZkyngYDlFdBLey+pjeV/cHso3bV3Ev9SH7ZwK/Ph+ThDhIUdsWAjIAidna+P 3ehE4OUWBkBeKBsPept5h7+UjFJzC43vdRtPcS96/UMSKc6bI+/tMOrjn24RH0o/Qh8kSD ZYtvR1xkxPRrY4Nsl/81kzq3IhHUryb2uOY5HyGXnLnAwdi1FbIjmI/uHhWICwuCBTm/kb yb7adJu6DrK7iNdh/R+YnenQBw4gou837NPLjsNfQOhpCepJ2/IBELq39GzhJcDWDA4TzT +Ya6hjdPjMRUOuKiiMpXfEvb7bzdeKRXDuUGyynMQiloNDU44Rg6DhKNUTjYBA== From: Miquel Raynal To: "Rob Herring (Arm)" Cc: 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 In-Reply-To: <20260424-worried-renewal-924d34ed945c@thorsis.com> (Alexander Dahl's message of "Fri, 24 Apr 2026 12:56:27 +0200") 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> User-Agent: mu4e 1.12.7; emacs 30.2 Date: Mon, 27 Apr 2026 17:07:01 +0200 Message-ID: <87tsswfnmy.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=utf-8 Content-Transfer-Encoding: quoted-printable X-Last-TLS-Session-Version: TLSv1.3 Hi Alexander, On 24/04/2026 at 12:56:27 +02, Alexander Dahl wrote: > Hei hei, > > after few hints in IRC yesterday, I tried to understand why neither > the ebi driver nor the nand driver are probed, but I failed. See > below. Just to be clear, I would not expect the NAND driver to probe "alone", it is described as a child node of the EBI controller which has its own compatible. As a result, only the of_platform_populate() at the end of the probe of the EBI can lead to the NAND controller to probe. The EBI node being a child node of a "simple-bus", this is the one we should focus on, because it should be probed. One reason (trying to be creative) could the that Rob's patch is dropping an explicit populate that maybe kind of bypasses checks that the "official" populate does. So maybe there is one resource that is missing and which is not ignored as it used to be by the core device driver (likely, dd.c). Can you enable CONFIG_DEBUG_DRIVER and see in the logs if anything pops up? Maybe trying to trace (manually) in the core why we do not attempt to probe the EBI controller by looking for possible conditions to bail out early. Pinctrl is one of them, so maybe just removing all pinctrl references in the DT may help troubleshooting this (obviously probe will fail if pinctrl is incorrect, but if it is attempted we will have a culprit). Good luck, Miqu=C3=A8l