[GIT PULL] Allwinner DT changes for 4.10

[GIT PULL] Allwinner DT changes for 4.10

[Maxime Ripard]

Hi Arnd, Olof,

Here is our pull request for the next merge window.

Thanks!
Maxime

The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:

  Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)

are available in the git repository at:

  https://git.kernel.org/pub/scm/linux/kernel/git/mripard/linux.git tags/sunxi-dt-for-4.10

for you to fetch changes up to e39a30cf736144814b0bddb3fff28fbbc2a8be0f:

  ARM: sunxi: Add the missing clocks to the pinctrl nodes (2016-11-15 18:49:47 +0100)

----------------------------------------------------------------
Allwinner DT additions for 4.10

The usual bunch of DT additions, but most notably:
  - A31 DRM driver
  - A31 audio codec
  - WiFi for the A80-Based boards and the CHIP
  - Support for the NextThing Co CHIP Pro (the first board with NAND
    enabled)
  - New boards: NanoPi M1,

----------------------------------------------------------------
Antoine Tenart (1):
      ARM: sun5i: chip: add a node for the w1 gpio controller

Chen-Yu Tsai (12):
      ARM: dts: sun6i: Sort pinmux setting nodes
      ARM: dts: sun6i: Add device nodes for first display pipeline
      ARM: dts: sun6i: Add A31 LCD0 RGB888 pins
      ARM: dts: sun9i: Add mmc1 pinmux setting
      ARM: dts: sun9i: a80-optimus: Enable AP6330 WiFi
      ARM: dts: sun9i: cubieboard4: Enable AP6330 WiFi
      ARM: dts: sun6i: Add audio codec device node
      ARM: dts: sun6i: hummingbird: Enable internal audio codec
      ARM: dts: sun6i: sina31s: Enable internal audio codec
      ARM: dts: sun7i: bananapi-m1-plus: Enable USB PHY for USB host support
      ARM: dts: sun7i: bananapi-m1-plus: Add PMIC regulators
      ARM: dts: sun7i: bananapi-m1-plus: Enable USB OTG

Emmanuel Vadot (1):
      ARM: dts: sunxi: Add cpu-supply for Olimex A20 EVB

Hans de Goede (1):
      ARM: dts: sun8i: reference-design-tablet: ldo_io1 is vcc-touchscreen

Maxime Ripard (16):
      ARM: sun5i: a13-olinuxino: Enable VGA bridge
      ARM: gr8: Add the UART3
      ARM: gr8: Fix typo in the i2s mclk pin group
      ARM: gr8: Add missing pwm channel 1 pin
      ARM: gr8: Add UART2 pins
      ARM: gr8: Add UART3 pins
      ARM: gr8: Add CHIP Pro support
      ARM: sun5i: chip: Enable Wi-Fi SDIO chip
      ARM: sun5i: Rename A10s pins
      ARM: sun5i: Add SPI2 pins
      ARM: sun5i: Add RGB 565 LCD pins
      ARM: sun5i: chip: Add optional buses
      ARM: gr8: evb: Enable SPDIF
      ARM: gr8: evb: Add i2s codec
      ARM: sun8i: sina33: Enable USB gadget
      ARM: sunxi: Add the missing clocks to the pinctrl nodes

Milo Kim (5):
      ARM: dts: sun8i: Add common dtsi file for NanoPi SBCs
      ARM: dts: sun8i: Use the common file in NanoPi NEO SBC
      ARM: dts: sun8i: Add dts file for NanoPi M1 SBC
      ARM: dts: sun8i: Add SPI pinctrl node in H3
      ARM: dts: sun8i: Add SPI controller node in H3

