From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtpout-02.galae.net (smtpout-02.galae.net [185.246.84.56]) (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 868353E7BCE for ; Fri, 17 Jul 2026 13:10:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.246.84.56 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784293833; cv=none; b=sQ5kH5zU9J2tAcKPrYyxFqk5rhNbCFxxv+x7Qf4I0lzjXyyhwyRGkaStmrsMHvk7IYuyEp6FOIgBJrjHJAwYQyy8vBvTXdaNvvX9S2/r6ZRkTlzjZEEhC8g1jYmid8/QFFLCqUUrm7p7OphD92p6toCWgfvHp0ow3gHU7uCo0HY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784293833; c=relaxed/simple; bh=ZA5A1RfyXw1NcBcIE6GQxTFOitqcRCcOtZAix7uUu8I=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=dcpC8BXirKYwlvunj/id/r5NbYccnOp836Ud6ouLBZXIITRQJ8v1hwcSh0uy5NAJ0nEZAS67Ig5lafznMFMfdi+AKtgT56N8ACEtnC4GVJ1f4RES242kKMFYy7zwYzfr8KBjTZh3Sr2tHJdaRVfZ/Egt5UTHxwCPf/jTwPfa4s8= 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=T56X3ZEp; arc=none smtp.client-ip=185.246.84.56 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="T56X3ZEp" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-02.galae.net (Postfix) with ESMTPS id E43C21A1050; Fri, 17 Jul 2026 13:10:28 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id B815660361; Fri, 17 Jul 2026 13:10:28 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id ECD9911BD0158; Fri, 17 Jul 2026 15:10:26 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1784293828; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=ZA5A1RfyXw1NcBcIE6GQxTFOitqcRCcOtZAix7uUu8I=; b=T56X3ZEp6kjp8s5bdSWKrXi8BFivp/bxG232PAvB7Jzzi3xZSCN/PJPd9ppxuyRzTbuZvZ HG/jq9OqRhJFrWmgXvLTIKyDKQTd8WLq8nltp+zTdGTxtbut7V1em045SC4rCJqtMPjfJS RoYUPFlHGeJbposcOLoDUylHilVmrryWHkGJQycbLdTbxLXNfCV684NMvsQJjpj8Rs+bB+ NABWLhlwqx3cDG4Vk663BLr0+2M1DqHNA2ZgKYDQxpDGJh9aUCyzSbSgd9WiKVJrogyuMY 6aICRLbGUV9rx85r6IRkQBPoOmWsVUHEl+y/n6GeqyL93PILTlV5bUd0RDlDmg== From: Miquel Raynal To: Aleksandr Mineev Cc: Richard Weinberger , Vignesh Raghavendra , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] mtd: spinand: fmsh: fix FM25G01B/FM25G02B Quad I/O read dummy cycles In-Reply-To: <20260715203602.1462-1-sanderrrs@gmail.com> (Aleksandr Mineev's message of "Wed, 15 Jul 2026 23:36:02 +0300") References: <20260715203602.1462-1-sanderrrs@gmail.com> User-Agent: mu4e 1.12.7; emacs 30.2 Date: Fri, 17 Jul 2026 15:10:26 +0200 Message-ID: <87pl0liwxp.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 Aleksandr, On 15/07/2026 at 23:36:02 +03, Aleksandr Mineev wrote: > The FM25G01B/FM25G02B datasheets specify a single dummy byte for the > 0xEB Quad I/O read-from-cache operation, but the generic > read_cache_variants set uses two dummy bytes for the 1S-4S-4S variant. > The extra dummy byte shifts the data phase and returns corrupted data > with no ECC error, breaking boot on boards using these chips. > > Use a dedicated read-from-cache variant set with ndummy=3D1 for the > 1S-4S-4S (0xEB) operation. > > FM25G01B datasheet: https://www.fmsh.com/nvm/FM25G01B_ds_eng.pdf > FM25G02B datasheet: https://www.fmsh.com/nvm/FM25G02B_ds_eng.pdf > > Fixes: d5a5c9eb2ee9 ("mtd: spinand: fmsh: add support for FM25G{01,02}B") Missing Cc: stable tag. > Signed-off-by: Aleksandr Mineev Otherwise LGTM. Thanks, Miqu=C3=A8l