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 7A6C731ED7D for ; Mon, 6 Jul 2026 13:45:25 +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=1783345530; cv=none; b=MaQwUneKgFjHI/Z3+xHb1KCvbif3eU0juxAiueLNTGYpdP0hMNIVRoZ0WDuIG0aIPthjGrPRBPB/bZadjVQfhVOdI7DFSylcwBljCsypTGCM+8e6z3j+bK/F/IbMgpzUdUbMzZlPfvCvIn0SNh0XWI9XAbnNUsc5Ubkxy/erH6w= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783345530; c=relaxed/simple; bh=9Ve2U/uhV+HCAQSBqxYMQ94zcSk2C16Cki5oPDKfUYU=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=BtVVFU2z4Zc0v3wZ6acLvT1OwptgbYCznM4dagF5BP40KO/SQGJC4BJE8t3thcH/c52J1kmqj0Pl86yMiyNFxuVsMk3pn17Cmkc9iniVGCe/Md2xSYtqIrIXTuIDoSn4jiMFciIU0Zgx/JmUbSRDdDz2rsV82d3q2y27+7HWnsA= 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=NMHrrFun; 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="NMHrrFun" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-03.galae.net (Postfix) with ESMTPS id 8E8924E40CB2; Mon, 6 Jul 2026 13:45:22 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 4D91F601A2; Mon, 6 Jul 2026 13:45:22 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 9E84D11BB96B6; Mon, 6 Jul 2026 15:45:19 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1783345521; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=MOT9NILUsJfEsKS9HNrbXmBhyVr3vywo7d2Dyi6BPHE=; b=NMHrrFunwGIV3DI6HTWpfRLYEm2k39hTTb9xiBvNkha6VmePhBWCPIGtUcvSEhqZ9uVt+v GhmsCbGJoXG1wckFjQ54dNa1An5m9bReDstio7kBPlIdRTXYt9cxAt8lzG9VmljO35YW06 bV9i/SB28/emS34Fsbpwtump0Ls0xyT5nS4R8rYcwmppypKdYoDj2RSvLdXuX9Wohteg/9 MnYkdojMQswOHH0ARMu6u8IocYg7FyM5k7BbNtndwv7fJo/TbMcqR5i+9pJMINgSZrDVIB i3/zAufO6t4axAzqebhByzns/dRB7LETHzVhP5sif7Qu7q2TGIurqTSgOzzeAQ== From: Miquel Raynal To: "Michael Walle" Cc: "Pratyush Yadav" , "Takahiro Kuwano" , "Richard Weinberger" , "Vignesh Raghavendra" , "Tudor Ambarus" , , Subject: Re: [RFC PATCH 0/3] mtd: spi-nor: Rework flash parameter initialization In-Reply-To: (Michael Walle's message of "Mon, 06 Jul 2026 09:33:38 +0200") References: <20260601125438.3481722-1-mwalle@kernel.org> <178331904816.868671.4599503585231715805.b4-ty@bootlin.com> User-Agent: mu4e 1.12.7; emacs 30.2 Date: Mon, 06 Jul 2026 15:45:19 +0200 Message-ID: <87zf04kz9s.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 Hello, On 06/07/2026 at 09:33:38 +02, "Michael Walle" wrote: > On Mon Jul 6, 2026 at 8:24 AM CEST, Miquel Raynal wrote: >> On Mon, 01 Jun 2026 14:52:42 +0200, Michael Walle wrote: >>> Try to simplify the flash initialization and get rid of the legacy >>> handling. As default, all the flags of the in-kernel database are >>> taken and amended with the SFDP data. >>>=20 >>> This might have the consequence that all the flashes now get a >>> RDSFPD opcode which might be an unknown opcode. But that was already >>> the case for any flashes which were unknown to the linux kernel. So >>> far, there was not a single complaint. >>>=20 >>> [...] >> >> Applied to mtd/fixes, thanks! >> >> [1/3] mtd: spi-nor: spansion: s25fl256s0: remove SKIP_SFDP flag >> (no commit info) >> [2/3] mtd: spi-nor: don't clear the SNOR_F_4B_OPCODES flag on failure >> (no commit info) >> [3/3] mtd: spi-nor: rework flash parameter initialization >> (no commit info) >> >> Patche(s) should be available on mtd/linux.git and will be >> part of the next PR (provided that no robot complains by then). > > Did you see, that there was a v1 of this series? There wasn't many > changes for these three patches. Just what Tudor mentioned that the > unsupported RDSFDP opcode should be mentioned in the commit message, > too. > > Apart from that, the series now includes fixes that Sashiko found, > that is, the rollback in case of a failed SFDP parsing is broken. I > can also rebase that on top of these three patches. Sorry, I actually applied it last Friday and pushed only this morning. I will drop the patches and apply v1. Thanks, Miqu=C3=A8l