Sudeep Holla (1):
      ARM: dts: sun8i: replace enable-sdio-wakeup with wakeup-source for BananaPi M1+

 arch/arm/boot/dts/Makefile                         |   1 +
 arch/arm/boot/dts/ntc-gr8-chip-pro.dts             | 266 +++++++++++++++++++++
 arch/arm/boot/dts/ntc-gr8-evb.dts                  |  33 +++
 arch/arm/boot/dts/ntc-gr8.dtsi                     |  47 +++-
 arch/arm/boot/dts/sun4i-a10.dtsi                   |   3 +-
 arch/arm/boot/dts/sun5i-a10s-olinuxino-micro.dts   |   4 +-
 arch/arm/boot/dts/sun5i-a10s.dtsi                  |   4 +-
 arch/arm/boot/dts/sun5i-a13-olinuxino.dts          |  54 +++++
 arch/arm/boot/dts/sun5i-r8-chip.dts                |  69 ++++++
 arch/arm/boot/dts/sun5i.dtsi                       |  27 ++-
 arch/arm/boot/dts/sun6i-a31-hummingbird.dts        |  13 +
 arch/arm/boot/dts/sun6i-a31.dtsi                   | 266 +++++++++++++++++----
 arch/arm/boot/dts/sun6i-a31s-sina31s.dts           |   8 +
 arch/arm/boot/dts/sun6i-a31s.dtsi                  |   8 +
 arch/arm/boot/dts/sun7i-a20-bananapi-m1-plus.dts   |  62 ++++-
 arch/arm/boot/dts/sun7i-a20-olimex-som-evb.dts     |   4 +
 arch/arm/boot/dts/sun7i-a20.dtsi                   |   3 +-
 arch/arm/boot/dts/sun8i-a23-a33.dtsi               |   6 +-
 .../boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts   |   7 -
 arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts     |   5 +
 arch/arm/boot/dts/sun8i-h3-nanopi-m1.dts           |  64 +++++
 arch/arm/boot/dts/sun8i-h3-nanopi-neo.dts          |  79 +-----
 arch/arm/boot/dts/sun8i-h3-nanopi.dtsi             | 144 +++++++++++
 arch/arm/boot/dts/sun8i-h3.dtsi                    |  52 +++-
 .../boot/dts/sun8i-reference-design-tablet.dtsi    |   7 +
 arch/arm/boot/dts/sun9i-a80-cubieboard4.dts        |  32 +++
 arch/arm/boot/dts/sun9i-a80-optimus.dts            |  30 +++
 arch/arm/boot/dts/sun9i-a80.dtsi                   |  14 +-
 28 files changed, 1165 insertions(+), 147 deletions(-)
 create mode 100644 arch/arm/boot/dts/ntc-gr8-chip-pro.dts
 create mode 100644 arch/arm/boot/dts/sun8i-h3-nanopi-m1.dts
 create mode 100644 arch/arm/boot/dts/sun8i-h3-nanopi.dtsi

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161115/c1f6ab49/attachment.sig>

[GIT PULL] Allwinner DT changes for 4.10

[Olof Johansson]

Hi,

On Tue, Nov 15, 2016 at 09:41:22PM +0100, Maxime Ripard wrote:
> Hi Arnd, Olof,
> 
> Here is our pull request for the next merge window.
> 
> Thanks!
> Maxime
> 
> The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:
> 
>   Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)
> 
> are available in the git repository at:
> 
>   https://git.kernel.org/pub/scm/linux/kernel/git/mripard/linux.git tags/sunxi-dt-for-4.10
> 
> for you to fetch changes up to e39a30cf736144814b0bddb3fff28fbbc2a8be0f:
> 
>   ARM: sunxi: Add the missing clocks to the pinctrl nodes (2016-11-15 18:49:47 +0100)
> 
> ----------------------------------------------------------------
> Allwinner DT additions for 4.10
> 
> The usual bunch of DT additions, but most notably:
>   - A31 DRM driver
>   - A31 audio codec
>   - WiFi for the A80-Based boards and the CHIP
>   - Support for the NextThing Co CHIP Pro (the first board with NAND
>     enabled)
>   - New boards: NanoPi M1,
> 
[...]

> Maxime Ripard (16):
>       ARM: sun5i: a13-olinuxino: Enable VGA bridge
>       ARM: gr8: Add the UART3
>       ARM: gr8: Fix typo in the i2s mclk pin group
>       ARM: gr8: Add missing pwm channel 1 pin
>       ARM: gr8: Add UART2 pins
>       ARM: gr8: Add UART3 pins
>       ARM: gr8: Add CHIP Pro support
>       ARM: sun5i: chip: Enable Wi-Fi SDIO chip
>       ARM: sun5i: Rename A10s pins
>       ARM: sun5i: Add SPI2 pins
>       ARM: sun5i: Add RGB 565 LCD pins
>       ARM: sun5i: chip: Add optional buses
>       ARM: gr8: evb: Enable SPDIF
>       ARM: gr8: evb: Add i2s codec
>       ARM: sun8i: sina33: Enable USB gadget
>       ARM: sunxi: Add the missing clocks to the pinctrl nodes
> 
[...]

