RE: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
Hi, Boris, > On Thu, 5 Jan 2017 02:02:30 + > Alison Wang wrote: > > > > On 01/04/2017 02:46 AM, Alison Wang wrote: > > > >> On 01/03/2017 03:41 AM, Alison Wang wrote: > > > >>> As NAND support for Freescale/NXP IFC controller is available > on > > > >>> LS1021A, the dependency for LS1021A is added. > > > >> > > > >> Does LS stand for LayerScape ? Yes it does. So why does > > > >> ARCH_LAYERSCAPE not cover LS1021 ? > > > > [Alison Wang] LS1021A is an earlier product and is not compatible > > > with later Layerscape architecture. So ARCH_LAYERSCAPE can't cover > > > LS1021A. > > > > > > Ah ok, I see. That information would be useful in the commit > message > > > ;-) > > > > > [Alison Wang] Ok. :) > > Would you mind sending a v3 with the updated commit message? > [Alison Wang] I will send the v3 patches at once. :) Best Regards, Alison Wang
Re: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
Hi Alison, On Thu, 5 Jan 2017 02:02:30 + Alison Wang wrote: > > On 01/04/2017 02:46 AM, Alison Wang wrote: > > >> On 01/03/2017 03:41 AM, Alison Wang wrote: > > >>> As NAND support for Freescale/NXP IFC controller is available on > > >>> LS1021A, the dependency for LS1021A is added. > > >> > > >> Does LS stand for LayerScape ? Yes it does. So why does > > >> ARCH_LAYERSCAPE not cover LS1021 ? > > > [Alison Wang] LS1021A is an earlier product and is not compatible > > with later Layerscape architecture. So ARCH_LAYERSCAPE can't cover > > LS1021A. > > > > Ah ok, I see. That information would be useful in the commit message ;-) > > > [Alison Wang] Ok. :) Would you mind sending a v3 with the updated commit message? Thanks, Boris
RE: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
> On 01/04/2017 02:46 AM, Alison Wang wrote: > >> On 01/03/2017 03:41 AM, Alison Wang wrote: > >>> As NAND support for Freescale/NXP IFC controller is available on > >>> LS1021A, the dependency for LS1021A is added. > >> > >> Does LS stand for LayerScape ? Yes it does. So why does > >> ARCH_LAYERSCAPE not cover LS1021 ? > > [Alison Wang] LS1021A is an earlier product and is not compatible > with later Layerscape architecture. So ARCH_LAYERSCAPE can't cover > LS1021A. > > Ah ok, I see. That information would be useful in the commit message ;-) > [Alison Wang] Ok. :) > >> > >>> Signed-off-by: Alison Wang > >>> --- > >>> Changes in v2: > >>> - None > >>> > >>> drivers/mtd/nand/Kconfig | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig > >> index > >>> 353a9dd..85e3860 100644 > >>> --- a/drivers/mtd/nand/Kconfig > >>> +++ b/drivers/mtd/nand/Kconfig > >>> @@ -441,7 +441,7 @@ config MTD_NAND_FSL_ELBC > >>> > >>> config MTD_NAND_FSL_IFC > >>> tristate "NAND support for Freescale IFC controller" > >>> - depends on FSL_SOC || ARCH_LAYERSCAPE > >>> + depends on FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A > >>> select FSL_IFC > >>> select MEMORY > >>> help > >>> > > > > Best Regards, > > Alison Wang > > > > > -- > Best regards, > Marek Vasut Best Regards, Alison Wang
Re: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
On 01/04/2017 02:46 AM, Alison Wang wrote: >> On 01/03/2017 03:41 AM, Alison Wang wrote: >>> As NAND support for Freescale/NXP IFC controller is available on >>> LS1021A, the dependency for LS1021A is added. >> >> Does LS stand for LayerScape ? Yes it does. So why does ARCH_LAYERSCAPE >> not cover LS1021 ? > [Alison Wang] LS1021A is an earlier product and is not compatible with later > Layerscape architecture. So ARCH_LAYERSCAPE can't cover LS1021A. Ah ok, I see. That information would be useful in the commit message ;-) >> >>> Signed-off-by: Alison Wang >>> --- >>> Changes in v2: >>> - None >>> >>> drivers/mtd/nand/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig >> index >>> 353a9dd..85e3860 100644 >>> --- a/drivers/mtd/nand/Kconfig >>> +++ b/drivers/mtd/nand/Kconfig >>> @@ -441,7 +441,7 @@ config MTD_NAND_FSL_ELBC >>> >>> config MTD_NAND_FSL_IFC >>> tristate "NAND support for Freescale IFC controller" >>> - depends on FSL_SOC || ARCH_LAYERSCAPE >>> + depends on FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A >>> select FSL_IFC >>> select MEMORY >>> help >>> > > Best Regards, > Alison Wang > -- Best regards, Marek Vasut
RE: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
> On 01/03/2017 03:41 AM, Alison Wang wrote: > > As NAND support for Freescale/NXP IFC controller is available on > > LS1021A, the dependency for LS1021A is added. > > Does LS stand for LayerScape ? Yes it does. So why does ARCH_LAYERSCAPE > not cover LS1021 ? [Alison Wang] LS1021A is an earlier product and is not compatible with later Layerscape architecture. So ARCH_LAYERSCAPE can't cover LS1021A. > > > Signed-off-by: Alison Wang > > --- > > Changes in v2: > > - None > > > > drivers/mtd/nand/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig > index > > 353a9dd..85e3860 100644 > > --- a/drivers/mtd/nand/Kconfig > > +++ b/drivers/mtd/nand/Kconfig > > @@ -441,7 +441,7 @@ config MTD_NAND_FSL_ELBC > > > > config MTD_NAND_FSL_IFC > > tristate "NAND support for Freescale IFC controller" > > - depends on FSL_SOC || ARCH_LAYERSCAPE > > + depends on FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A > > select FSL_IFC > > select MEMORY > > help > > Best Regards, Alison Wang
Re: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
On 01/03/2017 03:41 AM, Alison Wang wrote: > As NAND support for Freescale/NXP IFC controller is available on > LS1021A, the dependency for LS1021A is added. Does LS stand for LayerScape ? Yes it does. So why does ARCH_LAYERSCAPE not cover LS1021 ? > Signed-off-by: Alison Wang > --- > Changes in v2: > - None > > drivers/mtd/nand/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig > index 353a9dd..85e3860 100644 > --- a/drivers/mtd/nand/Kconfig > +++ b/drivers/mtd/nand/Kconfig > @@ -441,7 +441,7 @@ config MTD_NAND_FSL_ELBC > > config MTD_NAND_FSL_IFC > tristate "NAND support for Freescale IFC controller" > - depends on FSL_SOC || ARCH_LAYERSCAPE > + depends on FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A > select FSL_IFC > select MEMORY > help > -- Best regards, Marek Vasut
