From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf0-x241.google.com ([2607:f8b0:400e:c00::241]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bOcxp-00062j-Qo for linux-mtd@lists.infradead.org; Sun, 17 Jul 2016 03:41:58 +0000 Received: by mail-pf0-x241.google.com with SMTP id i6so8694246pfe.0 for ; Sat, 16 Jul 2016 20:41:37 -0700 (PDT) Date: Sat, 16 Jul 2016 20:41:27 -0700 From: Brian Norris To: Florian Fainelli Cc: linux-mtd@lists.infradead.org, Kamal Dasu , Boris Brezillon , Richard Weinberger , David Woodhouse , "open list:BROADCOM STB NAND FLASH DRIVER" , open list Subject: Re: [PATCH] mtd: nand: brcmnand: Change BUG_ON in brcmnand_send_cmd Message-ID: <20160717034127.GA14096@brian-ubuntu> References: <1467999399-22288-1-git-send-email-f.fainelli@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1467999399-22288-1-git-send-email-f.fainelli@gmail.com> List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Jul 08, 2016 at 10:36:39AM -0700, Florian Fainelli wrote: > Change the BUG_ON() condition in brcmnand_send_cmd() which checks for > the interrupt status "controller ready" bit to a WARN_ON. > > There is no good reason to kill the system when this condition occur > because we could have systems which listed the NAND controller as > available (e.g: from Device Tree), but the NAND chip could be > malfunctioning and not responding. > > Signed-off-by: Florian Fainelli Applied to l2-mtd.git