linux-tegra.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thierry Reding <thierry.reding@gmail.com>
To: Dmitry Osipenko <digetx@gmail.com>
Cc: "Rob Herring" <robh+dt@kernel.org>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Michał Mirosław" <mirq-linux@rere.qmqm.pl>,
	"David Heidelberg" <david@ixit.cz>,
	"Peter Geis" <pgwipeout@gmail.com>,
	"Stephen Warren" <swarren@wwwdotorg.org>,
	"Nicolas Chauvet" <kwizart@gmail.com>,
	"Pedro Ângelo" <pangelo@void.io>,
	"Matt Merhar" <mattmerhar@protonmail.com>,
	"Zack Pearsall" <zpearsall@yahoo.com>,
	linux-tegra@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v10 0/6] Support NVIDIA Tegra-based Acer A500 and Nexus 7 devices
Date: Thu, 16 Jul 2020 14:38:14 +0200	[thread overview]
Message-ID: <20200716123814.GB535268@ulmo> (raw)
In-Reply-To: <20200629025456.28124-1-digetx@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 7292 bytes --]

On Mon, Jun 29, 2020 at 05:54:50AM +0300, Dmitry Osipenko wrote:
> Hello,
> 
> This series introduces upstream kernel support for Acer Iconia Tab A500
> and ASUS Google Nexus 7 tablet devices. Please review and apply, thanks
> in advance.
> 
> Changelog:
> 
> v10:- Rebased on a recent linux-next which got some device-tree changes:
> 
>         - Renamed sdhci@ DT nodes to mmc@
>         - Removed simple-busses from DTs
>         - Checked DTBs using upcoming YAML bindings from Thierry Reding
> 
> v9: - Nexus 7 memory timings are moved out to a separate dtsi file to ease
>       reviewing of the device-trees since the sparse definition of the timings
>       within multiple DT files created some unnecessary confusion.
>       Thanks to Michał Mirosław for the suggestion.
> 
> v8: - The eMMC voltage regulator is now marked as always-ON in all
>       device-trees because turned out that some bootloader versions have
>       assumption about the enabled regulator at a boot time. This fixes
>       machine hanging on a warm reboot, it's also matching the downstream
>       kernel regulator configuration.
> 
>     - The core regulator of the Ti PMIC Nexus 7 variant is now limited
>       to a 1.35V, which is a more realistic limit than 1.5V that was borrowed
>       from the Cardhu device-tree. Now both Maxim and Ti variants have the
>       common limits.
> 
>     - Added acks from Rob Herring.
> 
> v7: - This version brings support for a Nexus 7 variant that uses Ti PMIC
>       instead of the Maxim PMIC. Previously we assumed that variant with
>       the Ti PMIC doesn't exist in a wild, but turned out that it was a
>       wrong assumption. In a result the device-trees are separated into
>       more DTSI pieces, which combined together form the final device-tree.
>       Thanks to Zack Pearsall for testing the Ti version!
> 
>     - The camera voltage regulator is now preset to a correct 1.8v in the
>       Nexus 7 device-tree.
> 
> v6: - Corrected PMIC's RTC alias in the DTs. Sometime ago I renamed the
>       PMIC's node to match the modern upstream styling, but forgot to change
>       the RTC alias. Now PMIC RTC alias uses node's handle instead of string.
> 
>     - Removed "panel: " handle from device trees, which became unused after
>       v5 and I just missed to remove it.
> 
> v5: - After spending some more time on adding LVDS encoder bridge support
>       to the Tegra DRM driver, I'm now having a second thought and agree
>       with Thierry Reding that in a longer run it should be better not to
>       mix old nvidia,panel with new output graph in device-tree. So the
>       nvidia,panel phandle is removed now in both A500 and Nexus 7 DTs.
> 
> v4: - Corrected CPU's thermal zone on both A500 and Nexus 7, which
>       should use the remote thermal sensor (Tegra chip) instead of the
>       local ("skin" temperature).
> 
>     - Added default PMIC pinmux state on Nexus 7, for completeness.
> 
>     - Added PMIC cpu_pwr_req GPIO hog on Nexus 7, for completeness.
> 
>     - Renamed thermal sensor node label to NCT72 on Nexus 7 to match the
>       actual hardware (NCT72 is compatible with NCT1008). For completeness.
> 
>     - Added always-on USB AVDD regulator on Nexus 7, which is controlled by
>       the PMIC GPIO. For completeness.
> 
>     - Added PMIC GPIO enable-control to the 3v3_sys regulator on Nexus 7,
>       for completeness.
> 
>     - Added CONFIG_NAMESPACES, CONFIG_USER_NS, CONFIG_SECCOMP and
>       CONFIG_PINCTRL_MAX77620 to the tegra_defconfig. The first three
>       options are must-have for the modern userspace, the last is somewhat
>       actual now since the default PMIC pinmux state is added to the N7 DT.
> 
> v3: - Improved device-tree node-names in accordance to review comments that
>       were made by Thierry Reding to v2.
> 
>     - Corrected LVDS encoder powerdown-GPIO polarity on both A500 and Nexus 7
>       because I implemented the DRM bridges support in the Tegra DRM driver
>       and found that there was a mistake in the device-trees. The updated
>       polarity also matches the boards schematics. Now DRM bridges support
>       becomes mandatory since LVDS is getting disabled during of the encoder
>       driver probing. I'll send the DRM patch separately from this series.
> 
>     - Replaced recently deprecated CONFIG_DRM_LVDS_ENCODER with the new
>       CONFIG_DRM_LVDS_CODEC in tegra_defconfig.
> 
>     - Added more config options to tegra_defconfig, like pstore and thermal.
> 
>     - Added atmel,cfg_name to the A500 DT, which is a new upcoming property
>       in 5.8+ that allows to specify the per-board hardware config file name.
> 
> v2: - Corrected "volume down" key-label in the grouper's device-tree and
>       improved some other names in device-trees.
> 
>     - Added optional (upcoming in 5.8+) VDD/AVDD regulators to the touchscreen
>       node in A500 device-tree.
> 
> Dmitry Osipenko (6):
>   ARM: tegra: Add device-tree for Acer Iconia Tab A500
>   ARM: tegra: Add device-tree for ASUS Google Nexus 7
>   dt-bindings: Add vendor prefix for Acer Inc.
>   dt-bindings: ARM: tegra: Add Acer Iconia Tab A500
>   dt-bindings: ARM: tegra: Add ASUS Google Nexus 7
>   ARM: tegra_defconfig: Enable options useful for Nexus 7 and Acer A500
> 
>  .../devicetree/bindings/arm/tegra.yaml        |   10 +
>  .../devicetree/bindings/vendor-prefixes.yaml  |    2 +
>  arch/arm/boot/dts/Makefile                    |    4 +
>  .../boot/dts/tegra20-acer-a500-picasso.dts    | 1438 +++++++++++++++
>  .../dts/tegra30-asus-nexus7-grouper-E1565.dts |    9 +
>  .../dts/tegra30-asus-nexus7-grouper-PM269.dts |    9 +
>  .../tegra30-asus-nexus7-grouper-common.dtsi   | 1232 +++++++++++++
>  ...egra30-asus-nexus7-grouper-maxim-pmic.dtsi |  185 ++
>  ...30-asus-nexus7-grouper-memory-timings.dtsi | 1565 +++++++++++++++++
>  .../tegra30-asus-nexus7-grouper-ti-pmic.dtsi  |  149 ++
>  .../boot/dts/tegra30-asus-nexus7-grouper.dtsi |  149 ++
>  .../dts/tegra30-asus-nexus7-tilapia-E1565.dts |    9 +
>  ...30-asus-nexus7-tilapia-memory-timings.dtsi |  325 ++++
>  .../boot/dts/tegra30-asus-nexus7-tilapia.dtsi |  235 +++
>  arch/arm/configs/tegra_defconfig              |   42 +
>  15 files changed, 5363 insertions(+)
>  create mode 100644 arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-grouper-E1565.dts
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-grouper-PM269.dts
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-grouper-memory-timings.dtsi
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-grouper.dtsi
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-tilapia-E1565.dts
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-tilapia-memory-timings.dtsi
>  create mode 100644 arch/arm/boot/dts/tegra30-asus-nexus7-tilapia.dtsi

