[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 at 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>