From: Vladimir Zapolskiy <vz-ChpfBGZJDbMAvxtiuMwx3w@public.gmane.org> To: Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Jonathan Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>, Greg Kroah-Hartman <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Mauro Carvalho Chehab <mchehab-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> Cc: Dan Carpenter <dan.carpenter-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>, linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Subject: Re: [PATCH v4 1/5] ARM: tegra: Add device tree node to describe IRAM Date: Sat, 11 Nov 2017 16:18:00 +0200 [thread overview] Message-ID: <ffa5f1d0-f156-0bc3-6adb-bb55a0a855c4@mleia.com> (raw) In-Reply-To: <8ce696bc2b4b1808f6c7f7a967a3dacd954d2a4e.1508448293.git.digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Hi Dmitry, On 10/20/2017 12:34 AM, Dmitry Osipenko wrote: > From: Vladimir Zapolskiy <vz-ChpfBGZJDbMAvxtiuMwx3w@public.gmane.org> > > All Tegra SoCs contain 256KiB IRAM, which is used to store CPU resume code > and by hardware engines like a video decoder. > > Signed-off-by: Vladimir Zapolskiy <vz-ChpfBGZJDbMAvxtiuMwx3w@public.gmane.org> Please add also your own closing "Signed-off-by" tag, please reference to "Developer's Certificate of Origin 1.1", point (c), it is found in Documentation/process/submitting-patches.rst > --- > arch/arm/boot/dts/tegra114.dtsi | 8 ++++++++ > arch/arm/boot/dts/tegra124.dtsi | 8 ++++++++ > arch/arm/boot/dts/tegra20.dtsi | 8 ++++++++ > arch/arm/boot/dts/tegra30.dtsi | 8 ++++++++ My assumption is that Thierry would prefer to get 4 separate patches, one for each platform, please split the patch. Also thanks for your time and your efforts applied to push my occasional change, please feel free to take your own authorship for 3 out of 4 patches. > 4 files changed, 32 insertions(+) > > diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi > index 8932ea3afd5f..13f6087790c8 100644 > --- a/arch/arm/boot/dts/tegra114.dtsi > +++ b/arch/arm/boot/dts/tegra114.dtsi > @@ -10,6 +10,14 @@ > compatible = "nvidia,tegra114"; > interrupt-parent = <&lic>; > > + iram@40000000 { > + compatible = "mmio-sram"; Unfortunately Thierry hasn't yet replied, but my assumption is that the list of compatibles should be extended with one more SoC specific value like compatible = "nvidia,tegra114-sysram", "mmio-sram"; I'm not sure, if Tegra maintainers want to see a new compatible described in Documentation/devicetree/bindings. > + reg = <0x40000000 0x40000>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0x40000000 0x40000>; > + }; > + > host1x@50000000 { > compatible = "nvidia,tegra114-host1x", "simple-bus"; > reg = <0x50000000 0x00028000>; > diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi > index 8baf00b89efb..a3585ed82646 100644 > --- a/arch/arm/boot/dts/tegra124.dtsi > +++ b/arch/arm/boot/dts/tegra124.dtsi The considerations from above are applicable to the rest of the touched platforms. -- With best wishes, Vladimir
WARNING: multiple messages have this Message-ID (diff)
From: Vladimir Zapolskiy <vz@mleia.com> To: Dmitry Osipenko <digetx@gmail.com>, Thierry Reding <thierry.reding@gmail.com>, Jonathan Hunter <jonathanh@nvidia.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Rob Herring <robh+dt@kernel.org>, Mauro Carvalho Chehab <mchehab@kernel.org>, Stephen Warren <swarren@wwwdotorg.org> Cc: Dan Carpenter <dan.carpenter@oracle.com>, linux-media@vger.kernel.org, linux-tegra@vger.kernel.org, devel@driverdev.osuosl.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 1/5] ARM: tegra: Add device tree node to describe IRAM Date: Sat, 11 Nov 2017 16:18:00 +0200 [thread overview] Message-ID: <ffa5f1d0-f156-0bc3-6adb-bb55a0a855c4@mleia.com> (raw) In-Reply-To: <8ce696bc2b4b1808f6c7f7a967a3dacd954d2a4e.1508448293.git.digetx@gmail.com> Hi Dmitry, On 10/20/2017 12:34 AM, Dmitry Osipenko wrote: > From: Vladimir Zapolskiy <vz@mleia.com> > > All Tegra SoCs contain 256KiB IRAM, which is used to store CPU resume code > and by hardware engines like a video decoder. > > Signed-off-by: Vladimir Zapolskiy <vz@mleia.com> Please add also your own closing "Signed-off-by" tag, please reference to "Developer's Certificate of Origin 1.1", point (c), it is found in Documentation/process/submitting-patches.rst > --- > arch/arm/boot/dts/tegra114.dtsi | 8 ++++++++ > arch/arm/boot/dts/tegra124.dtsi | 8 ++++++++ > arch/arm/boot/dts/tegra20.dtsi | 8 ++++++++ > arch/arm/boot/dts/tegra30.dtsi | 8 ++++++++ My assumption is that Thierry would prefer to get 4 separate patches, one for each platform, please split the patch. Also thanks for your time and your efforts applied to push my occasional change, please feel free to take your own authorship for 3 out of 4 patches. > 4 files changed, 32 insertions(+) > > diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi > index 8932ea3afd5f..13f6087790c8 100644 > --- a/arch/arm/boot/dts/tegra114.dtsi > +++ b/arch/arm/boot/dts/tegra114.dtsi > @@ -10,6 +10,14 @@ > compatible = "nvidia,tegra114"; > interrupt-parent = <&lic>; > > + iram@40000000 { > + compatible = "mmio-sram"; Unfortunately Thierry hasn't yet replied, but my assumption is that the list of compatibles should be extended with one more SoC specific value like compatible = "nvidia,tegra114-sysram", "mmio-sram"; I'm not sure, if Tegra maintainers want to see a new compatible described in Documentation/devicetree/bindings. > + reg = <0x40000000 0x40000>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0x40000000 0x40000>; > + }; > + > host1x@50000000 { > compatible = "nvidia,tegra114-host1x", "simple-bus"; > reg = <0x50000000 0x00028000>; > diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi > index 8baf00b89efb..a3585ed82646 100644 > --- a/arch/arm/boot/dts/tegra124.dtsi > +++ b/arch/arm/boot/dts/tegra124.dtsi The considerations from above are applicable to the rest of the touched platforms. -- With best wishes, Vladimir
next prev parent reply other threads:[~2017-11-11 14:18 UTC|newest] Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-10-19 21:34 [PATCH v4 0/5] NVIDIA Tegra20 video decoder driver Dmitry Osipenko 2017-10-19 21:34 ` Dmitry Osipenko 2017-10-19 21:34 ` [PATCH v4 1/5] ARM: tegra: Add device tree node to describe IRAM Dmitry Osipenko 2017-10-19 21:34 ` Dmitry Osipenko [not found] ` <8ce696bc2b4b1808f6c7f7a967a3dacd954d2a4e.1508448293.git.digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2017-11-11 14:18 ` Vladimir Zapolskiy [this message] 2017-11-11 14:18 ` Vladimir Zapolskiy 2017-11-12 16:28 ` Dmitry Osipenko 2017-11-12 16:28 ` Dmitry Osipenko 2017-10-19 21:34 ` [PATCH v4 2/5] media: dt: bindings: Add binding for NVIDIA Tegra Video Decoder Engine Dmitry Osipenko 2017-10-19 21:34 ` Dmitry Osipenko [not found] ` <bf5b91666229f9e46ed8c73d6ca2e4b65f86b5ab.1508448293.git.digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2017-10-27 1:01 ` Rob Herring 2017-10-27 1:01 ` Rob Herring 2017-11-11 14:21 ` Vladimir Zapolskiy 2017-11-11 14:21 ` Vladimir Zapolskiy [not found] ` <6492d1af-19fa-253f-2b75-2c37ccd44cbe-ChpfBGZJDbMAvxtiuMwx3w@public.gmane.org> 2017-11-12 16:19 ` Dmitry Osipenko 2017-11-12 16:19 ` Dmitry Osipenko 2017-10-19 21:34 ` [PATCH v4 3/5] staging: Introduce NVIDIA Tegra video decoder driver Dmitry Osipenko 2017-10-19 21:34 ` Dmitry Osipenko 2017-11-11 14:06 ` Vladimir Zapolskiy 2017-11-11 14:06 ` Vladimir Zapolskiy 2017-11-12 16:17 ` Dmitry Osipenko 2017-11-12 16:17 ` Dmitry Osipenko 2017-11-13 8:16 ` Dan Carpenter 2017-11-13 8:16 ` Dan Carpenter 2017-12-04 14:04 ` Hans Verkuil 2017-12-04 14:04 ` Hans Verkuil [not found] ` <ad2da9f4-8899-7db3-493f-5aa15297c33c-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> 2017-12-05 12:17 ` Dmitry Osipenko 2017-12-05 12:17 ` Dmitry Osipenko 2017-12-05 13:03 ` Hans Verkuil 2017-12-05 13:03 ` Hans Verkuil 2017-12-10 18:56 ` Dmitry Osipenko 2017-12-10 18:56 ` Dmitry Osipenko 2017-12-10 19:29 ` Nicolas Dufresne [not found] ` <1512934179.4281.15.camel-dDhyB4GVkw9AFePFGvp55w@public.gmane.org> 2017-12-11 0:12 ` Dmitry Osipenko 2017-12-11 0:12 ` Dmitry Osipenko 2017-12-12 12:54 ` Hans Verkuil 2017-12-12 12:54 ` Hans Verkuil 2017-10-19 21:34 ` [PATCH v4 4/5] ARM: dts: tegra20: Add video decoder node Dmitry Osipenko 2017-10-19 21:34 ` Dmitry Osipenko 2017-10-19 21:34 ` [PATCH v4 5/5] ARM: defconfig: tegra: Enable Video Decoder driver Dmitry Osipenko 2017-10-19 21:34 ` 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=ffa5f1d0-f156-0bc3-6adb-bb55a0a855c4@mleia.com \ --to=vz-chpfbgzjdbmavxtiumwx3w@public.gmane.org \ --cc=dan.carpenter-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org \ --cc=devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org \ --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \ --cc=jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \ --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=mchehab-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \ --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ /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: linkBe 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.