Applied, thanks.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  parent reply	other threads:[~2020-07-16 12:38 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-29  2:54 [PATCH v10 0/6] Support NVIDIA Tegra-based Acer A500 and Nexus 7 devices Dmitry Osipenko
     [not found] ` <20200629025456.28124-1-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-06-29  2:54   ` [PATCH v10 1/6] ARM: tegra: Add device-tree for Acer Iconia Tab A500 Dmitry Osipenko
2020-06-29  2:54   ` [PATCH v10 2/6] ARM: tegra: Add device-tree for ASUS Google Nexus 7 Dmitry Osipenko
2020-06-29  2:54   ` [PATCH v10 4/6] dt-bindings: ARM: tegra: Add Acer Iconia Tab A500 Dmitry Osipenko
2020-06-29  2:54 ` [PATCH v10 3/6] dt-bindings: Add vendor prefix for Acer Inc Dmitry Osipenko
2020-06-29  2:54 ` [PATCH v10 5/6] dt-bindings: ARM: tegra: Add ASUS Google Nexus 7 Dmitry Osipenko
2020-06-29  2:54 ` [PATCH v10 6/6] ARM: tegra_defconfig: Enable options useful for Nexus 7 and Acer A500 Dmitry Osipenko
2020-07-16 12:38 ` Thierry Reding [this message]
2020-07-16 20:43   ` [PATCH v10 0/6] Support NVIDIA Tegra-based Acer A500 and Nexus 7 devices Dmitry Osipenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200716123814.GB535268@ulmo \
    --to=thierry.reding@gmail.com \
    --cc=david@ixit.cz \
    --cc=devicetree@vger.kernel.org \
    --cc=digetx@gmail.com \
    --cc=jonathanh@nvidia.com \
    --cc=kwizart@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=mattmerhar@protonmail.com \
    --cc=mirq-linux@rere.qmqm.pl \
    --cc=pangelo@void.io \
    --cc=pgwipeout@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=swarren@wwwdotorg.org \
    --cc=zpearsall@yahoo.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).