* [PATCH v2 1/2] mtd: ifc: Update dependency of IFC for LS1021A
@ 2017-01-03 2:41 Alison Wang
2017-01-03 2:41 ` [PATCH v2 2/2] mtd: nand: " Alison Wang
2017-01-03 9:04 ` [PATCH v2 1/2] mtd: ifc: " Boris Brezillon
0 siblings, 2 replies; 10+ messages in thread
From: Alison Wang @ 2017-01-03 2:41 UTC (permalink / raw)
To: boris.brezillon, marek.vasut, cyrille.pitchen, linux-kernel,
linux-arm-kernel, linux-mtd, dwmw2, computersforpeace
Cc: alison.wang
As Freescale/NXP IFC controller is available on LS1021A, the dependency
for LS1021A is added.
Signed-off-by: Alison Wang <alison.wang@nxp.com>
---
Changes in v2:
- New patch
drivers/memory/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig
index ec80e35..fff8345 100644
--- a/drivers/memory/Kconfig
+++ b/drivers/memory/Kconfig
@@ -115,7 +115,7 @@ config FSL_CORENET_CF
config FSL_IFC
bool
- depends on FSL_SOC || ARCH_LAYERSCAPE
+ depends on FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A
config JZ4780_NEMC
bool "Ingenic JZ4780 SoC NEMC driver"
--
2.1.0.27.g96db324
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
2017-01-03 2:41 [PATCH v2 1/2] mtd: ifc: Update dependency of IFC for LS1021A Alison Wang
@ 2017-01-03 2:41 ` Alison Wang
2017-01-03 11:46 ` Marek Vasut
2017-01-03 9:04 ` [PATCH v2 1/2] mtd: ifc: " Boris Brezillon
1 sibling, 1 reply; 10+ messages in thread
From: Alison Wang @ 2017-01-03 2:41 UTC (permalink / raw)
To: boris.brezillon, marek.vasut, cyrille.pitchen, linux-kernel,
linux-arm-kernel, linux-mtd, dwmw2, computersforpeace
Cc: alison.wang
As NAND support for Freescale/NXP IFC controller is available on
LS1021A, the dependency for LS1021A is added.
Signed-off-by: Alison Wang <alison.wang@nxp.com>
---
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
--
2.1.0.27.g96db324
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2 1/2] mtd: ifc: Update dependency of IFC for LS1021A
2017-01-03 2:41 [PATCH v2 1/2] mtd: ifc: Update dependency of IFC for LS1021A Alison Wang
2017-01-03 2:41 ` [PATCH v2 2/2] mtd: nand: " Alison Wang
@ 2017-01-03 9:04 ` Boris Brezillon
2017-01-03 9:19 ` Alison Wang
1 sibling, 1 reply; 10+ messages in thread
From: Boris Brezillon @ 2017-01-03 9:04 UTC (permalink / raw)
To: Alison Wang
Cc: marek.vasut, cyrille.pitchen, linux-kernel, linux-arm-kernel,
linux-mtd, dwmw2, computersforpeace, alison.wang
Hi Alison,
The subject prefix should be "memory: " and not "mtd: ifc: ".
Looks good otherwise.
No need to resend, I can fix that when applying.
Regards,
Boris
On Tue, 3 Jan 2017 10:41:05 +0800
Alison Wang <b18965@freescale.com> wrote:
> As Freescale/NXP IFC controller is available on LS1021A, the dependency
> for LS1021A is added.
>
> Signed-off-by: Alison Wang <alison.wang@nxp.com>
> ---
> Changes in v2:
> - New patch
>
> drivers/memory/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig
> index ec80e35..fff8345 100644
> --- a/drivers/memory/Kconfig
> +++ b/drivers/memory/Kconfig
> @@ -115,7 +115,7 @@ config FSL_CORENET_CF
>
> config FSL_IFC
> bool
> - depends on FSL_SOC || ARCH_LAYERSCAPE
> + depends on FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A
>
> config JZ4780_NEMC
> bool "Ingenic JZ4780 SoC NEMC driver"
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v2 1/2] mtd: ifc: Update dependency of IFC for LS1021A
2017-01-03 9:04 ` [PATCH v2 1/2] mtd: ifc: " Boris Brezillon
@ 2017-01-03 9:19 ` Alison Wang
0 siblings, 0 replies; 10+ messages in thread
From: Alison Wang @ 2017-01-03 9:19 UTC (permalink / raw)
To: Boris Brezillon, Alison Wang
Cc: marek.vasut, cyrille.pitchen, linux-kernel, linux-arm-kernel,
linux-mtd, dwmw2, computersforpeace
Ok, I see. Thanks.
Best Regards,
Alison Wang
> -----Original Message-----
> From: Boris Brezillon [mailto:boris.brezillon@free-electrons.com]
> Sent: Tuesday, January 03, 2017 5:05 PM
> To: Alison Wang <b18965@freescale.com>
> Cc: marek.vasut@gmail.com; cyrille.pitchen@atmel.com; linux-
> kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> mtd@lists.infradead.org; dwmw2@infradead.org;
> computersforpeace@gmail.com; Alison Wang <alison.wang@nxp.com>
> Subject: Re: [PATCH v2 1/2] mtd: ifc: Update dependency of IFC for
> LS1021A
>
> Hi Alison,
>
> The subject prefix should be "memory: " and not "mtd: ifc: ".
>
> Looks good otherwise.
>
> No need to resend, I can fix that when applying.
>
> Regards,
>
> Boris
>
> On Tue, 3 Jan 2017 10:41:05 +0800
> Alison Wang <b18965@freescale.com> wrote:
>
> > As Freescale/NXP IFC controller is available on LS1021A, the
> > dependency for LS1021A is added.
> >
> > Signed-off-by: Alison Wang <alison.wang@nxp.com>
> > ---
> > Changes in v2:
> > - New patch
> >
> > drivers/memory/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig index
> > ec80e35..fff8345 100644
> > --- a/drivers/memory/Kconfig
> > +++ b/drivers/memory/Kconfig
> > @@ -115,7 +115,7 @@ config FSL_CORENET_CF
> >
> > config FSL_IFC
> > bool
> > - depends on FSL_SOC || ARCH_LAYERSCAPE
> > + depends on FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A
> >
> > config JZ4780_NEMC
> > bool "Ingenic JZ4780 SoC NEMC driver"
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
2017-01-03 2:41 ` [PATCH v2 2/2] mtd: nand: " Alison Wang
@ 2017-01-03 11:46 ` Marek Vasut
2017-01-04 1:46 ` Alison Wang
0 siblings, 1 reply; 10+ messages in thread
From: Marek Vasut @ 2017-01-03 11:46 UTC (permalink / raw)
To: Alison Wang, boris.brezillon, cyrille.pitchen, linux-kernel,
linux-arm-kernel, linux-mtd, dwmw2, computersforpeace
Cc: alison.wang
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 <alison.wang@nxp.com>
> ---
> 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
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
2017-01-03 11:46 ` Marek Vasut
@ 2017-01-04 1:46 ` Alison Wang
2017-01-04 1:50 ` Marek Vasut
0 siblings, 1 reply; 10+ messages in thread
From: Alison Wang @ 2017-01-04 1:46 UTC (permalink / raw)
To: Marek Vasut, Alison Wang, boris.brezillon, cyrille.pitchen,
linux-kernel, linux-arm-kernel, linux-mtd, dwmw2,
computersforpeace
> 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 <alison.wang@nxp.com>
> > ---
> > 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
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
2017-01-04 1:46 ` Alison Wang
@ 2017-01-04 1:50 ` Marek Vasut
2017-01-05 2:02 ` Alison Wang
0 siblings, 1 reply; 10+ messages in thread
From: Marek Vasut @ 2017-01-04 1:50 UTC (permalink / raw)
To: Alison Wang, Alison Wang, boris.brezillon, cyrille.pitchen,
linux-kernel, linux-arm-kernel, linux-mtd, dwmw2,
computersforpeace
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 <alison.wang@nxp.com>
>>> ---
>>> 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
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
2017-01-04 1:50 ` Marek Vasut
@ 2017-01-05 2:02 ` Alison Wang
2017-02-06 9:05 ` Boris Brezillon
0 siblings, 1 reply; 10+ messages in thread
From: Alison Wang @ 2017-01-05 2:02 UTC (permalink / raw)
To: Marek Vasut, Alison Wang, boris.brezillon, cyrille.pitchen,
linux-kernel, linux-arm-kernel, linux-mtd, dwmw2,
computersforpeace
> 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 <alison.wang@nxp.com>
> >>> ---
> >>> 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
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
2017-01-05 2:02 ` Alison Wang
@ 2017-02-06 9:05 ` Boris Brezillon
2017-02-13 6:59 ` Alison Wang
0 siblings, 1 reply; 10+ messages in thread
From: Boris Brezillon @ 2017-02-06 9:05 UTC (permalink / raw)
To: Alison Wang
Cc: Marek Vasut, Alison Wang, cyrille.pitchen, linux-kernel,
linux-arm-kernel, linux-mtd, dwmw2, computersforpeace
Hi Alison,
On Thu, 5 Jan 2017 02:02:30 +0000
Alison Wang <alison.wang@nxp.com> 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
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v2 2/2] mtd: nand: Update dependency of IFC for LS1021A
2017-02-06 9:05 ` Boris Brezillon
@ 2017-02-13 6:59 ` Alison Wang
0 siblings, 0 replies; 10+ messages in thread
From: Alison Wang @ 2017-02-13 6:59 UTC (permalink / raw)
To: Boris Brezillon
Cc: Marek Vasut, Alison Wang, cyrille.pitchen, linux-kernel,
linux-arm-kernel, linux-mtd, dwmw2, computersforpeace
Hi, Boris,
> On Thu, 5 Jan 2017 02:02:30 +0000
> Alison Wang <alison.wang@nxp.com> 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
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-02-13 6:59 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-03 2:41 [PATCH v2 1/2] mtd: ifc: Update dependency of IFC for LS1021A Alison Wang
2017-01-03 2:41 ` [PATCH v2 2/2] mtd: nand: " Alison Wang
2017-01-03 11:46 ` Marek Vasut
2017-01-04 1:46 ` Alison Wang
2017-01-04 1:50 ` Marek Vasut
2017-01-05 2:02 ` Alison Wang
2017-02-06 9:05 ` Boris Brezillon
2017-02-13 6:59 ` Alison Wang
2017-01-03 9:04 ` [PATCH v2 1/2] mtd: ifc: " Boris Brezillon
2017-01-03 9:19 ` Alison Wang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).