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 28C84175A60 for ; Tue, 5 May 2026 08:10:12 +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=1777968615; cv=none; b=JwPGWNCmSyrxTINsQoSfuoyFMbuViRMsCazdnIhpMCs2q1VgsygAaeRRdyp3sUawl4Ghk514dPesmzMfDAXW9PFylaLOWFbY+IzkEqbi/DH0zRwXQB/6cIGq8QkkB8kKxYn6MmvJbYXjKNtlkVeVCDD71DxTjx/l6dkvYiM4iuM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777968615; c=relaxed/simple; bh=2ajGsto+fkULMbVHALYVrcuruVtiBBl0r54Ob0QDCTQ=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=d3PvcSkcKueSp0Mz1+fZ7y+5bk2NANaPdP3udtD3yoBz1myXU0X2yxooW26qsmtKUeTxVUHbraoC07zfER10X7lwg1et/ry9e7oAQ4aiIfjIYQTbftzRGDFbLVXNzrCiIv3sMbek7nqwJD1+3EI7fo46jPgk1Im/UZkLbTtV+t0= 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=rbi2S0zs; 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="rbi2S0zs" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-04.galae.net (Postfix) with ESMTPS id 2F749C5D739; Tue, 5 May 2026 08:10:58 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 5A8175FD9D; Tue, 5 May 2026 08:10:11 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 25FC211AD249A; Tue, 5 May 2026 10:10:06 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1777968610; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=wqojrbNmyZDtYjYTfDSv+NteblAz01LppSlPWqNNyS4=; b=rbi2S0zs00YEewZbf9Qul0BnoxDxqZDYJfeVDTSeREai7hgjWz+xCHL0KlY9fCGbDFVHgF GBp7wXS6xkK9FovJ4aqUwvev9RFjvQ1ES3MFZXJMWiLK2ZSLNacSPBatnezCzIg7wPJ68n axtaipDDOY8XETclJFiiBH6f8Tam0kY7OJLHMSfVsB+g7T1ue0ofN6fvvNTqq/JwU+p5kj 1buQRC21xCZz9kCM4d39Ql4dxygd0r3gQzUKP2o1iY79TPYMBB+XlxoTV6c8bTMOCr4gS3 1avcfIaLz/TI2CEecbITwm3wDqGMEAMeDSDY/KVrc6rJIs6JGzW7R0XFmfs0iA== From: Miquel Raynal To: Arseniy Krasnov Cc: Richard Weinberger , Vignesh Raghavendra , Frieder Schrempf , Boris Brezillon , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, rulkc@linuxtesting.org, oxffffaa@gmail.com, stable@vger.kernel.org Subject: Re: [PATCH v1] mtd: rawnand: fix condition in 'nand_select_target()' In-Reply-To: <57b0cc2a-6d62-405c-bfa5-68d1c46dbad9@rulkc.org> (Arseniy Krasnov's message of "Tue, 5 May 2026 10:59:16 +0300") References: <20260504221012.1310605-1-avkrasnov@rulkc.org> <87mryeqoqs.fsf@bootlin.com> <57b0cc2a-6d62-405c-bfa5-68d1c46dbad9@rulkc.org> User-Agent: mu4e 1.12.7; emacs 30.2 Date: Tue, 05 May 2026 10:10:06 +0200 Message-ID: <87h5omqntt.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 On 05/05/2026 at 10:59:16 +03, Arseniy Krasnov wrote: > 05.05.2026 10:50, Miquel Raynal wrote: >> Hi, >> >> On 05/05/2026 at 01:10:12 +03, Arseniy Krasnov wro= te: >> >> Two important typos in the commit log :-) >> >>> 'cs' here must in range [0:nanddev_ntargets). >> be [ > > > Hi, sorry, You mean? > > > 'cs' here must be in range [0:nanddev_ntargets].=C2=A0 I meant [0:nanddev_ntargets[ which is the mathematical way, IIRC, to indicate that the last value is out of scope/excluded. [0:nanddev_ntargets] means that nanddev_ntargets is included in the scope of values and here since you are explicitly showing that it is not, it feels wrong to use that convention. Thanks, Miqu=C3=A8l