Message ID | 1376286173-12581-10-git-send-email-b32955@freescale.com |
---|---|
State | New, archived |
Headers | show |
On Mon, Aug 12, 2013 at 01:42:52PM +0800, Huang Shijie wrote: > In mtd_type_show(), the MTD_NANDFLASH case shows "nand". > In actually, the printed information is not accurate enough. > The MTD_NANDFLASH stands for SLC nand now. > > This patch adds the SLC information for the MTD_NANDFLASH case. > > Signed-off-by: Huang Shijie <b32955@freescale.com> > --- > drivers/mtd/mtdcore.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c > index c7cee29..78b8f2e 100644 > --- a/drivers/mtd/mtdcore.c > +++ b/drivers/mtd/mtdcore.c > @@ -149,7 +149,7 @@ static ssize_t mtd_type_show(struct device *dev, > type = "nor"; > break; > case MTD_NANDFLASH: > - type = "nand"; > + type = "SLC nand"; Nak. This is clearly an ABI change, doesn't follow the existing naming convention, and isn't even necessary (just differentiating the "mlc-nand" case in the prior patch is sufficient). > break; > case MTD_DATAFLASH: > type = "dataflash"; Brian
diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c index c7cee29..78b8f2e 100644 --- a/drivers/mtd/mtdcore.c +++ b/drivers/mtd/mtdcore.c @@ -149,7 +149,7 @@ static ssize_t mtd_type_show(struct device *dev, type = "nor"; break; case MTD_NANDFLASH: - type = "nand"; + type = "SLC nand"; break; case MTD_DATAFLASH: type = "dataflash";
In mtd_type_show(), the MTD_NANDFLASH case shows "nand". In actually, the printed information is not accurate enough. The MTD_NANDFLASH stands for SLC nand now. This patch adds the SLC information for the MTD_NANDFLASH case. Signed-off-by: Huang Shijie <b32955@freescale.com> --- drivers/mtd/mtdcore.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)