On Fri, Mar 23, 2012 at 9:28 AM, Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> wrote: > On Thu, Mar 22, 2012 at 10:05:17PM +0100, Marek Belisko wrote: >> Fix following warning: >> drivers/mtd/core.c: In function 'mtd_ioctl': >> drivers/mtd/core.c:122:24: warning: unused variable 'ecc' >> >> Found in at91rm9200ek_defconfig. >> >> Signed-off-by: Marek Belisko <marek.belisko@xxxxxxxxxxxxxxx> >> --- >> drivers/mtd/core.c | 3 ++- >> 1 files changed, 2 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/mtd/core.c b/drivers/mtd/core.c >> index ccc5c26..e11ba64 100644 >> --- a/drivers/mtd/core.c >> +++ b/drivers/mtd/core.c >> @@ -119,7 +119,6 @@ int mtd_ioctl(struct cdev *cdev, int request, void *buf) >> int ret = 0; >> struct mtd_info *mtd = cdev->priv; >> struct mtd_info_user *user = buf; >> - struct mtd_ecc_stats *ecc = buf; >> struct region_info_user *reg = buf; >> >> switch (request) { >> @@ -146,6 +145,8 @@ int mtd_ioctl(struct cdev *cdev, int request, void *buf) >> break; >> #if (defined(CONFIG_NAND_ECC_HW) || defined(CONFIG_NAND_ECC_SOFT)) >> case ECCGETSTATS: >> + struct mtd_ecc_stats *ecc = buf; >> + > > Won't that introduce a new warning about mixed code and declarations? I know C don't like such a construction but my CodeSourcery (2009q1) compiler don't give me any warning after that change. Maybe newer one will do. > > Sascha > >> ecc->corrected = mtd->ecc_stats.corrected; >> ecc->failed = mtd->ecc_stats.failed; >> ecc->badblocks = mtd->ecc_stats.badblocks; >> -- >> 1.7.5.4 >> >> >> _______________________________________________ >> barebox mailing list >> barebox@xxxxxxxxxxxxxxxxxxx >> http://lists.infradead.org/mailman/listinfo/barebox >> > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | marek -- as simple and primitive as possible ------------------------------------------------- Marek Belisko - OPEN-NANDRA Freelance Developer Ruska Nova Ves 219 | Presov, 08005 Slovak Republic Tel: +421 915 052 184 skype: marekwhite twitter: #opennandra web: http://open-nandra.com _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox