Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 24/01/2017 11:31, Boris Brezillon wrote: > On Sun, 22 Jan 2017 10:36:40 +0800 > Guochun Mao wrote: > >> Hi, >> On Thu, 2017-01-19 at 08:18 -0600, Rob Herring wrote: >>> On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon One last question and I'm done: is

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 24/01/2017 11:31, Boris Brezillon wrote: > On Sun, 22 Jan 2017 10:36:40 +0800 > Guochun Mao wrote: > >> Hi, >> On Thu, 2017-01-19 at 08:18 -0600, Rob Herring wrote: >>> On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon One last question and I'm done: is something like that acceptable?

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Sun, 22 Jan 2017 10:36:40 +0800 Guochun Mao wrote: > Hi, > On Thu, 2017-01-19 at 08:18 -0600, Rob Herring wrote: > > On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon > > > One last question and I'm done: is something like that acceptable? > > > > > >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Sun, 22 Jan 2017 10:36:40 +0800 Guochun Mao wrote: > Hi, > On Thu, 2017-01-19 at 08:18 -0600, Rob Herring wrote: > > On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon > > > One last question and I'm done: is something like that acceptable? > > > > > > compatible = ",",","; > > > > >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hi, On Thu, 2017-01-19 at 08:18 -0600, Rob Herring wrote: > On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon > > One last question and I'm done: is something like that acceptable? > > > > compatible = ",",","; > > > > This can happen when someone adds support for an unsupported feature > >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hi, On Thu, 2017-01-19 at 08:18 -0600, Rob Herring wrote: > On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon > > One last question and I'm done: is something like that acceptable? > > > > compatible = ",",","; > > > > This can happen when someone adds support for an unsupported feature > >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon wrote: > Hi Rob, > > On Wed, 18 Jan 2017 20:51:08 -0600 > Rob Herring wrote: > >> On Wed, Jan 18, 2017 at 5:38 PM, Thomas Petazzoni >> wrote: >> > Hello, >>

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Thu, Jan 19, 2017 at 2:14 AM, Boris Brezillon wrote: > Hi Rob, > > On Wed, 18 Jan 2017 20:51:08 -0600 > Rob Herring wrote: > >> On Wed, Jan 18, 2017 at 5:38 PM, Thomas Petazzoni >> wrote: >> > Hello, >> > >> > On Wed, 18 Jan 2017 16:20:10 -0600, Rob Herring wrote: >> > >> >> > > Rob, Mark,

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hi Rob, On Wed, 18 Jan 2017 20:51:08 -0600 Rob Herring wrote: > On Wed, Jan 18, 2017 at 5:38 PM, Thomas Petazzoni > wrote: > > Hello, > > > > On Wed, 18 Jan 2017 16:20:10 -0600, Rob Herring wrote: > > > >> > > Rob, Mark, any opinion? >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hi Rob, On Wed, 18 Jan 2017 20:51:08 -0600 Rob Herring wrote: > On Wed, Jan 18, 2017 at 5:38 PM, Thomas Petazzoni > wrote: > > Hello, > > > > On Wed, 18 Jan 2017 16:20:10 -0600, Rob Herring wrote: > > > >> > > Rob, Mark, any opinion? > >> > > >> > >> Sigh, is how to do compatibles really

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Wed, 18 Jan 2017 16:20:10 -0600 Rob Herring wrote: > On Tue, Jan 17, 2017 at 02:36:50PM +1100, Thomas Petazzoni wrote: > > Hello, > > > > (Side note: you guys should learn about stripping irrelevant parts of > > an e-mail when replying!) > > > > On Mon, 16 Jan 2017 09:40:32

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Wed, 18 Jan 2017 16:20:10 -0600 Rob Herring wrote: > On Tue, Jan 17, 2017 at 02:36:50PM +1100, Thomas Petazzoni wrote: > > Hello, > > > > (Side note: you guys should learn about stripping irrelevant parts of > > an e-mail when replying!) > > > > On Mon, 16 Jan 2017 09:40:32 +0100, Boris

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Wed, Jan 18, 2017 at 5:38 PM, Thomas Petazzoni wrote: > Hello, > > On Wed, 18 Jan 2017 16:20:10 -0600, Rob Herring wrote: > >> > > Rob, Mark, any opinion? >> > >> >> Sigh, is how to do compatibles really not yet understood? > > Well, it seems like not

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Wed, Jan 18, 2017 at 5:38 PM, Thomas Petazzoni wrote: > Hello, > > On Wed, 18 Jan 2017 16:20:10 -0600, Rob Herring wrote: > >> > > Rob, Mark, any opinion? >> > >> >> Sigh, is how to do compatibles really not yet understood? > > Well, it seems like not everyone necessarily understands what is

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hello, On Wed, 18 Jan 2017 16:20:10 -0600, Rob Herring wrote: > > > Rob, Mark, any opinion? > > > > Sigh, is how to do compatibles really not yet understood? Well, it seems like not everyone necessarily understands what is the best strategy to adopt (me included). > > I agree that a

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hello, On Wed, 18 Jan 2017 16:20:10 -0600, Rob Herring wrote: > > > Rob, Mark, any opinion? > > > > Sigh, is how to do compatibles really not yet understood? Well, it seems like not everyone necessarily understands what is the best strategy to adopt (me included). > > I agree that a

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Tue, Jan 17, 2017 at 02:36:50PM +1100, Thomas Petazzoni wrote: > Hello, > > (Side note: you guys should learn about stripping irrelevant parts of > an e-mail when replying!) > > On Mon, 16 Jan 2017 09:40:32 +0100, Boris Brezillon wrote: > > > > Well this is OK I guess, but then you can also

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Tue, Jan 17, 2017 at 02:36:50PM +1100, Thomas Petazzoni wrote: > Hello, > > (Side note: you guys should learn about stripping irrelevant parts of > an e-mail when replying!) > > On Mon, 16 Jan 2017 09:40:32 +0100, Boris Brezillon wrote: > > > > Well this is OK I guess, but then you can also

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hello, (Side note: you guys should learn about stripping irrelevant parts of an e-mail when replying!) On Mon, 16 Jan 2017 09:40:32 +0100, Boris Brezillon wrote: > > Well this is OK I guess, but then you can also use "mediatek,mt8173-nor" > > as the oldest supported compatible and be done with

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hello, (Side note: you guys should learn about stripping irrelevant parts of an e-mail when replying!) On Mon, 16 Jan 2017 09:40:32 +0100, Boris Brezillon wrote: > > Well this is OK I guess, but then you can also use "mediatek,mt8173-nor" > > as the oldest supported compatible and be done with

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hello, On Fri, 13 Jan 2017 17:56:28 +0100, Boris Brezillon wrote: > because both compatible are referring to very specific IP version. It's > not the same as > > compatible = "mediatek,mt8173-nor", "mediatek,mt81xx-nor"; mt81xx-nor is a bogus compatible string, and DT binding maintainers

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Hello, On Fri, 13 Jan 2017 17:56:28 +0100, Boris Brezillon wrote: > because both compatible are referring to very specific IP version. It's > not the same as > > compatible = "mediatek,mt8173-nor", "mediatek,mt81xx-nor"; mt81xx-nor is a bogus compatible string, and DT binding maintainers

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/16/2017 09:40 AM, Boris Brezillon wrote: > On Sun, 15 Jan 2017 01:23:48 +0100 > Marek Vasut wrote: > >> On 01/14/2017 09:29 AM, Boris Brezillon wrote: >>> On Fri, 13 Jan 2017 18:33:40 +0100 >>> Marek Vasut wrote: >>> On 01/13/2017

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/16/2017 09:40 AM, Boris Brezillon wrote: > On Sun, 15 Jan 2017 01:23:48 +0100 > Marek Vasut wrote: > >> On 01/14/2017 09:29 AM, Boris Brezillon wrote: >>> On Fri, 13 Jan 2017 18:33:40 +0100 >>> Marek Vasut wrote: >>> On 01/13/2017 05:56 PM, Boris Brezillon wrote: > On Fri,

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Sun, 15 Jan 2017 01:23:48 +0100 Marek Vasut wrote: > On 01/14/2017 09:29 AM, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 18:33:40 +0100 > > Marek Vasut wrote: > > > >> On 01/13/2017 05:56 PM, Boris Brezillon wrote: > >>> On Fri, 13 Jan

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Sun, 15 Jan 2017 01:23:48 +0100 Marek Vasut wrote: > On 01/14/2017 09:29 AM, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 18:33:40 +0100 > > Marek Vasut wrote: > > > >> On 01/13/2017 05:56 PM, Boris Brezillon wrote: > >>> On Fri, 13 Jan 2017 17:44:12 +0100 > >>> Marek Vasut wrote: >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/14/2017 09:29 AM, Boris Brezillon wrote: > On Fri, 13 Jan 2017 18:33:40 +0100 > Marek Vasut wrote: > >> On 01/13/2017 05:56 PM, Boris Brezillon wrote: >>> On Fri, 13 Jan 2017 17:44:12 +0100 >>> Marek Vasut wrote: >>> On 01/13/2017

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/14/2017 09:29 AM, Boris Brezillon wrote: > On Fri, 13 Jan 2017 18:33:40 +0100 > Marek Vasut wrote: > >> On 01/13/2017 05:56 PM, Boris Brezillon wrote: >>> On Fri, 13 Jan 2017 17:44:12 +0100 >>> Marek Vasut wrote: >>> On 01/13/2017 05:28 PM, Boris Brezillon wrote: > On Fri,

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 18:33:40 +0100 Marek Vasut wrote: > On 01/13/2017 05:56 PM, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 17:44:12 +0100 > > Marek Vasut wrote: > > > >> On 01/13/2017 05:28 PM, Boris Brezillon wrote: > >>> On Fri, 13 Jan

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 18:33:40 +0100 Marek Vasut wrote: > On 01/13/2017 05:56 PM, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 17:44:12 +0100 > > Marek Vasut wrote: > > > >> On 01/13/2017 05:28 PM, Boris Brezillon wrote: > >>> On Fri, 13 Jan 2017 17:13:55 +0100 > >>> Marek Vasut wrote: >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 05:56 PM, Boris Brezillon wrote: > On Fri, 13 Jan 2017 17:44:12 +0100 > Marek Vasut wrote: > >> On 01/13/2017 05:28 PM, Boris Brezillon wrote: >>> On Fri, 13 Jan 2017 17:13:55 +0100 >>> Marek Vasut wrote: >>> On 01/13/2017

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 05:56 PM, Boris Brezillon wrote: > On Fri, 13 Jan 2017 17:44:12 +0100 > Marek Vasut wrote: > >> On 01/13/2017 05:28 PM, Boris Brezillon wrote: >>> On Fri, 13 Jan 2017 17:13:55 +0100 >>> Marek Vasut wrote: >>> On 01/13/2017 04:12 PM, Matthias Brugger wrote: > >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 17:44:12 +0100 Marek Vasut wrote: > On 01/13/2017 05:28 PM, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 17:13:55 +0100 > > Marek Vasut wrote: > > > >> On 01/13/2017 04:12 PM, Matthias Brugger wrote: > >>> > >>> > >>> On

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 17:44:12 +0100 Marek Vasut wrote: > On 01/13/2017 05:28 PM, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 17:13:55 +0100 > > Marek Vasut wrote: > > > >> On 01/13/2017 04:12 PM, Matthias Brugger wrote: > >>> > >>> > >>> On 13/01/17 15:17, Boris Brezillon wrote: >

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 05:28 PM, Boris Brezillon wrote: > On Fri, 13 Jan 2017 17:13:55 +0100 > Marek Vasut wrote: > >> On 01/13/2017 04:12 PM, Matthias Brugger wrote: >>> >>> >>> On 13/01/17 15:17, Boris Brezillon wrote: On Fri, 13 Jan 2017 15:13:29 +0800 Guochun Mao

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 05:28 PM, Boris Brezillon wrote: > On Fri, 13 Jan 2017 17:13:55 +0100 > Marek Vasut wrote: > >> On 01/13/2017 04:12 PM, Matthias Brugger wrote: >>> >>> >>> On 13/01/17 15:17, Boris Brezillon wrote: On Fri, 13 Jan 2017 15:13:29 +0800 Guochun Mao wrote: > Add

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 17:13:55 +0100 Marek Vasut wrote: > On 01/13/2017 04:12 PM, Matthias Brugger wrote: > > > > > > On 13/01/17 15:17, Boris Brezillon wrote: > >> On Fri, 13 Jan 2017 15:13:29 +0800 > >> Guochun Mao wrote: > >> > >>> Add

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 17:13:55 +0100 Marek Vasut wrote: > On 01/13/2017 04:12 PM, Matthias Brugger wrote: > > > > > > On 13/01/17 15:17, Boris Brezillon wrote: > >> On Fri, 13 Jan 2017 15:13:29 +0800 > >> Guochun Mao wrote: > >> > >>> Add Mediatek nor flash node. > >>> > >>> Signed-off-by:

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 04:12 PM, Matthias Brugger wrote: > > > On 13/01/17 15:17, Boris Brezillon wrote: >> On Fri, 13 Jan 2017 15:13:29 +0800 >> Guochun Mao wrote: >> >>> Add Mediatek nor flash node. >>> >>> Signed-off-by: Guochun Mao >>> --- >>>

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 04:12 PM, Matthias Brugger wrote: > > > On 13/01/17 15:17, Boris Brezillon wrote: >> On Fri, 13 Jan 2017 15:13:29 +0800 >> Guochun Mao wrote: >> >>> Add Mediatek nor flash node. >>> >>> Signed-off-by: Guochun Mao >>> --- >>> arch/arm/boot/dts/mt2701-evb.dts | 25

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 16:12:20 +0100 Matthias Brugger wrote: > On 13/01/17 15:17, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 15:13:29 +0800 > > Guochun Mao wrote: > > > >> Add Mediatek nor flash node. > >> > >> Signed-off-by: Guochun Mao

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 16:12:20 +0100 Matthias Brugger wrote: > On 13/01/17 15:17, Boris Brezillon wrote: > > On Fri, 13 Jan 2017 15:13:29 +0800 > > Guochun Mao wrote: > > > >> Add Mediatek nor flash node. > >> > >> Signed-off-by: Guochun Mao > >> --- > >> arch/arm/boot/dts/mt2701-evb.dts |

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 13/01/17 15:17, Boris Brezillon wrote: On Fri, 13 Jan 2017 15:13:29 +0800 Guochun Mao wrote: Add Mediatek nor flash node. Signed-off-by: Guochun Mao --- arch/arm/boot/dts/mt2701-evb.dts | 25 +

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 13/01/17 15:17, Boris Brezillon wrote: On Fri, 13 Jan 2017 15:13:29 +0800 Guochun Mao wrote: Add Mediatek nor flash node. Signed-off-by: Guochun Mao --- arch/arm/boot/dts/mt2701-evb.dts | 25 + arch/arm/boot/dts/mt2701.dtsi| 12 2 files

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 15:13:29 +0800 Guochun Mao wrote: > Add Mediatek nor flash node. > > Signed-off-by: Guochun Mao > --- > arch/arm/boot/dts/mt2701-evb.dts | 25 + > arch/arm/boot/dts/mt2701.dtsi| 12

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On Fri, 13 Jan 2017 15:13:29 +0800 Guochun Mao wrote: > Add Mediatek nor flash node. > > Signed-off-by: Guochun Mao > --- > arch/arm/boot/dts/mt2701-evb.dts | 25 + > arch/arm/boot/dts/mt2701.dtsi| 12 > 2 files changed, 37 insertions(+) > > diff

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 08:13 AM, Guochun Mao wrote: > Add Mediatek nor flash node. > > Signed-off-by: Guochun Mao > --- > arch/arm/boot/dts/mt2701-evb.dts | 25 + > arch/arm/boot/dts/mt2701.dtsi| 12 > 2 files changed, 37

Re: [PATCH v1 2/2] arm: dts: mt2701: add nor flash node

On 01/13/2017 08:13 AM, Guochun Mao wrote: > Add Mediatek nor flash node. > > Signed-off-by: Guochun Mao > --- > arch/arm/boot/dts/mt2701-evb.dts | 25 + > arch/arm/boot/dts/mt2701.dtsi| 12 > 2 files changed, 37 insertions(+) > > diff --git

[PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Add Mediatek nor flash node. Signed-off-by: Guochun Mao --- arch/arm/boot/dts/mt2701-evb.dts | 25 + arch/arm/boot/dts/mt2701.dtsi| 12 2 files changed, 37 insertions(+) diff --git a/arch/arm/boot/dts/mt2701-evb.dts

[PATCH v1 2/2] arm: dts: mt2701: add nor flash node

Add Mediatek nor flash node. Signed-off-by: Guochun Mao --- arch/arm/boot/dts/mt2701-evb.dts | 25 + arch/arm/boot/dts/mt2701.dtsi| 12 2 files changed, 37 insertions(+) diff --git a/arch/arm/boot/dts/mt2701-evb.dts