Re: [PATCH] mtd: nand: remove unused function input parameter
On Wed, Dec 25, 2013 at 12:40:50PM +, Caizhiyong wrote: > From: Cai Zhiyong > Date: Wed, 25 Dec 2013 20:11:15 +0800 > Subject: [PATCH] mtd: nand: remove unused function input parameter > > The nand_get_flash_type parameter "busw" input value is not used by any > branch, and it is updated before use it in the function, so remove it, > define the "busw" as an internal variable. > > Signed-off-by: Cai Zhiyong I merged this patch and sent it upstream for 3.15-rc1, but I don't think I ever sent you a notification... here goes: Applied, thanks! Brian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mtd: nand: remove unused function input parameter
On Wed, Dec 25, 2013 at 12:40:50PM +, Caizhiyong wrote: > From: Cai Zhiyong > Date: Wed, 25 Dec 2013 20:11:15 +0800 > Subject: [PATCH] mtd: nand: remove unused function input parameter > > The nand_get_flash_type parameter "busw" input value is not used by any > branch, and it is updated before use it in the function, so remove it, > define the "busw" as an internal variable. > > Signed-off-by: Cai Zhiyong > --- > drivers/mtd/nand/nand_base.c | 12 +--- > 1 file changed, 5 insertions(+), 7 deletions(-) > > diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c > index 9b3bb3c..a123afa 100644 > --- a/drivers/mtd/nand/nand_base.c > +++ b/drivers/mtd/nand/nand_base.c > @@ -3337,10 +3337,10 @@ static bool find_full_id_nand(struct mtd_info *mtd, > struct nand_chip *chip, > */ > static struct nand_flash_dev *nand_get_flash_type(struct mtd_info *mtd, > struct nand_chip *chip, > - int busw, > int *maf_id, int *dev_id, > struct nand_flash_dev *type) > { > + int busw; > int i, maf_idx; > u8 id_data[8]; > > @@ -3497,18 +3497,16 @@ ident_done: > int nand_scan_ident(struct mtd_info *mtd, int maxchips, > struct nand_flash_dev *table) > { > - int i, busw, nand_maf_id, nand_dev_id; > + int i, nand_maf_id, nand_dev_id; > struct nand_chip *chip = mtd->priv; > struct nand_flash_dev *type; > > - /* Get buswidth to select the correct functions */ > - busw = chip->options & NAND_BUSWIDTH_16; > /* Set the default functions */ > - nand_set_defaults(chip, busw); > + nand_set_defaults(chip, chip->options & NAND_BUSWIDTH_16); > > /* Read the flash type */ > - type = nand_get_flash_type(mtd, chip, busw, > - &nand_maf_id, &nand_dev_id, table); > + type = nand_get_flash_type(mtd, chip, &nand_maf_id, > +&nand_dev_id, table); I think the "busw" is to make the code more readable, since the nand_set_defaults also needs a busw. Let Brian judge it. thanks Huang Shijie -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] mtd: nand: remove unused function input parameter
From: Cai Zhiyong Date: Wed, 25 Dec 2013 20:11:15 +0800 Subject: [PATCH] mtd: nand: remove unused function input parameter The nand_get_flash_type parameter "busw" input value is not used by any branch, and it is updated before use it in the function, so remove it, define the "busw" as an internal variable. Signed-off-by: Cai Zhiyong --- drivers/mtd/nand/nand_base.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index 9b3bb3c..a123afa 100644 --- a/drivers/mtd/nand/nand_base.c +++ b/drivers/mtd/nand/nand_base.c @@ -3337,10 +3337,10 @@ static bool find_full_id_nand(struct mtd_info *mtd, struct nand_chip *chip, */ static struct nand_flash_dev *nand_get_flash_type(struct mtd_info *mtd, struct nand_chip *chip, - int busw, int *maf_id, int *dev_id, struct nand_flash_dev *type) { + int busw; int i, maf_idx; u8 id_data[8]; @@ -3497,18 +3497,16 @@ ident_done: int nand_scan_ident(struct mtd_info *mtd, int maxchips, struct nand_flash_dev *table) { - int i, busw, nand_maf_id, nand_dev_id; + int i, nand_maf_id, nand_dev_id; struct nand_chip *chip = mtd->priv; struct nand_flash_dev *type; - /* Get buswidth to select the correct functions */ - busw = chip->options & NAND_BUSWIDTH_16; /* Set the default functions */ - nand_set_defaults(chip, busw); + nand_set_defaults(chip, chip->options & NAND_BUSWIDTH_16); /* Read the flash type */ - type = nand_get_flash_type(mtd, chip, busw, - &nand_maf_id, &nand_dev_id, table); + type = nand_get_flash_type(mtd, chip, &nand_maf_id, + &nand_dev_id, table); if (IS_ERR(type)) { if (!(chip->options & NAND_SCAN_SILENT_NODEV)) -- 1.8.1.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/