Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp2662684pxb; Tue, 23 Feb 2021 12:18:50 -0800 (PST) X-Google-Smtp-Source: ABdhPJzitGuFhf9r1WwWAG0tn5iXEQQOy2jUf043AFEi6UCoOUZyKIzSwrKsnGtG4ddM/iGJRAqo X-Received: by 2002:a17:906:b819:: with SMTP id dv25mr28598792ejb.476.1614111530651; Tue, 23 Feb 2021 12:18:50 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1614111530; cv=none; d=google.com; s=arc-20160816; b=encGA/WM3xwR4lUqClg8kpG9qAryNZy8L5TdxVnn2/Un7efpshuSTMhPmt6RzHws7K udpwROytYKVSboqx8x5SXenQ5PUiTPmDWl0es56HCh6L44A8htoR5Tc/4EKTfxXpygH3 6aVCWoHUYFiXao5X0+yZ4Ho6gtpQtF57FFW3DDfUccVARwvEAyDS42CIWB5KVO10GpoN S95ihYfBatGC434oAE6MtcmT8Ue1Yfs55xFV8rXLxfbC2obujLxfBVIl50Ge2jEuTL2p aeKlbEyd27ouCpTPqinzx6PPqNqY87e/aSOfC5zbseCaeSKzYOx0/AA7pC1gRoM44n69 U5PA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :organization:references:in-reply-to:message-id:subject:cc:to:from :date; bh=oZcC0+i8RNP48c0XGm2CNu9Bygx1szm79cMGGDbUZSQ=; b=LVU60CID/4a6K2FD8mmy+u4xaRoPe65kJ+m5HB/C2IZg/elOzdx2GjaldtC8eeCGJe wcazwaGBGq6WAZltPZoWY58Ak9t/8Va90HeGrjRCW0B6ivdy76pgslr6wzlZ95Tri8AQ SrgYlV/ipb9WT/66jHB3NC6Pg4UTaP5R7Vx0mVuhzVF6QIo/9p6Qo6pxFOeVaQ1uKd9F sr1y7AZnro3qOZ0nHuP+kI598aG498KSID/Ofvg0wZDblVV6DuEcODU9NjdlOO5UBXwU frEk058DT1nO5/I5zZu5EeFO9SDLe3qatH0LXuDj86FGWwkU98y6vT0Scn5pczYz+m8r p8Yw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id g26si1753185edm.268.2021.02.23.12.18.17; Tue, 23 Feb 2021 12:18:50 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233221AbhBWQKz convert rfc822-to-8bit (ORCPT + 99 others); Tue, 23 Feb 2021 11:10:55 -0500 Received: from relay5-d.mail.gandi.net ([217.70.183.197]:41615 "EHLO relay5-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232599AbhBWQKx (ORCPT ); Tue, 23 Feb 2021 11:10:53 -0500 X-Originating-IP: 86.210.203.113 Received: from xps13 (lfbn-tou-1-972-113.w86-210.abo.wanadoo.fr [86.210.203.113]) (Authenticated sender: miquel.raynal@bootlin.com) by relay5-d.mail.gandi.net (Postfix) with ESMTPSA id 02CFB1C0009; Tue, 23 Feb 2021 16:10:07 +0000 (UTC) Date: Tue, 23 Feb 2021 17:10:06 +0100 From: Miquel Raynal To: Md Sadre Alam Cc: mani@kernel.org, richard@nod.at, boris.brezillon@collabora.com, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, sricharan@codeaurora.org Subject: Re: [PATCH] mtd: rawnand: qcom: Convert nandc to chip in Read/Write helper Message-ID: <20210223171006.3e2030ce@xps13> In-Reply-To: <1613939101-23001-1-git-send-email-mdalam@codeaurora.org> References: <1613939101-23001-1-git-send-email-mdalam@codeaurora.org> Organization: Bootlin X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, Md Sadre Alam wrote on Mon, 22 Feb 2021 01:55:01 +0530: > This change will convert nandc to chip in Read/Write helper, this > change is needed because if we wnated to access number of steps > in Read/Write helper then we need to get the chip->ecc.steps, > currentlly its not possible.After this change we can directly > acces chip->ecc.steps in Read/Write helper. > > Signed-off-by: Md Sadre Alam Thanks for splitting your series, I think it's much easier to review and contains much less imprecise changes. I have a few minor comments (see the following e-mails), please address them and then please send all your patches a single series, not like 6+ independent patches. I'll then require someone to test it if we are good I'll merge it. Thanks, Miquèl