>  arch/arm/boot/dts/Makefile                         |   1 +
>  arch/arm/boot/dts/ntc-gr8-chip-pro.dts             | 266 +++++++++++++++++++++
>  arch/arm/boot/dts/ntc-gr8-evb.dts                  |  33 +++
>  arch/arm/boot/dts/ntc-gr8.dtsi                     |  47 +++-
>  arch/arm/boot/dts/sun4i-a10.dtsi                   |   3 +-

NTC isn't the SoC manufacturer, and we try to keep the prefixes down to
manufacturer to keep the namespace a little more manageable, even if
we never got subdirectories setup as on arm64.

I think this should probably be sun4i-a10-gr8 or sun4i-r8-gr8 as prefix?


-Olof

[GIT PULL] Allwinner DT changes for 4.10

[Maxime Ripard]

Hi Olof,

On Fri, Nov 18, 2016 at 04:23:16PM -0800, Olof Johansson wrote:
> Hi,
> 
> On Tue, Nov 15, 2016 at 09:41:22PM +0100, Maxime Ripard wrote:
> > Hi Arnd, Olof,
> > 
> > Here is our pull request for the next merge window.
> > 
> > Thanks!
> > Maxime
> > 
> > The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:
> > 
> >   Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)
> > 
> > are available in the git repository at:
> > 
> >   https://git.kernel.org/pub/scm/linux/kernel/git/mripard/linux.git tags/sunxi-dt-for-4.10
> > 
> > for you to fetch changes up to e39a30cf736144814b0bddb3fff28fbbc2a8be0f:
> > 
> >   ARM: sunxi: Add the missing clocks to the pinctrl nodes (2016-11-15 18:49:47 +0100)
> > 
> > ----------------------------------------------------------------
> > Allwinner DT additions for 4.10
> > 
> > The usual bunch of DT additions, but most notably:
> >   - A31 DRM driver
> >   - A31 audio codec
> >   - WiFi for the A80-Based boards and the CHIP
> >   - Support for the NextThing Co CHIP Pro (the first board with NAND
> >     enabled)
> >   - New boards: NanoPi M1,
> > 
> [...]
> 
> > Maxime Ripard (16):
> >       ARM: sun5i: a13-olinuxino: Enable VGA bridge
> >       ARM: gr8: Add the UART3
> >       ARM: gr8: Fix typo in the i2s mclk pin group
> >       ARM: gr8: Add missing pwm channel 1 pin
> >       ARM: gr8: Add UART2 pins
> >       ARM: gr8: Add UART3 pins
> >       ARM: gr8: Add CHIP Pro support
> >       ARM: sun5i: chip: Enable Wi-Fi SDIO chip
> >       ARM: sun5i: Rename A10s pins
> >       ARM: sun5i: Add SPI2 pins
> >       ARM: sun5i: Add RGB 565 LCD pins
> >       ARM: sun5i: chip: Add optional buses
> >       ARM: gr8: evb: Enable SPDIF
> >       ARM: gr8: evb: Add i2s codec
> >       ARM: sun8i: sina33: Enable USB gadget
> >       ARM: sunxi: Add the missing clocks to the pinctrl nodes
> > 
> [...]
> 
> >  arch/arm/boot/dts/Makefile                         |   1 +
> >  arch/arm/boot/dts/ntc-gr8-chip-pro.dts             | 266 +++++++++++++++++++++
> >  arch/arm/boot/dts/ntc-gr8-evb.dts                  |  33 +++
> >  arch/arm/boot/dts/ntc-gr8.dtsi                     |  47 +++-
> >  arch/arm/boot/dts/sun4i-a10.dtsi                   |   3 +-
> 
> NTC isn't the SoC manufacturer, and we try to keep the prefixes down to
> manufacturer to keep the namespace a little more manageable, even if
> we never got subdirectories setup as on arm64.
> 
> I think this should probably be sun4i-a10-gr8 or sun4i-r8-gr8 as prefix?

The users really expect a SoC from Nextthing, it's always been
marketed that way, the marking on the SoC also says so, etc. The fact
that it's been a design in cooperation with Allwinner, and that the
design is based on some earlier family is an implementation detail,
and I'd really like not for it to have the sun5i prefix, it's just
confusing.

And the ntc prefix has been asked for during the reviews...

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161121/93468ce6/attachment.sig>

[GIT PULL] Allwinner DT changes for 4.10

[Olof Johansson]

HI,

