From mboxrd@z Thu Jan 1 00:00:00 1970 From: Miquel Raynal Subject: Re: [PATCH v3 3/4] mtd: rawnand: mtk: Add validity check for CE# pin setting Date: Mon, 3 Jun 2019 10:04:36 +0200 Message-ID: <20190603080436.28793-1-miquel.raynal@bootlin.com> References: <20190507102541.34341-4-xiaolei.li@mediatek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190507102541.34341-4-xiaolei.li-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+glpam-linux-mediatek=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Xiaolei Li , miquel.raynal-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org, richard-/L3Ra7n9ekc@public.gmane.org Cc: linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org List-Id: linux-mediatek@lists.infradead.org On Tue, 2019-05-07 at 10:25:40 UTC, Xiaolei Li wrote: > Currently, we only check how many CE# pins are set in device tree. > But it should be necessary to check whether CE# pin setting is > duplicated or if CE# pin index exceeds the maximum CE# number that > controller supports. > > So, add validity check to avoid these invalid settings. > > Signed-off-by: Xiaolei Li > Reviewed-by: Miquel Raynal Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks. Miquel