Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1579482 > unrolled thread

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

Started byBoris Brezillon <boris.brezillon@free-electrons.com>
First post2017-02-13 08:40 +0100
Last post2017-02-13 09:10 +0100
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v3 1/2] mtd: ifc: Update dependency of IFC for LS1021A Boris Brezillon <boris.brezillon@free-electrons.com> - 2017-02-13 08:40 +0100
    RE: [PATCH v3 1/2] mtd: ifc: Update dependency of IFC for LS1021A Alison Wang <alison.wang@nxp.com> - 2017-02-13 08:40 +0100
      Re: [PATCH v3 1/2] mtd: ifc: Update dependency of IFC for LS1021A Boris Brezillon <boris.brezillon@free-electrons.com> - 2017-02-13 09:00 +0100
        RE: [PATCH v3 1/2] mtd: ifc: Update dependency of IFC for LS1021A Alison Wang <alison.wang@nxp.com> - 2017-02-13 09:10 +0100

#1579482 — Re: [PATCH v3 1/2] mtd: ifc: Update dependency of IFC for LS1021A

FromBoris Brezillon <boris.brezillon@free-electrons.com>
Date2017-02-13 08:40 +0100
SubjectRe: [PATCH v3 1/2] mtd: ifc: Update dependency of IFC for LS1021A
Message-ID<tajoS-EX-15@gated-at.bofh.it>
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"

[toc] | [next] | [standalone]

#1579484

FromAlison Wang <alison.wang@nxp.com>
Date2017-02-13 08:40 +0100
Message-ID<tajoS-EX-11@gated-at.bofh.it>
In reply to#1579482
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"

[toc] | [prev] | [next] | [standalone]

#1579494

FromBoris Brezillon <boris.brezillon@free-electrons.com>
Date2017-02-13 09:00 +0100
Message-ID<tajId-M9-3@gated-at.bofh.it>
In reply to#1579484
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"  
>

[toc] | [prev] | [next] | [standalone]

#1579501

FromAlison Wang <alison.wang@nxp.com>
Date2017-02-13 09:10 +0100
Message-ID<tajRT-14I-7@gated-at.bofh.it>
In reply to#1579494
> 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"
> >

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web