On Mon, Nov 21, 2016 at 5:27 AM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> Hi Olof,
>
> On Fri, Nov 18, 2016 at 04:23:16PM -0800, Olof Johansson wrote:
>> Hi,
>>
>> On Tue, Nov 15, 2016 at 09:41:22PM +0100, Maxime Ripard wrote:
>> > Hi Arnd, Olof,
>> >
>> > Here is our pull request for the next merge window.
>> >
>> > Thanks!
>> > Maxime
>> >
>> > The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:
>> >
>> >   Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)
>> >
>> > are available in the git repository at:
>> >
>> >   https://git.kernel.org/pub/scm/linux/kernel/git/mripard/linux.git tags/sunxi-dt-for-4.10
>> >
>> > for you to fetch changes up to e39a30cf736144814b0bddb3fff28fbbc2a8be0f:
>> >
>> >   ARM: sunxi: Add the missing clocks to the pinctrl nodes (2016-11-15 18:49:47 +0100)
>> >
>> > ----------------------------------------------------------------
>> > Allwinner DT additions for 4.10
>> >
>> > The usual bunch of DT additions, but most notably:
>> >   - A31 DRM driver
>> >   - A31 audio codec
>> >   - WiFi for the A80-Based boards and the CHIP
>> >   - Support for the NextThing Co CHIP Pro (the first board with NAND
>> >     enabled)
>> >   - New boards: NanoPi M1,
>> >
>> [...]
>>
>> > Maxime Ripard (16):
>> >       ARM: sun5i: a13-olinuxino: Enable VGA bridge
>> >       ARM: gr8: Add the UART3
>> >       ARM: gr8: Fix typo in the i2s mclk pin group
>> >       ARM: gr8: Add missing pwm channel 1 pin
>> >       ARM: gr8: Add UART2 pins
>> >       ARM: gr8: Add UART3 pins
>> >       ARM: gr8: Add CHIP Pro support
>> >       ARM: sun5i: chip: Enable Wi-Fi SDIO chip
>> >       ARM: sun5i: Rename A10s pins
>> >       ARM: sun5i: Add SPI2 pins
>> >       ARM: sun5i: Add RGB 565 LCD pins
>> >       ARM: sun5i: chip: Add optional buses
>> >       ARM: gr8: evb: Enable SPDIF
>> >       ARM: gr8: evb: Add i2s codec
>> >       ARM: sun8i: sina33: Enable USB gadget
>> >       ARM: sunxi: Add the missing clocks to the pinctrl nodes
>> >
>> [...]
>>
>> >  arch/arm/boot/dts/Makefile                         |   1 +
>> >  arch/arm/boot/dts/ntc-gr8-chip-pro.dts             | 266 +++++++++++++++++++++
>> >  arch/arm/boot/dts/ntc-gr8-evb.dts                  |  33 +++
>> >  arch/arm/boot/dts/ntc-gr8.dtsi                     |  47 +++-
>> >  arch/arm/boot/dts/sun4i-a10.dtsi                   |   3 +-
>>
>> NTC isn't the SoC manufacturer, and we try to keep the prefixes down to
>> manufacturer to keep the namespace a little more manageable, even if
>> we never got subdirectories setup as on arm64.
>>
>> I think this should probably be sun4i-a10-gr8 or sun4i-r8-gr8 as prefix?
>
> The users really expect a SoC from Nextthing, it's always been
> marketed that way, the marking on the SoC also says so, etc. The fact
> that it's been a design in cooperation with Allwinner, and that the
> design is based on some earlier family is an implementation detail,
> and I'd really like not for it to have the sun5i prefix, it's just
> confusing.

I don't care so much about what's printed on the top of the package, I
care a lot more about what's on the insides. We've got a long
tradition of not renaming things randomly when companies get acquired
or renames themselves, and I think that same spirit is applicable
here.

Calling it an SoC is inaccurate as well, it's really a
system-in-package. It's just a new way to integrate an SoC into a
module to build boards out of. Compare to Octavo OSD335x, for example.

System-in-package solutions are going to get more and more common, and
it's going to become really chaotic if we expect to use the prefix of
the company custom-ordering the package for each and every one of
them.

> And the ntc prefix has been asked for during the reviews...

Having a link to that requeset/email would be helpful if you try to
use it as an argument.


-Olof

[GIT PULL] Allwinner DT changes for 4.10

[Maxime Ripard]

