From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay8-d.mail.gandi.net (relay8-d.mail.gandi.net [217.70.183.201]) (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 4EAE020DD74 for ; Fri, 10 Jan 2025 14:46:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736520403; cv=none; b=OXiwrw4Wda8j84PBz9XDnU8osumgpxlN5XJaZvVR3vJe7wuzNdznE16gDNEDnZeg5bjlzs4b4uU7YBJIbDHNPUcvq7Av/m/8kduQY+454bNE7p6HKuqpQGI4SrGSDZkjly96+TsaNJHRYBZC/b4tdTTWSpixQPe40xWB4PYG78c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736520403; c=relaxed/simple; bh=1mbfo4mprdY88fC5FUdk6yylv2/Z+C7SPsz1OAPkfmI=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=ivAJkth5g9RS3adPSWwPLykUc24yNJRPcibWQzdunFdzz5F/RYWbjgjmcpLNKoyO/wBfAz42Y67fbItm0Y9eIxR3oAlMqpsbfAgQ7fqjEEa4YDzCU5h59Z+GygAHCfLH6Y+vwCgoq9Zquvx7P5goQ4ChcyON0Rwz9UWKy9r/pEM= 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=I36JP0l8; arc=none smtp.client-ip=217.70.183.201 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="I36JP0l8" Received: by mail.gandi.net (Postfix) with ESMTPSA id 60ED21BF209; Fri, 10 Jan 2025 14:46:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1736520394; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Lq1eamULlCmRyHyCPksQXtjcgP0Gfwj+Ra+BImrmBcA=; b=I36JP0l8C3qWbjoakKvRungkRnThzZAPs29qRxzq/O7yjjaX3N3osUbmGLZeExN1HOPfhX 6OrDTrjVt85P4MX4Zia/kmkWVZis/3uZD7BmOV8YmO3E0bu6FJ4SmfSAwmv/Aex5jaWMBm Rr0aQcp7TTh+wlACobcihnSC0tbli0GvDPjNxEo/KAgDsfzc+pj9ZTL+85JV9btMcYc4a+ K6eKqbyD0pgDvFdtWHXFmWviH8OomEFzU7xAqStVbfne/FvUOciWiN17b1Nt9AN+jPVQxN tdCayUa2dDGLMCaGe9uLnvs10XBgxg3EL92H58UuHlhgfrxc/BomwQyTSsOsyA== From: Miquel Raynal To: Christophe JAILLET Cc: Tudor Ambarus , Pratyush Yadav , Michael Walle , Richard Weinberger , Vignesh Raghavendra , Thomas Petazzoni , Steam Lin , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/2] mtd: spi-nor: winbond: Add support for w25q02jv In-Reply-To: <956f0490-2e47-4655-ab10-a10ef790cfad@wanadoo.fr> (Christophe JAILLET's message of "Thu, 9 Jan 2025 21:23:13 +0100") References: <20250109-winbond-6-12-rc1-nor-volatile-bit-v2-0-e8cd0812fe4e@bootlin.com> <20250109-winbond-6-12-rc1-nor-volatile-bit-v2-2-e8cd0812fe4e@bootlin.com> <956f0490-2e47-4655-ab10-a10ef790cfad@wanadoo.fr> User-Agent: mu4e 1.12.7; emacs 29.4 Date: Fri, 10 Jan 2025 15:46:32 +0100 Message-ID: <87cyguafqv.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-GND-Sasl: miquel.raynal@bootlin.com Hi Christophe, On 09/01/2025 at 21:23:13 +01, Christophe JAILLET wrote: > Le 09/01/2025 =C3=A0 21:17, Miquel Raynal a =C3=A9crit=C2=A0: >> Add support for Winbond w25q02jv spi-nor chip which shares most of >> w25q01jv's specificities as, this time, the chip is made of 4 different >> dies. >> Link: >> https://www.winbond.com/resource-files/W25Q02JV_DTR%20RevD%2007092024%20= Plus.pdf >> Signed-off-by: Miquel Raynal >> --- > > ... > >> diff --git a/drivers/mtd/spi-nor/winbond.c b/drivers/mtd/spi-nor/winbond= .c >> index a4c0d99dde4fcd2b123eb7ded4f7822110c6a4ae..d82962e0bf7d8e00b305fd83= 8274a58c34dd6c4e 100644 >> --- a/drivers/mtd/spi-nor/winbond.c >> +++ b/drivers/mtd/spi-nor/winbond.c >> @@ -305,6 +305,10 @@ static const struct flash_info winbond_nor_parts[] = =3D { >> }, { >> .id =3D SNOR_ID(0xef, 0x70, 0x19), >> .name =3D "w25q256jvm", >> + }, { >> + .id =3D SNOR_ID(0xef, 0x70, 0x22), >> + /* W25Q02JV */ > > Nitpick: The place of the comment is not consistent with patch 1/2. That's right, I'll correct. Thanks, Miqu=C3=A8l