* [GIT PULL] Allwinner DT changes for 4.10 @ 2016-11-15 20:41 Maxime Ripard 2016-11-19 0:23 ` Olof Johansson 0 siblings, 1 reply; 7+ messages in thread From: Maxime Ripard @ 2016-11-15 20:41 UTC (permalink / raw) To: linux-arm-kernel 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> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] Allwinner DT changes for 4.10 2016-11-15 20:41 [GIT PULL] Allwinner DT changes for 4.10 Maxime Ripard @ 2016-11-19 0:23 ` Olof Johansson 2016-11-21 13:27 ` Maxime Ripard 0 siblings, 1 reply; 7+ messages in thread From: Olof Johansson @ 2016-11-19 0:23 UTC (permalink / raw) To: linux-arm-kernel 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] Allwinner DT changes for 4.10 2016-11-19 0:23 ` Olof Johansson @ 2016-11-21 13:27 ` Maxime Ripard 2016-11-21 16:28 ` Olof Johansson 0 siblings, 1 reply; 7+ messages in thread From: Maxime Ripard @ 2016-11-21 13:27 UTC (permalink / raw) To: linux-arm-kernel 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> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] Allwinner DT changes for 4.10 2016-11-21 13:27 ` Maxime Ripard @ 2016-11-21 16:28 ` Olof Johansson 2016-11-21 22:22 ` Maxime Ripard 0 siblings, 1 reply; 7+ messages in thread From: Olof Johansson @ 2016-11-21 16:28 UTC (permalink / raw) To: linux-arm-kernel 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] Allwinner DT changes for 4.10 2016-11-21 16:28 ` Olof Johansson @ 2016-11-21 22:22 ` Maxime Ripard 2016-11-21 23:34 ` Olof Johansson 0 siblings, 1 reply; 7+ messages in thread From: Maxime Ripard @ 2016-11-21 22:22 UTC (permalink / raw) To: linux-arm-kernel 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> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] Allwinner DT changes for 4.10 2016-11-21 22:22 ` Maxime Ripard @ 2016-11-21 23:34 ` Olof Johansson 2016-11-22 14:26 ` Maxime Ripard 0 siblings, 1 reply; 7+ messages in thread From: Olof Johansson @ 2016-11-21 23:34 UTC (permalink / raw) To: linux-arm-kernel 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] Allwinner DT changes for 4.10 2016-11-21 23:34 ` Olof Johansson @ 2016-11-22 14:26 ` Maxime Ripard 0 siblings, 0 replies; 7+ messages in thread From: Maxime Ripard @ 2016-11-22 14:26 UTC (permalink / raw) To: linux-arm-kernel 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> ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-11-22 14:26 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-11-15 20:41 [GIT PULL] Allwinner DT changes for 4.10 Maxime Ripard 2016-11-19 0:23 ` Olof Johansson 2016-11-21 13:27 ` Maxime Ripard 2016-11-21 16:28 ` Olof Johansson 2016-11-21 22:22 ` Maxime Ripard 2016-11-21 23:34 ` Olof Johansson 2016-11-22 14:26 ` Maxime Ripard
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).