On Mon, Nov 21, 2016 at 08:28:09AM -0800, Olof Johansson wrote:
> HI,
> 
> On Mon, Nov 21, 2016 at 5:27 AM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> > Hi Olof,
> >
> > On Fri, Nov 18, 2016 at 04:23:16PM -0800, Olof Johansson wrote:
> >> Hi,
> >>
> >> On Tue, Nov 15, 2016 at 09:41:22PM +0100, Maxime Ripard wrote:
> >> > Hi Arnd, Olof,
> >> >
> >> > Here is our pull request for the next merge window.
> >> >
> >> > Thanks!
> >> > Maxime
> >> >
> >> > The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:
> >> >
> >> >   Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)
> >> >
> >> > are available in the git repository at:
> >> >
> >> >   https://git.kernel.org/pub/scm/linux/kernel/git/mripard/linux.git tags/sunxi-dt-for-4.10
> >> >
> >> > for you to fetch changes up to e39a30cf736144814b0bddb3fff28fbbc2a8be0f:
> >> >
> >> >   ARM: sunxi: Add the missing clocks to the pinctrl nodes (2016-11-15 18:49:47 +0100)
> >> >
> >> > ----------------------------------------------------------------
> >> > Allwinner DT additions for 4.10
> >> >
> >> > The usual bunch of DT additions, but most notably:
> >> >   - A31 DRM driver
> >> >   - A31 audio codec
> >> >   - WiFi for the A80-Based boards and the CHIP
> >> >   - Support for the NextThing Co CHIP Pro (the first board with NAND
> >> >     enabled)
> >> >   - New boards: NanoPi M1,
> >> >
> >> [...]
> >>
> >> > Maxime Ripard (16):
> >> >       ARM: sun5i: a13-olinuxino: Enable VGA bridge
> >> >       ARM: gr8: Add the UART3
> >> >       ARM: gr8: Fix typo in the i2s mclk pin group
> >> >       ARM: gr8: Add missing pwm channel 1 pin
> >> >       ARM: gr8: Add UART2 pins
> >> >       ARM: gr8: Add UART3 pins
> >> >       ARM: gr8: Add CHIP Pro support
> >> >       ARM: sun5i: chip: Enable Wi-Fi SDIO chip
> >> >       ARM: sun5i: Rename A10s pins
> >> >       ARM: sun5i: Add SPI2 pins
> >> >       ARM: sun5i: Add RGB 565 LCD pins
> >> >       ARM: sun5i: chip: Add optional buses
> >> >       ARM: gr8: evb: Enable SPDIF
> >> >       ARM: gr8: evb: Add i2s codec
> >> >       ARM: sun8i: sina33: Enable USB gadget
> >> >       ARM: sunxi: Add the missing clocks to the pinctrl nodes
> >> >
> >> [...]
> >>
> >> >  arch/arm/boot/dts/Makefile                         |   1 +
> >> >  arch/arm/boot/dts/ntc-gr8-chip-pro.dts             | 266 +++++++++++++++++++++
> >> >  arch/arm/boot/dts/ntc-gr8-evb.dts                  |  33 +++
> >> >  arch/arm/boot/dts/ntc-gr8.dtsi                     |  47 +++-
> >> >  arch/arm/boot/dts/sun4i-a10.dtsi                   |   3 +-
> >>
> >> NTC isn't the SoC manufacturer, and we try to keep the prefixes down to
> >> manufacturer to keep the namespace a little more manageable, even if
> >> we never got subdirectories setup as on arm64.
> >>
> >> I think this should probably be sun4i-a10-gr8 or sun4i-r8-gr8 as prefix?
> >
> > The users really expect a SoC from Nextthing, it's always been
> > marketed that way, the marking on the SoC also says so, etc. The fact
> > that it's been a design in cooperation with Allwinner, and that the
> > design is based on some earlier family is an implementation detail,
> > and I'd really like not for it to have the sun5i prefix, it's just
> > confusing.
> 
> I don't care so much about what's printed on the top of the package, I
> care a lot more about what's on the insides. We've got a long
> tradition of not renaming things randomly when companies get acquired
> or renames themselves, and I think that same spirit is applicable
> here.

Yet, we called the imx23 and imx28 that way while it was really a
sigmatel design. And I'm pretty sure there's other examples too.

> Calling it an SoC is inaccurate as well, it's really a
> system-in-package. It's just a new way to integrate an SoC into a
> module to build boards out of. Compare to Octavo OSD335x, for example.
> 
> System-in-package solutions are going to get more and more common, and
> it's going to become really chaotic if we expect to use the prefix of
> the company custom-ordering the package for each and every one of
> them.

This is indeed a SIP, but one with a custom SoC (or whatever the !RAM
part in a SIP is called) that was designed by Allwinner *and* Next
Thing Co., unlike the OSD335x that has a standard AM335x SoC in it.

