From: Dmitry Osipenko <digetx@gmail.com>
To: Thierry Reding <thierry.reding@gmail.com>
Cc: "Jonathan Hunter" <jonathanh@nvidia.com>,
"David Heidelberg" <david@ixit.cz>,
"Svyatoslav Ryhel" <clamor95@gmail.com>,
"Anton Bambura" <jenneron@protonmail.com>,
"Antoni Aloy Torrens" <aaloytorrens@gmail.com>,
"Nikola Milosavljevic" <mnidza@outlook.com>,
"Ion Agorria" <ion@agorria.com>,
"Michał Mirosław" <mirq-linux@rere.qmqm.pl>,
"Ihor Didenko" <tailormoon@rambler.ru>,
"Andreas Westman Dorcsak" <hedmoo@yahoo.com>,
"Maxim Schwalm" <maxim.schwalm@gmail.com>,
"Raffaele Tranquillini" <raffaele.tranquillini@gmail.com>,
"Jasper Korten" <jja2000@gmail.com>,
"Thomas Graichen" <thomas.graichen@gmail.com>,
"Stefan Eichenberger" <stefan.eichenberger@toradex.com>,
devicetree@vger.kernel.org, linux-tegra@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 06/24] ARM: tegra: Add common device-tree base for Tegra30 ASUS Transformers
Date: Fri, 10 Dec 2021 20:02:14 +0300 [thread overview]
Message-ID: <55a397be-a1cd-efaa-28bb-c0291c200295@gmail.com> (raw)
In-Reply-To: <YbN3OektEKoHY3s1@orome>
10.12.2021 18:50, Thierry Reding пишет:
> On Wed, Dec 08, 2021 at 08:35:51PM +0300, Dmitry Osipenko wrote:
>> From: Svyatoslav Ryhel <clamor95@gmail.com>
>>
>> Add common DTSI for Tegra30 ASUS Transformers. It will be used by multiple
>> device-trees of ASUS devices. The common part initially was born out of
>> the ASUS TF300T tablet's device-tree that was created by Michał Mirosław.
>> It was heavily reworked and improved by Svyatoslav Ryhel, Maxim Schwalm,
>> Ion Agorria et al.
>>
>> [digetx@gmail.com: factored out common part into separate patch and wrote commit message]
>> Co-developed-by: Ion Agorria <ion@agorria.com>
>> Signed-off-by: Ion Agorria <ion@agorria.com>
>> Co-developed-by: Maxim Schwalm <maxim.schwalm@gmail.com>
>> Signed-off-by: Maxim Schwalm <maxim.schwalm@gmail.com>
>> Co-developed-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
>> Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
>> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
>> Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
>> ---
>> .../dts/tegra30-asus-transformer-common.dtsi | 1729 +++++++++++++++++
>> 1 file changed, 1729 insertions(+)
>> create mode 100644 arch/arm/boot/dts/tegra30-asus-transformer-common.dtsi
>>
>> diff --git a/arch/arm/boot/dts/tegra30-asus-transformer-common.dtsi b/arch/arm/boot/dts/tegra30-asus-transformer-common.dtsi
>> new file mode 100644
>> index 000000000000..be77212dd8c7
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/tegra30-asus-transformer-common.dtsi
>> @@ -0,0 +1,1729 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +
>> +#include <dt-bindings/input/gpio-keys.h>
>> +#include <dt-bindings/input/input.h>
>> +#include <dt-bindings/thermal/thermal.h>
>> +
>> +#include "tegra30.dtsi"
>> +#include "tegra30-cpu-opp.dtsi"
>> +#include "tegra30-cpu-opp-microvolt.dtsi"
>> +
>> +/ {
>> + chassis-type = "convertible";
>> +
>> + aliases {
>> + mmc0 = &sdmmc4; /* eMMC */
>
> Looks like a tab snuck in there... otherwise this also has some nodes
> sorted in the wrong order.
I was fixing these tabs, but missed that one. Good catch.
Apparently I missed to recheck the order after the most recent changes,
good that you noticed it.
> [...]
>> + pad-keys {
>
> Any specific reason why this is called pad-keys? We call it gpio-keys
> everywhere else.
Not sure about the name. Perhaps Svyatoslav likes the pad-keys name
more. I recall it was named gpio-keys at some point in the past.
Again, will you change it all by yourself or should I make v6?
next prev parent reply other threads:[~2021-12-10 17:02 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-08 17:35 [PATCH v5 00/24] NVIDIA Tegra ARM32 device-tree patches for 5.17 (new devices and more) Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 01/24] dt-bindings: ARM: tegra: Document ASUS Transformers Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 02/24] dt-bindings: ARM: tegra: Document Pegatron Chagall Dmitry Osipenko
2021-12-10 15:19 ` Thierry Reding
2021-12-10 16:20 ` Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 03/24] ARM: tegra: Add labels to tegra30.dtsi Dmitry Osipenko
2021-12-10 15:22 ` Thierry Reding
2021-12-10 16:26 ` Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 04/24] ARM: tegra: Add labels to tegra114.dtsi Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 05/24] ARM: tegra: Add device-tree for ASUS Transformer EeePad TF101 Dmitry Osipenko
2021-12-10 15:46 ` Thierry Reding
2021-12-10 16:42 ` Dmitry Osipenko
2021-12-11 15:28 ` Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 06/24] ARM: tegra: Add common device-tree base for Tegra30 ASUS Transformers Dmitry Osipenko
2021-12-10 15:50 ` Thierry Reding
2021-12-10 17:02 ` Dmitry Osipenko [this message]
2021-12-08 17:35 ` [PATCH v5 07/24] ARM: tegra: Add common device-tree for LVDS display panels of Tegra30 ASUS tablets Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 08/24] ARM: tegra: Add device-tree for ASUS Transformer Prime TF201 Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 09/24] ARM: tegra: Add device-tree for ASUS Transformer Pad TF300T Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 10/24] ARM: tegra: Add device-tree for ASUS Transformer Pad TF300TG Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 11/24] ARM: tegra: Add device-tree for ASUS Transformer Infinity TF700T Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 12/24] ARM: tegra: Add device-tree for ASUS Transformer Pad TF701T Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 13/24] ARM: tegra: Add device-tree for Pegatron Chagall Dmitry Osipenko
2021-12-08 17:35 ` [PATCH v5 14/24] ARM: tegra: Add device-tree for 1080p version of Nyan Big Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 15/24] ARM: tegra: Add usb-role-switch property to USB OTG ports Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 16/24] ARM: tegra: Enable HDMI CEC on Nyan Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 17/24] ARM: tegra: Enable CPU DFLL " Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 18/24] ARM: tegra: Add CPU thermal zones to Nyan device-tree Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 19/24] ARM: tegra: Name clock and regulator nodes according to DT-schema Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 20/24] ARM: tegra: nexus7: Drop clock-frequency from NFC node Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 21/24] ARM: tegra: nexus7: Use common LVDS display device-tree Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 22/24] ARM: tegra: Enable video decoder on Tegra114 Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 23/24] ARM: tegra_defconfig: Enable drivers wanted by Acer Chromebooks and ASUS tablets Dmitry Osipenko
2021-12-08 17:36 ` [PATCH v5 24/24] ARM: config: multi v7: Enable display drivers used by Tegra 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=55a397be-a1cd-efaa-28bb-c0291c200295@gmail.com \
--to=digetx@gmail.com \
--cc=aaloytorrens@gmail.com \
--cc=clamor95@gmail.com \
--cc=david@ixit.cz \
--cc=devicetree@vger.kernel.org \
--cc=hedmoo@yahoo.com \
--cc=ion@agorria.com \
--cc=jenneron@protonmail.com \
--cc=jja2000@gmail.com \
--cc=jonathanh@nvidia.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=maxim.schwalm@gmail.com \
--cc=mirq-linux@rere.qmqm.pl \
--cc=mnidza@outlook.com \
--cc=raffaele.tranquillini@gmail.com \
--cc=stefan.eichenberger@toradex.com \
--cc=tailormoon@rambler.ru \
--cc=thierry.reding@gmail.com \
--cc=thomas.graichen@gmail.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).