[PATCH] mtd: nand: fix a typo in a comment line

Brian Norris computersforpeace at gmail.com
Tue Apr 29 12:14:17 PDT 2014


On Thu, Apr 17, 2014 at 06:59:16PM +0900, Masahiro Yamada wrote:
> On Wed, 16 Apr 2014 00:23:11 -0700 Brian Norris <computersforpeace at gmail.com> wrote:
> > On Wed, Apr 09, 2014 at 04:26:26PM +0900, Masahiro Yamada wrote:
> > > Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
> > > ---
> > >  drivers/mtd/nand/nand_base.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
> > > index 9d01c4d..33ec641 100644
> > > --- a/drivers/mtd/nand/nand_base.c
> > > +++ b/drivers/mtd/nand/nand_base.c
> > > @@ -3607,7 +3607,7 @@ static struct nand_flash_dev *nand_get_flash_type(struct mtd_info *mtd,
> > >  
> > >  	chip->onfi_version = 0;
> > >  	if (!type->name || !type->pagesize) {
> > > -		/* Check is chip is ONFI compliant */
> > > +		/* Check if the chip is ONFI compliant */
> > 
> > Really? We don't spell out "the chip" every time we refer to "chip". I
> > think this is just fine as-is.
> 
> I am fixing two things here.
...

My apologies. Apparently I can't tell the difference between "is" and
"if". This patch is fine!

Pushed to l2-mtd.git!

Thanks,
Brian



More information about the linux-mtd mailing list