> > And the ntc prefix has been asked for during the reviews...
>
> Having a link to that requeset/email would be helpful if you try to
> use it as an argument.

Yeah, it turns out it was off list... so it's a pretty weak one :)

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161121/a2e39d2c/attachment.sig>

[GIT PULL] Allwinner DT changes for 4.10

[Olof Johansson]

On Mon, Nov 21, 2016 at 2:22 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> On Mon, Nov 21, 2016 at 08:28:09AM -0800, Olof Johansson wrote:
>> HI,
>>
>> On Mon, Nov 21, 2016 at 5:27 AM, Maxime Ripard
>> <maxime.ripard@free-electrons.com> wrote:
>> > Hi Olof,
>> >
>> > On Fri, Nov 18, 2016 at 04:23:16PM -0800, Olof Johansson wrote:
>> >> Hi,
>> >>
>> >> On Tue, Nov 15, 2016 at 09:41:22PM +0100, Maxime Ripard wrote:
>> >> > Hi Arnd, Olof,
>> >> >
>> >> > Here is our pull request for the next merge window.
>> >> >
>> >> > Thanks!
>> >> > Maxime
>> >> >
>> >> > The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:
>> >> >
>> >> >   Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)
>> >> >
>> >> > are available in the git repository at:
>> >> >
>> >> >   https://git.kernel.org/pub/scm/linux/kernel/git/mripard/linux.git tags/sunxi-dt-for-4.10
>> >> >
>> >> > for you to fetch changes up to e39a30cf736144814b0bddb3fff28fbbc2a8be0f:
>> >> >
>> >> >   ARM: sunxi: Add the missing clocks to the pinctrl nodes (2016-11-15 18:49:47 +0100)
>> >> >
>> >> > ----------------------------------------------------------------
>> >> > Allwinner DT additions for 4.10
>> >> >
>> >> > The usual bunch of DT additions, but most notably:
>> >> >   - A31 DRM driver
>> >> >   - A31 audio codec
>> >> >   - WiFi for the A80-Based boards and the CHIP
>> >> >   - Support for the NextThing Co CHIP Pro (the first board with NAND
>> >> >     enabled)
>> >> >   - New boards: NanoPi M1,
>> >> >
>> >> [...]
>> >>
>> >> > Maxime Ripard (16):
>> >> >       ARM: sun5i: a13-olinuxino: Enable VGA bridge
>> >> >       ARM: gr8: Add the UART3
>> >> >       ARM: gr8: Fix typo in the i2s mclk pin group
>> >> >       ARM: gr8: Add missing pwm channel 1 pin
>> >> >       ARM: gr8: Add UART2 pins
>> >> >       ARM: gr8: Add UART3 pins
>> >> >       ARM: gr8: Add CHIP Pro support
>> >> >       ARM: sun5i: chip: Enable Wi-Fi SDIO chip
>> >> >       ARM: sun5i: Rename A10s pins
>> >> >       ARM: sun5i: Add SPI2 pins
>> >> >       ARM: sun5i: Add RGB 565 LCD pins
>> >> >       ARM: sun5i: chip: Add optional buses
>> >> >       ARM: gr8: evb: Enable SPDIF
>> >> >       ARM: gr8: evb: Add i2s codec
>> >> >       ARM: sun8i: sina33: Enable USB gadget
>> >> >       ARM: sunxi: Add the missing clocks to the pinctrl nodes
>> >> >
>> >> [...]
>> >>
>> >> >  arch/arm/boot/dts/Makefile                         |   1 +
>> >> >  arch/arm/boot/dts/ntc-gr8-chip-pro.dts             | 266 +++++++++++++++++++++
>> >> >  arch/arm/boot/dts/ntc-gr8-evb.dts                  |  33 +++
>> >> >  arch/arm/boot/dts/ntc-gr8.dtsi                     |  47 +++-
>> >> >  arch/arm/boot/dts/sun4i-a10.dtsi                   |   3 +-
>> >>
>> >> NTC isn't the SoC manufacturer, and we try to keep the prefixes down to
>> >> manufacturer to keep the namespace a little more manageable, even if
>> >> we never got subdirectories setup as on arm64.
>> >>
>> >> I think this should probably be sun4i-a10-gr8 or sun4i-r8-gr8 as prefix?
>> >
>> > The users really expect a SoC from Nextthing, it's always been
>> > marketed that way, the marking on the SoC also says so, etc. The fact
>> > that it's been a design in cooperation with Allwinner, and that the
>> > design is based on some earlier family is an implementation detail,
>> > and I'd really like not for it to have the sun5i prefix, it's just
>> > confusing.
>>
>> I don't care so much about what's printed on the top of the package, I
>> care a lot more about what's on the insides. We've got a long
>> tradition of not renaming things randomly when companies get acquired
>> or renames themselves, and I think that same spirit is applicable
>> here.
>
> Yet, we called the imx23 and imx28 that way while it was really a
> sigmatel design. And I'm pretty sure there's other examples too.

