Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Shawn Guo
On Mon, Jul 16, 2018 at 09:06:51PM -0700, Andrey Smirnov wrote:
> Add support for Zodiac Inflight Innovations SSMB SPU3
> board (VF610-based).
> 
> Cc: Shawn Guo 
> Cc: Fabio Estevam 
> Cc: cphe...@gmail.com
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: devicet...@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Andrew Lunn 
> Signed-off-by: Andrey Smirnov 

Applied, thanks.


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Shawn Guo
On Mon, Jul 16, 2018 at 09:06:51PM -0700, Andrey Smirnov wrote:
> Add support for Zodiac Inflight Innovations SSMB SPU3
> board (VF610-based).
> 
> Cc: Shawn Guo 
> Cc: Fabio Estevam 
> Cc: cphe...@gmail.com
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: devicet...@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Andrew Lunn 
> Signed-off-by: Andrey Smirnov 

Applied, thanks.


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Fabio Estevam
Hi Andrey,

On Wed, Jul 18, 2018 at 9:51 PM, Andrey Smirnov
 wrote:

> It's not used in Linux, but I do rely on it in Barebox here:
>
> https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n94
>
> and here:
>
> https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n137
>
> so it's pretty handy to have. I can remove it if it is too much of an eyesore.

Thanks for the clarification. If you use it in Barebox, then it's fine
to keep it.


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Fabio Estevam
Hi Andrey,

On Wed, Jul 18, 2018 at 9:51 PM, Andrey Smirnov
 wrote:

> It's not used in Linux, but I do rely on it in Barebox here:
>
> https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n94
>
> and here:
>
> https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n137
>
> so it's pretty handy to have. I can remove it if it is too much of an eyesore.

