All of lore.kernel.org
 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: 14+ 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
2020-06-29  2:54 ` Dmitry Osipenko
2020-06-29  2:54 ` [PATCH v10 3/6] dt-bindings: Add vendor prefix for Acer Inc 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     ` 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     ` 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     ` 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
2020-07-16 20:43     ` 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.