Had Sigmatel had a whole bunch of SoCs in-tree we might have had
discussions about that as well.

>> Calling it an SoC is inaccurate as well, it's really a
>> system-in-package. It's just a new way to integrate an SoC into a
>> module to build boards out of. Compare to Octavo OSD335x, for example.
>>
>> System-in-package solutions are going to get more and more common, and
>> it's going to become really chaotic if we expect to use the prefix of
>> the company custom-ordering the package for each and every one of
>> them.
>
> This is indeed a SIP, but one with a custom SoC (or whatever the !RAM
> part in a SIP is called) that was designed by Allwinner *and* Next
> Thing Co., unlike the OSD335x that has a standard AM335x SoC in it.

Even NTC's marketing material says it's an Allwinner R8, at
https://getchip.com/pages/chippro: "Powered by a chip you can actually
buy. R8 SoC + 256MB DDR3. 1 package. 1 price. $6 in any quantity."

This is really quite similar to the situation with SoM manufacturers
such as Toradex, who integrate SoC, memory and some I/O on a module
that you then build your product around. The only difference is the
level it's integrated at. We've chosen to slice our naming across SoC
family lines, not who makes the substrate that the SoC is mounted to.

>> > And the ntc prefix has been asked for during the reviews...
>>
>> Having a link to that requeset/email would be helpful if you try to
>> use it as an argument.
>
> Yeah, it turns out it was off list... so it's a pretty weak one :)

:)


-Olof

[GIT PULL] Allwinner DT changes for 4.10

[Maxime Ripard]

