diff mbox

[v3,1/2] mtd: ifc: Update dependency of IFC for LS1021A

Message ID 1486968416-13536-1-git-send-email-b18965@freescale.com
State Accepted
Delegated to: Boris Brezillon
Headers show

Commit Message

Alison Wang Feb. 13, 2017, 6:46 a.m. UTC 
As Freescale/NXP IFC controller is available on LS1021A, the dependency
for LS1021A is added.

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 v3:
- Update the commit message.

Changes in v2:
- New patch

 drivers/memory/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Boris Brezillon Feb. 13, 2017, 7:34 a.m. UTC | #1 
Hi Alison,

The subject prefix is still wrong, should be 'memory: ifc: '.

On Mon, 13 Feb 2017 14:46:55 +0800
Alison Wang <b18965@freescale.com> wrote:

> As Freescale/NXP IFC controller is available on LS1021A, the dependency
> for LS1021A is added.
> 
> 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 v3:
> - Update the commit message.
> 
> 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"
Alison Wang Feb. 13, 2017, 7:39 a.m. UTC | #2 
Hi, Boris,

	Sorry, I forgot to change them. Should I resend them or you help to fix that when applying?


Best Regards,
Alison Wang

> 
> Hi Alison,
> 
> The subject prefix is still wrong, should be 'memory: ifc: '.
> 
> On Mon, 13 Feb 2017 14:46:55 +0800
> Alison Wang <b18965@freescale.com> wrote:
> 
> > As Freescale/NXP IFC controller is available on LS1021A, the
> > dependency for LS1021A is added.
> >
> > 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 v3:
> > - Update the commit message.
> >
> > 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"
Boris Brezillon Feb. 13, 2017, 7:55 a.m. UTC | #3 
On Mon, 13 Feb 2017 07:39:41 +0000
Alison Wang <alison.wang@nxp.com> wrote:

> Hi, Boris,
> 
> 	Sorry, I forgot to change them. Should I resend them or you help to fix that when applying?

I can fix that when applying, no need to resend.
Note that you missed 4.11 (already sent my PR to Brian), I'll queue it
for 4.12.

Regards,

Boris

> 
> 
> Best Regards,
> Alison Wang
> 
> > 
> > Hi Alison,
> > 
> > The subject prefix is still wrong, should be 'memory: ifc: '.
> > 
> > On Mon, 13 Feb 2017 14:46:55 +0800
> > Alison Wang <b18965@freescale.com> wrote:
> >   
> > > As Freescale/NXP IFC controller is available on LS1021A, the
> > > dependency for LS1021A is added.
> > >
> > > 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 v3:
> > > - Update the commit message.
> > >
> > > 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"  
>
Alison Wang Feb. 13, 2017, 8:01 a.m. UTC | #4 
> On Mon, 13 Feb 2017 07:39:41 +0000
> Alison Wang <alison.wang@nxp.com> wrote:
> 
> > Hi, Boris,
> >
> > 	Sorry, I forgot to change them. Should I resend them or you help
> to fix that when applying?
> 
> I can fix that when applying, no need to resend.
> Note that you missed 4.11 (already sent my PR to Brian), I'll queue it
> for 4.12.
> 
[Alison Wang] Ok, I see. Thanks a lot.


Best Regards,
Alison Wang
> 
> 
> >
> >
> > Best Regards,
> > Alison Wang
> >
> > >
> > > Hi Alison,
> > >
> > > The subject prefix is still wrong, should be 'memory: ifc: '.
> > >
> > > On Mon, 13 Feb 2017 14:46:55 +0800
> > > Alison Wang <b18965@freescale.com> wrote:
> > >
> > > > As Freescale/NXP IFC controller is available on LS1021A, the
> > > > dependency for LS1021A is added.
> > > >
> > > > 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 v3:
> > > > - Update the commit message.
> > > >
> > > > 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"
> >
Boris Brezillon March 16, 2017, 9:31 a.m. UTC | #5 
On Mon, 13 Feb 2017 08:01:17 +0000
Alison Wang <alison.wang@nxp.com> wrote:

> > On Mon, 13 Feb 2017 07:39:41 +0000
> > Alison Wang <alison.wang@nxp.com> wrote:
> >   
> > > Hi, Boris,
> > >
> > > 	Sorry, I forgot to change them. Should I resend them or you help  
> > to fix that when applying?
> > 
> > I can fix that when applying, no need to resend.
> > Note that you missed 4.11 (already sent my PR to Brian), I'll queue it
> > for 4.12.
> >   
> [Alison Wang] Ok, I see. Thanks a lot.

Applied both.

Thanks,

Boris
diff mbox

Patch

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"