Thanks for the clarification. If you use it in Barebox, then it's fine
to keep it.


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Andrey Smirnov
On Wed, Jul 18, 2018 at 3:50 PM Fabio Estevam  wrote:
>
> Hi Andrey,
>
> On Tue, Jul 17, 2018 at 1:06 AM, Andrey Smirnov
>  wrote:
>
> > +/dts-v1/;
> > +#include "vf610.dtsi"
> > +
> > +/ {
> > +   model = "ZII VF610 SSMB SPU3 Board";
> > +   compatible = "zii,vf610spu3", "zii,vf610dev", "fsl,vf610";
>
> Looks good to me.
>
> Just a minor comment: Is " "zii,vf610dev" really needed here?
>

It's not used in Linux, but I do rely on it in Barebox here:

https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n94

and here:

https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n137

so it's pretty handy to have. I can remove it if it is too much of an eyesore.

Thanks,
Andrey Smirnov


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Andrey Smirnov
On Wed, Jul 18, 2018 at 3:50 PM Fabio Estevam  wrote:
>
> Hi Andrey,
>
> On Tue, Jul 17, 2018 at 1:06 AM, Andrey Smirnov
>  wrote:
>
> > +/dts-v1/;
> > +#include "vf610.dtsi"
> > +
> > +/ {
> > +   model = "ZII VF610 SSMB SPU3 Board";
> > +   compatible = "zii,vf610spu3", "zii,vf610dev", "fsl,vf610";
>
> Looks good to me.
>
> Just a minor comment: Is " "zii,vf610dev" really needed here?
>

It's not used in Linux, but I do rely on it in Barebox here:

https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n94

and here:

https://git.pengutronix.de/cgit/barebox/tree/arch/arm/boards/zii-vf610-dev/board.c#n137

so it's pretty handy to have. I can remove it if it is too much of an eyesore.

Thanks,
Andrey Smirnov


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Fabio Estevam
Hi Andrey,

On Tue, Jul 17, 2018 at 1:06 AM, Andrey Smirnov
 wrote:

> +/dts-v1/;
> +#include "vf610.dtsi"
> +
> +/ {
> +   model = "ZII VF610 SSMB SPU3 Board";
> +   compatible = "zii,vf610spu3", "zii,vf610dev", "fsl,vf610";

Looks good to me.

Just a minor comment: Is " "zii,vf610dev" really needed here?

Anyway:

Reviewed-by: Fabio Estevam 


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-18 Thread Fabio Estevam
Hi Andrey,

On Tue, Jul 17, 2018 at 1:06 AM, Andrey Smirnov
 wrote:

> +/dts-v1/;
> +#include "vf610.dtsi"
> +
> +/ {
> +   model = "ZII VF610 SSMB SPU3 Board";
> +   compatible = "zii,vf610spu3", "zii,vf610dev", "fsl,vf610";

Looks good to me.

Just a minor comment: Is " "zii,vf610dev" really needed here?

Anyway:

Reviewed-by: Fabio Estevam 


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Chris Healy
On Tue, Jul 17, 2018 at 8:20 AM, Andrew Lunn  wrote:
>> >> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>> >> @@ -0,0 +1,343 @@
>> >> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
>> >> +
>> >> +/*
>> >> + * Device tree file for ZII's SSMB SPU3 board
>> >> + *
>> >> + * SSMB - SPU3 Switch Management Board
>> >> + * SPU - Seat Power Unit
>> >
>> > I think this is the first Zodiac board with mutually recursive
>> > acronyms.
>> >
>> > Probably a question for Chris: Does SSMB specifically refer to version
>> > 3 of the SPU?
>> >
>> >Andrew
>>
>> SPU3 is the third generation of seat power unit.
>> SSMB is the SPU3 Switch Management Board.
>>
>> So, SSMB is a backronym.  This naming scheme breaks down a bit though
>> when follow on designs (that are not the SPU3) also use the SSMB, but
>> that's what it is... ;-)
>
> O.K, i was just wondering if the 3 was in the wrong place. More
> logically, it would be
>
> * SSMB - SPU Switch Management Board
> * SPU 3 - Seat Power Unit, version 3.
>
> But you are saying the Marketing guys messed up the naming and
> engineering is now stuck with it.
>
Yep


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Chris Healy
On Tue, Jul 17, 2018 at 8:20 AM, Andrew Lunn  wrote:
>> >> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>> >> @@ -0,0 +1,343 @@
>> >> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
>> >> +
>> >> +/*
>> >> + * Device tree file for ZII's SSMB SPU3 board
>> >> + *
>> >> + * SSMB - SPU3 Switch Management Board
>> >> + * SPU - Seat Power Unit
>> >
>> > I think this is the first Zodiac board with mutually recursive
>> > acronyms.
>> >
>> > Probably a question for Chris: Does SSMB specifically refer to version
>> > 3 of the SPU?
>> >
>> >Andrew
>>
>> SPU3 is the third generation of seat power unit.
>> SSMB is the SPU3 Switch Management Board.
>>
>> So, SSMB is a backronym.  This naming scheme breaks down a bit though
>> when follow on designs (that are not the SPU3) also use the SSMB, but
>> that's what it is... ;-)
>
> O.K, i was just wondering if the 3 was in the wrong place. More
> logically, it would be
>
> * SSMB - SPU Switch Management Board
> * SPU 3 - Seat Power Unit, version 3.
>
> But you are saying the Marketing guys messed up the naming and
> engineering is now stuck with it.
>
Yep


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Andrew Lunn
> >> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> >> @@ -0,0 +1,343 @@
> >> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> >> +
> >> +/*
> >> + * Device tree file for ZII's SSMB SPU3 board
> >> + *
> >> + * SSMB - SPU3 Switch Management Board
> >> + * SPU - Seat Power Unit
> >
> > I think this is the first Zodiac board with mutually recursive
> > acronyms.
> >
> > Probably a question for Chris: Does SSMB specifically refer to version
> > 3 of the SPU?
> >
> >Andrew
> 
> SPU3 is the third generation of seat power unit.
> SSMB is the SPU3 Switch Management Board.
> 
> So, SSMB is a backronym.  This naming scheme breaks down a bit though
> when follow on designs (that are not the SPU3) also use the SSMB, but
> that's what it is... ;-)

O.K, i was just wondering if the 3 was in the wrong place. More
logically, it would be

* SSMB - SPU Switch Management Board
* SPU 3 - Seat Power Unit, version 3.

But you are saying the Marketing guys messed up the naming and
engineering is now stuck with it.

Andrew


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Andrew Lunn
> >> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> >> @@ -0,0 +1,343 @@
> >> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> >> +
> >> +/*
> >> + * Device tree file for ZII's SSMB SPU3 board
> >> + *
> >> + * SSMB - SPU3 Switch Management Board
> >> + * SPU - Seat Power Unit
> >
> > I think this is the first Zodiac board with mutually recursive
> > acronyms.
> >
> > Probably a question for Chris: Does SSMB specifically refer to version
> > 3 of the SPU?
> >
> >Andrew
> 
> SPU3 is the third generation of seat power unit.
> SSMB is the SPU3 Switch Management Board.
> 
> So, SSMB is a backronym.  This naming scheme breaks down a bit though
> when follow on designs (that are not the SPU3) also use the SSMB, but
> that's what it is... ;-)

O.K, i was just wondering if the 3 was in the wrong place. More
logically, it would be

* SSMB - SPU Switch Management Board
* SPU 3 - Seat Power Unit, version 3.

But you are saying the Marketing guys messed up the naming and
engineering is now stuck with it.

Andrew


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Chris Healy
On Mon, Jul 16, 2018 at 9:06 PM, Andrey Smirnov
 wrote:
> Add support for Zodiac Inflight Innovations SSMB SPU3
> board (VF610-based).
>
> Cc: Shawn Guo 
> Cc: Fabio Estevam 
> Cc: cphe...@gmail.com
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: devicet...@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Andrew Lunn 
> Signed-off-by: Andrey Smirnov 
> ---
>  arch/arm/boot/dts/Makefile|   3 +-
>  arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
>  2 files changed, 345 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index bea41b129493..e331b2c16539 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
> vf610m4-cosmic.dtb \
> vf610-twr.dtb \
> vf610-zii-dev-rev-b.dtb \
> -   vf610-zii-dev-rev-c.dtb
> +   vf610-zii-dev-rev-c.dtb \
> +   vf610-zii-ssmb-spu3.dtb


Tested-by: Chris Healy 


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Chris Healy
On Mon, Jul 16, 2018 at 9:06 PM, Andrey Smirnov
 wrote:
> Add support for Zodiac Inflight Innovations SSMB SPU3
> board (VF610-based).
>
> Cc: Shawn Guo 
> Cc: Fabio Estevam 
> Cc: cphe...@gmail.com
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: devicet...@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Andrew Lunn 
> Signed-off-by: Andrey Smirnov 
> ---
>  arch/arm/boot/dts/Makefile|   3 +-
>  arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
>  2 files changed, 345 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index bea41b129493..e331b2c16539 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
> vf610m4-cosmic.dtb \
> vf610-twr.dtb \
> vf610-zii-dev-rev-b.dtb \
> -   vf610-zii-dev-rev-c.dtb
> +   vf610-zii-dev-rev-c.dtb \
> +   vf610-zii-ssmb-spu3.dtb


Tested-by: Chris Healy 


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Chris Healy
On Tue, Jul 17, 2018 at 7:46 AM, Andrew Lunn  wrote:
> On Mon, Jul 16, 2018 at 09:06:51PM -0700, Andrey Smirnov wrote:
>> Add support for Zodiac Inflight Innovations SSMB SPU3
>> board (VF610-based).
>>
>> Cc: Shawn Guo 
>> Cc: Fabio Estevam 
>> Cc: cphe...@gmail.com
>> Cc: linux-arm-ker...@lists.infradead.org
>> Cc: devicet...@vger.kernel.org
>> Cc: linux-kernel@vger.kernel.org
>> Signed-off-by: Andrew Lunn 
>> Signed-off-by: Andrey Smirnov 
>> ---
>>  arch/arm/boot/dts/Makefile|   3 +-
>>  arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
>>  2 files changed, 345 insertions(+), 1 deletion(-)
>>  create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index bea41b129493..e331b2c16539 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
>>   vf610m4-cosmic.dtb \
>>   vf610-twr.dtb \
>>   vf610-zii-dev-rev-b.dtb \
>> - vf610-zii-dev-rev-c.dtb
>> + vf610-zii-dev-rev-c.dtb \
>> + vf610-zii-ssmb-spu3.dtb
>>  dtb-$(CONFIG_ARCH_MXS) += \
>>   imx23-evk.dtb \
>>   imx23-olinuxino.dtb \
>> diff --git a/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts 
>> b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>> new file mode 100644
>> index ..b692117d7839
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>> @@ -0,0 +1,343 @@
>> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
>> +
>> +/*
>> + * Device tree file for ZII's SSMB SPU3 board
>> + *
>> + * SSMB - SPU3 Switch Management Board
>> + * SPU - Seat Power Unit
>
> I think this is the first Zodiac board with mutually recursive
> acronyms.
>
> Probably a question for Chris: Does SSMB specifically refer to version
> 3 of the SPU?
>
>Andrew

SPU3 is the third generation of seat power unit.
SSMB is the SPU3 Switch Management Board.

So, SSMB is a backronym.  This naming scheme breaks down a bit though
when follow on designs (that are not the SPU3) also use the SSMB, but
that's what it is... ;-)


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Chris Healy
On Tue, Jul 17, 2018 at 7:46 AM, Andrew Lunn  wrote:
> On Mon, Jul 16, 2018 at 09:06:51PM -0700, Andrey Smirnov wrote:
>> Add support for Zodiac Inflight Innovations SSMB SPU3
>> board (VF610-based).
>>
>> Cc: Shawn Guo 
>> Cc: Fabio Estevam 
>> Cc: cphe...@gmail.com
>> Cc: linux-arm-ker...@lists.infradead.org
>> Cc: devicet...@vger.kernel.org
>> Cc: linux-kernel@vger.kernel.org
>> Signed-off-by: Andrew Lunn 
>> Signed-off-by: Andrey Smirnov 
>> ---
>>  arch/arm/boot/dts/Makefile|   3 +-
>>  arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
>>  2 files changed, 345 insertions(+), 1 deletion(-)
>>  create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index bea41b129493..e331b2c16539 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
>>   vf610m4-cosmic.dtb \
>>   vf610-twr.dtb \
>>   vf610-zii-dev-rev-b.dtb \
>> - vf610-zii-dev-rev-c.dtb
>> + vf610-zii-dev-rev-c.dtb \
>> + vf610-zii-ssmb-spu3.dtb
>>  dtb-$(CONFIG_ARCH_MXS) += \
>>   imx23-evk.dtb \
>>   imx23-olinuxino.dtb \
>> diff --git a/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts 
>> b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>> new file mode 100644
>> index ..b692117d7839
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
>> @@ -0,0 +1,343 @@
>> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
>> +
>> +/*
>> + * Device tree file for ZII's SSMB SPU3 board
>> + *
>> + * SSMB - SPU3 Switch Management Board
>> + * SPU - Seat Power Unit
>
> I think this is the first Zodiac board with mutually recursive
> acronyms.
>
> Probably a question for Chris: Does SSMB specifically refer to version
> 3 of the SPU?
>
>Andrew

SPU3 is the third generation of seat power unit.
SSMB is the SPU3 Switch Management Board.

So, SSMB is a backronym.  This naming scheme breaks down a bit though
when follow on designs (that are not the SPU3) also use the SSMB, but
that's what it is... ;-)


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Andrew Lunn
On Mon, Jul 16, 2018 at 09:06:51PM -0700, Andrey Smirnov wrote:
> Add support for Zodiac Inflight Innovations SSMB SPU3
> board (VF610-based).
> 
> Cc: Shawn Guo 
> Cc: Fabio Estevam 
> Cc: cphe...@gmail.com
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: devicet...@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Andrew Lunn 
> Signed-off-by: Andrey Smirnov 
> ---
>  arch/arm/boot/dts/Makefile|   3 +-
>  arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
>  2 files changed, 345 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> 
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index bea41b129493..e331b2c16539 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
>   vf610m4-cosmic.dtb \
>   vf610-twr.dtb \
>   vf610-zii-dev-rev-b.dtb \
> - vf610-zii-dev-rev-c.dtb
> + vf610-zii-dev-rev-c.dtb \
> + vf610-zii-ssmb-spu3.dtb
>  dtb-$(CONFIG_ARCH_MXS) += \
>   imx23-evk.dtb \
>   imx23-olinuxino.dtb \
> diff --git a/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts 
> b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> new file mode 100644
> index ..b692117d7839
> --- /dev/null
> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> @@ -0,0 +1,343 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +
> +/*
> + * Device tree file for ZII's SSMB SPU3 board
> + *
> + * SSMB - SPU3 Switch Management Board
> + * SPU - Seat Power Unit

I think this is the first Zodiac board with mutually recursive
acronyms.

Probably a question for Chris: Does SSMB specifically refer to version
3 of the SPU?

   Andrew


Re: [PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-17 Thread Andrew Lunn
On Mon, Jul 16, 2018 at 09:06:51PM -0700, Andrey Smirnov wrote:
> Add support for Zodiac Inflight Innovations SSMB SPU3
> board (VF610-based).
> 
> Cc: Shawn Guo 
> Cc: Fabio Estevam 
> Cc: cphe...@gmail.com
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: devicet...@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Andrew Lunn 
> Signed-off-by: Andrey Smirnov 
> ---
>  arch/arm/boot/dts/Makefile|   3 +-
>  arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
>  2 files changed, 345 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> 
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index bea41b129493..e331b2c16539 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
>   vf610m4-cosmic.dtb \
>   vf610-twr.dtb \
>   vf610-zii-dev-rev-b.dtb \
> - vf610-zii-dev-rev-c.dtb
> + vf610-zii-dev-rev-c.dtb \
> + vf610-zii-ssmb-spu3.dtb
>  dtb-$(CONFIG_ARCH_MXS) += \
>   imx23-evk.dtb \
>   imx23-olinuxino.dtb \
> diff --git a/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts 
> b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> new file mode 100644
> index ..b692117d7839
> --- /dev/null
> +++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
> @@ -0,0 +1,343 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +
> +/*
> + * Device tree file for ZII's SSMB SPU3 board
> + *
> + * SSMB - SPU3 Switch Management Board
> + * SPU - Seat Power Unit

I think this is the first Zodiac board with mutually recursive
acronyms.

Probably a question for Chris: Does SSMB specifically refer to version
3 of the SPU?

   Andrew


[PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-16 Thread Andrey Smirnov
Add support for Zodiac Inflight Innovations SSMB SPU3
board (VF610-based).

Cc: Shawn Guo 
Cc: Fabio Estevam 
Cc: cphe...@gmail.com
Cc: linux-arm-ker...@lists.infradead.org
Cc: devicet...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Andrew Lunn 
Signed-off-by: Andrey Smirnov 
---
 arch/arm/boot/dts/Makefile|   3 +-
 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
 2 files changed, 345 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index bea41b129493..e331b2c16539 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
vf610m4-cosmic.dtb \
vf610-twr.dtb \
vf610-zii-dev-rev-b.dtb \
-   vf610-zii-dev-rev-c.dtb
+   vf610-zii-dev-rev-c.dtb \
+   vf610-zii-ssmb-spu3.dtb
 dtb-$(CONFIG_ARCH_MXS) += \
imx23-evk.dtb \
imx23-olinuxino.dtb \
diff --git a/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts 
b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
new file mode 100644
index ..b692117d7839
--- /dev/null
+++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
@@ -0,0 +1,343 @@
+// SPDX-License-Identifier: (GPL-2.0 OR MIT)
+
+/*
+ * Device tree file for ZII's SSMB SPU3 board
+ *
+ * SSMB - SPU3 Switch Management Board
+ * SPU - Seat Power Unit
+ *
+ * Copyright (C) 2015, 2016 Zodiac Inflight Innovations
+ *
+ * Based on an original 'vf610-twr.dts' which is Copyright 2015,
+ * Freescale Semiconductor, Inc.
+ */
+
+/dts-v1/;
+#include "vf610.dtsi"
+
+/ {
+   model = "ZII VF610 SSMB SPU3 Board";
+   compatible = "zii,vf610spu3", "zii,vf610dev", "fsl,vf610";
+
+   chosen {
+   stdout-path = 
+   };
+
+   memory {
+   reg = <0x8000 0x2000>;
+   };
+
+   gpio-leds {
+   compatible = "gpio-leds";
+   pinctrl-0 = <_leds_debug>;
+   pinctrl-names = "default";
+
+   led-debug {
+   label = "zii:green:debug1";
+   gpios = < 18 GPIO_ACTIVE_HIGH>;
+   linux,default-trigger = "heartbeat";
+   max-brightness = <1>;
+   };
+   };
+
+   reg_vcc_3v3_mcu: regulator {
+   compatible = "regulator-fixed";
+   regulator-name = "vcc_3v3_mcu";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   };
+};
+
+ {
+   vref-supply = <_vcc_3v3_mcu>;
+   status = "okay";
+};
+
+ {
+   vref-supply = <_vcc_3v3_mcu>;
+   status = "okay";
+};
+
+ {
+   bus-num = <1>;
+   pinctrl-names = "default";
+   pinctrl-0 = <_dspi1>;
+   /*
+* Some SPU3s come with SPI-NOR chip DNPed, so we leave this
+* node disabled by default and rely on bootloader to enable
+* it when appropriate.
+*/
+   status = "disabled";
+
+   m25p128@0 {
+   #address-cells = <1>;
+   #size-cells = <1>;
+   compatible = "m25p128", "jedec,spi-nor";
+   reg = <0>;
+   spi-max-frequency = <5000>;
+
+   partition@0 {
+   label = "m25p128-0";
+   reg = <0x0 0x0100>;
+   };
+   };
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   pinctrl-names = "default";
+   pinctrl-0 = <_esdhc0>;
+   bus-width = <8>;
+   non-removable;
+   no-1-8-v;
+   keep-power-in-suspend;
+   status = "okay";
+};
+
+ {
+   pinctrl-names = "default";
+   pinctrl-0 = <_esdhc1>;
+   bus-width = <4>;
+   status = "okay";
+};
+
+ {
+   phy-mode = "rmii";
+   pinctrl-names = "default";
+   pinctrl-0 = <_fec1>;
+   status = "okay";
+
+   fixed-link {
+   speed = <100>;
+   full-duplex;
+   };
+
+   mdio1: mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   status = "okay";
+
+   switch0: switch0@0 {
+   compatible = "marvell,mv88e6190";
+   pinctrl-0 = <_gpio_switch0>;
+   pinctrl-names = "default";
+   #address-cells = <1>;
+   #size-cells = <0>;
+   reg = <0>;
+   eeprom-length = <65536>;
+   reset-gpios = < 11 GPIO_ACTIVE_LOW>;
+   interrupt-parent = <>;
+   interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
+   interrupt-controller;
+   #interrupt-cells = <2>;
+
+   ports {
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   port@0 {
+  

[PATCH] ARM: dts: vf610: Add ZII SSMB SPU3 board

2018-07-16 Thread Andrey Smirnov
Add support for Zodiac Inflight Innovations SSMB SPU3
board (VF610-based).

Cc: Shawn Guo 
Cc: Fabio Estevam 
Cc: cphe...@gmail.com
Cc: linux-arm-ker...@lists.infradead.org
Cc: devicet...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Andrew Lunn 
Signed-off-by: Andrey Smirnov 
---
 arch/arm/boot/dts/Makefile|   3 +-
 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts | 343 ++
 2 files changed, 345 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index bea41b129493..e331b2c16539 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -570,7 +570,8 @@ dtb-$(CONFIG_SOC_VF610) += \
vf610m4-cosmic.dtb \
vf610-twr.dtb \
vf610-zii-dev-rev-b.dtb \
-   vf610-zii-dev-rev-c.dtb
+   vf610-zii-dev-rev-c.dtb \
+   vf610-zii-ssmb-spu3.dtb
 dtb-$(CONFIG_ARCH_MXS) += \
imx23-evk.dtb \
imx23-olinuxino.dtb \
diff --git a/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts 
b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
new file mode 100644
index ..b692117d7839
--- /dev/null
+++ b/arch/arm/boot/dts/vf610-zii-ssmb-spu3.dts
@@ -0,0 +1,343 @@
+// SPDX-License-Identifier: (GPL-2.0 OR MIT)
+
+/*
+ * Device tree file for ZII's SSMB SPU3 board
+ *
+ * SSMB - SPU3 Switch Management Board
+ * SPU - Seat Power Unit
+ *
+ * Copyright (C) 2015, 2016 Zodiac Inflight Innovations
+ *
+ * Based on an original 'vf610-twr.dts' which is Copyright 2015,
+ * Freescale Semiconductor, Inc.
+ */
+
+/dts-v1/;
+#include "vf610.dtsi"
+
+/ {
+   model = "ZII VF610 SSMB SPU3 Board";
+   compatible = "zii,vf610spu3", "zii,vf610dev", "fsl,vf610";
+
+   chosen {
+   stdout-path = 
+   };
+
+   memory {
+   reg = <0x8000 0x2000>;
+   };
+
+   gpio-leds {
+   compatible = "gpio-leds";
+   pinctrl-0 = <_leds_debug>;
+   pinctrl-names = "default";
+
+   led-debug {
+   label = "zii:green:debug1";
+   gpios = < 18 GPIO_ACTIVE_HIGH>;
+   linux,default-trigger = "heartbeat";
+   max-brightness = <1>;
+   };
+   };
+
+   reg_vcc_3v3_mcu: regulator {
+   compatible = "regulator-fixed";
+   regulator-name = "vcc_3v3_mcu";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   };
+};
+
+ {
+   vref-supply = <_vcc_3v3_mcu>;
+   status = "okay";
+};
+
+ {
+   vref-supply = <_vcc_3v3_mcu>;
+   status = "okay";
+};
+
+ {
+   bus-num = <1>;
+   pinctrl-names = "default";
+   pinctrl-0 = <_dspi1>;
+   /*
+* Some SPU3s come with SPI-NOR chip DNPed, so we leave this
+* node disabled by default and rely on bootloader to enable
+* it when appropriate.
+*/
+   status = "disabled";
+
+   m25p128@0 {
+   #address-cells = <1>;
+   #size-cells = <1>;
+   compatible = "m25p128", "jedec,spi-nor";
+   reg = <0>;
+   spi-max-frequency = <5000>;
+
+   partition@0 {
+   label = "m25p128-0";
+   reg = <0x0 0x0100>;
+   };
+   };
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   status = "okay";
+};
+
+ {
+   pinctrl-names = "default";
+   pinctrl-0 = <_esdhc0>;
+   bus-width = <8>;
+   non-removable;
+   no-1-8-v;
+   keep-power-in-suspend;
+   status = "okay";
+};
+
+ {
+   pinctrl-names = "default";
+   pinctrl-0 = <_esdhc1>;
+   bus-width = <4>;
+   status = "okay";
+};
+
+ {
+   phy-mode = "rmii";
+   pinctrl-names = "default";
+   pinctrl-0 = <_fec1>;
+   status = "okay";
+
+   fixed-link {
+   speed = <100>;
+   full-duplex;
+   };
+
+   mdio1: mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   status = "okay";
+
+   switch0: switch0@0 {
+   compatible = "marvell,mv88e6190";
+   pinctrl-0 = <_gpio_switch0>;
+   pinctrl-names = "default";
+   #address-cells = <1>;
+   #size-cells = <0>;
+   reg = <0>;
+   eeprom-length = <65536>;
+   reset-gpios = < 11 GPIO_ACTIVE_LOW>;
+   interrupt-parent = <>;
+   interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
+   interrupt-controller;
+   #interrupt-cells = <2>;
+
+   ports {
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   port@0 {
+