mtd: remove driver-core BUS_ID_SIZE
David Woodhouse
dwmw2 at infradead.org
Fri May 29 09:11:35 EDT 2009
On Thu, 2009-04-16 at 18:26 +0200, Kay Sievers wrote:
> From: Kay Sievers <kay.sievers at vrfy.org>
> Subject: mtd: remove driver-core BUS_ID_SIZE
>
> The name size limit is gone from the driver-core, the BUS_ID_SIZE
> value will be removed.
>
> Cc: dwmw2 at infradead.org
> Cc: linux-mtd at lists.infradead.org
> Acked-by: Greg Kroah-Hartman <gregkh at suse.de>
> Signed-off-by: Kay Sievers <kay.sievers at vrfy.org>
> ---
> drivers/mtd/nand/txx9ndfmc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/mtd/nand/txx9ndfmc.c
> +++ b/drivers/mtd/nand/txx9ndfmc.c
> @@ -64,7 +64,7 @@ struct txx9ndfmc_priv {
> struct nand_chip chip;
> struct mtd_info mtd;
> int cs;
> - char mtdname[BUS_ID_SIZE + 2];
> + char mtdname[20 + 2];
> };
>
> #define MAX_TXX9NDFMC_DEV 4
Not so cunning. You're explicitly noting that the limit on the input
string size is gone, and yet you do nothing about the code which uses
it...
if (plat->ch_mask != 1) {
txx9_priv->cs = i;
sprintf(txx9_priv->mtdname, "%s.%u",
dev_name(&dev->dev), i);
} else {
txx9_priv->cs = -1;
strcpy(txx9_priv->mtdname, dev_name(&dev->dev));
}
--
David Woodhouse Open Source Technology Centre
David.Woodhouse at intel.com Intel Corporation
More information about the linux-mtd
mailing list