From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ie0-x235.google.com (mail-ie0-x235.google.com [IPv6:2607:f8b0:4001:c03::235]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id A5BC82C00B0 for ; Tue, 11 Feb 2014 05:47:34 +1100 (EST) Received: by mail-ie0-f181.google.com with SMTP id to1so3828049ieb.40 for ; Mon, 10 Feb 2014 10:47:30 -0800 (PST) Date: Mon, 10 Feb 2014 10:47:26 -0800 From: Brian Norris To: "B48286@freescale.com" Subject: Re: [PATCH] mtd: m25p80: Make the name of mtd_info fixed Message-ID: <20140210184726.GC18440@ld-irv-0074> References: <1388990069-27066-1-git-send-email-b48286@freescale.com> <20140123021213.GZ8919@ld-irv-0074> <66941b677b7e4b8d9ce4cc44304b0955@DM2PR03MB301.namprd03.prod.outlook.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <66941b677b7e4b8d9ce4cc44304b0955@DM2PR03MB301.namprd03.prod.outlook.com> Cc: Scott Wood , "linuxppc-dev@ozlabs.org" , "Mingkai.Hu@freescale.com" , "linux-mtd@lists.infradead.org" , Ezequiel Garcia List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi Hou, On Thu, Jan 23, 2014 at 03:29:41AM +0000, B48286@freescale.com wrote: > > -----Original Message----- > > From: Brian Norris [mailto:computersforpeace@gmail.com] > > Sent: Thursday, January 23, 2014 10:12 AM > > To: Hou Zhiqiang-B48286 > > Cc: linux-mtd@lists.infradead.org; linuxppc-dev@ozlabs.org; Wood Scott- > > B07421; Hu Mingkai-B21284; Ezequiel Garcia > > Subject: Re: [PATCH] mtd: m25p80: Make the name of mtd_info fixed > > > > On Mon, Jan 06, 2014 at 02:34:29PM +0800, Hou Zhiqiang wrote: > > > --- a/drivers/mtd/devices/m25p80.c > > > +++ b/drivers/mtd/devices/m25p80.c > > > @@ -1012,7 +1012,8 @@ static int m25p_probe(struct spi_device *spi) > > > if (data && data->name) > > > flash->mtd.name = data->name; > > > else > > > - flash->mtd.name = dev_name(&spi->dev); > > > + flash->mtd.name = kasprintf(GFP_KERNEL, "%s.%d", > > > + id->name, spi->chip_select); > > > > Changing the mtd.name may have far-reaching consequences for users who > > already have mtdparts= command lines. But your concern is probably valid > > for dynamically-determined bus numbers. Perhaps you can edit this patch > > to only change the name when the busnum is dynamically-allocated? > > > It's a good idea, but in the case of mtd_info's name dynamically-allocated > using "mtdparts=..." in command lines is illegal obviously. I agree that users should never have relied on the dynamically-allocated name. But changing the name for non-dynamic schemes (e.g., where the SPI busnum is a fixed value) is not pleasant for users. > Would you tell > me what side-effect will be brought by the change of mtd_info's name. I can only think of the mtdparts= command line. Otherwise, I don't think the name is very important. Brian