On Mon, Nov 21, 2016 at 03:34:50PM -0800, Olof Johansson wrote:
> On Mon, Nov 21, 2016 at 2:22 PM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> > On Mon, Nov 21, 2016 at 08:28:09AM -0800, Olof Johansson wrote:
> >> HI,
> >>
> >> On Mon, Nov 21, 2016 at 5:27 AM, Maxime Ripard
> >> <maxime.ripard@free-electrons.com> wrote:
> >> > Hi Olof,
> >> >
> >> > On Fri, Nov 18, 2016 at 04:23:16PM -0800, Olof Johansson wrote:
> >> >> Hi,
> >> >>
> >> >> On Tue, Nov 15, 2016 at 09:41:22PM +0100, Maxime Ripard wrote:
> >> >> > Hi Arnd, Olof,
> >> >> >
> >> >> > Here is our pull request for the next merge window.
> >> >> >
> >> >> > Thanks!
> >> >> > Maxime
> >> >> >
> >> >> > The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:
> >> >> >
> >> >> >   Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)
> >> >> >
> >> >> > are available in the git repository at:
> >> >> >
> >> >> >   https://git.kernel.org/pub/scm/linux/kernel/git/mripard/linux.git tags/sunxi-dt-for-4.10
> >> >> >
> >> >> > for you to fetch changes up to e39a30cf736144814b0bddb3fff28fbbc2a8be0f:
> >> >> >
> >> >> >   ARM: sunxi: Add the missing clocks to the pinctrl nodes (2016-11-15 18:49:47 +0100)
> >> >> >
> >> >> > ----------------------------------------------------------------
> >> >> > Allwinner DT additions for 4.10
> >> >> >
> >> >> > The usual bunch of DT additions, but most notably:
> >> >> >   - A31 DRM driver
> >> >> >   - A31 audio codec
> >> >> >   - WiFi for the A80-Based boards and the CHIP
> >> >> >   - Support for the NextThing Co CHIP Pro (the first board with NAND
> >> >> >     enabled)
> >> >> >   - New boards: NanoPi M1,
> >> >> >
> >> >> [...]
> >> >>
> >> >> > Maxime Ripard (16):
> >> >> >       ARM: sun5i: a13-olinuxino: Enable VGA bridge
> >> >> >       ARM: gr8: Add the UART3
> >> >> >       ARM: gr8: Fix typo in the i2s mclk pin group
> >> >> >       ARM: gr8: Add missing pwm channel 1 pin
> >> >> >       ARM: gr8: Add UART2 pins
> >> >> >       ARM: gr8: Add UART3 pins
> >> >> >       ARM: gr8: Add CHIP Pro support
> >> >> >       ARM: sun5i: chip: Enable Wi-Fi SDIO chip
> >> >> >       ARM: sun5i: Rename A10s pins
> >> >> >       ARM: sun5i: Add SPI2 pins
> >> >> >       ARM: sun5i: Add RGB 565 LCD pins
> >> >> >       ARM: sun5i: chip: Add optional buses
> >> >> >       ARM: gr8: evb: Enable SPDIF
> >> >> >       ARM: gr8: evb: Add i2s codec
> >> >> >       ARM: sun8i: sina33: Enable USB gadget
> >> >> >       ARM: sunxi: Add the missing clocks to the pinctrl nodes
> >> >> >
> >> >> [...]
> >> >>
> >> >> >  arch/arm/boot/dts/Makefile                         |   1 +
> >> >> >  arch/arm/boot/dts/ntc-gr8-chip-pro.dts             | 266 +++++++++++++++++++++
> >> >> >  arch/arm/boot/dts/ntc-gr8-evb.dts                  |  33 +++
> >> >> >  arch/arm/boot/dts/ntc-gr8.dtsi                     |  47 +++-
> >> >> >  arch/arm/boot/dts/sun4i-a10.dtsi                   |   3 +-
> >> >>
> >> >> NTC isn't the SoC manufacturer, and we try to keep the prefixes down to
> >> >> manufacturer to keep the namespace a little more manageable, even if
> >> >> we never got subdirectories setup as on arm64.
> >> >>
> >> >> I think this should probably be sun4i-a10-gr8 or sun4i-r8-gr8 as prefix?
> >> >
> >> > The users really expect a SoC from Nextthing, it's always been
> >> > marketed that way, the marking on the SoC also says so, etc. The fact
> >> > that it's been a design in cooperation with Allwinner, and that the
> >> > design is based on some earlier family is an implementation detail,
> >> > and I'd really like not for it to have the sun5i prefix, it's just
> >> > confusing.
> >>
> >> I don't care so much about what's printed on the top of the package, I
> >> care a lot more about what's on the insides. We've got a long
> >> tradition of not renaming things randomly when companies get acquired
> >> or renames themselves, and I think that same spirit is applicable
> >> here.
> >
> > Yet, we called the imx23 and imx28 that way while it was really a
> > sigmatel design. And I'm pretty sure there's other examples too.
> 
> Had Sigmatel had a whole bunch of SoCs in-tree we might have had
> discussions about that as well.

So what happens if and when we have more NextThing SIP in tree, we
rename all of them once we have "a whole bunch"?

This is a circular argument.

> >> Calling it an SoC is inaccurate as well, it's really a
> >> system-in-package. It's just a new way to integrate an SoC into a
> >> module to build boards out of. Compare to Octavo OSD335x, for example.
> >>
> >> System-in-package solutions are going to get more and more common, and
> >> it's going to become really chaotic if we expect to use the prefix of
> >> the company custom-ordering the package for each and every one of
> >> them.
> >
> > This is indeed a SIP, but one with a custom SoC (or whatever the !RAM
> > part in a SIP is called) that was designed by Allwinner *and* Next
> > Thing Co., unlike the OSD335x that has a standard AM335x SoC in it.
> 
> Even NTC's marketing material says it's an Allwinner R8, at
> https://getchip.com/pages/chippro: "Powered by a chip you can actually
> buy. R8 SoC + 256MB DDR3. 1 package. 1 price. $6 in any quantity."

I've been working on it for the last couple of monthes, and I can tell
you it isn't an R8. It has more IPs, including some that were never
found in any SoC of that generation before (the others being just a
different mix of what was already there).

> This is really quite similar to the situation with SoM manufacturers
> such as Toradex, who integrate SoC, memory and some I/O on a module
> that you then build your product around. The only difference is the
> level it's integrated at. We've chosen to slice our naming across SoC
> family lines, not who makes the substrate that the SoC is mounted to.

I disagree with the comparison. Toradex bought plain off the shelves
SoCs from Freescale, NextThing didn't. It makes sense for the Toradex
case, but I'm still convinced it isn't in this case.

Anyway... I'll send a patch to fix the DTs we have already in 4.9, and
will resend that pull request.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161122/a637fee0/attachment-0001.sig>