Message ID | 1412945262-6068-5-git-send-email-tomeu.vizoso@collabora.com |
---|---|
State | Superseded, archived |
Headers | show |
On Fri, Oct 10, 2014 at 01:46:55PM +0100, Tomeu Vizoso wrote: > From: Mikko Perttunen <mperttunen@nvidia.com> > > Add binding documentation for the nvidia,tegra124-emc device tree node. > > Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com> > Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> > --- > .../bindings/memory-controllers/tegra-emc.txt | 41 ++++++++++++++++++++++ > 1 file changed, 41 insertions(+) > create mode 100644 Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt > > diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt > new file mode 100644 > index 0000000..6282c6b > --- /dev/null > +++ b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt > @@ -0,0 +1,41 @@ > +Tegra124 SoC EMC controller > + > +Required properties : > +- compatible : "nvidia,tegra124-emc". > +- reg : Should contain 1 entry: > + - EMC register set > + > +The node should contain a "timings@i" subnode for each supported RAM type > + (see field RAM_CODE in register PMC_STRAPPING_OPT_A) > +Required properties for "timings@i" nodes : > +- nvidia,ram-code : Should contain the value of RAM_CODE this timing set > + is used for. > + > +Each "timings@i" node should contain "timing@j" subnodes. One "timing@j" > + node should exist for each supported EMC clock rate. What do the i and j correspond to? > +Required properties for "timing@j" nodes : > +- clock-frequency : Should contain the memory clock rate. > +- nvidia,parent-clock-frequency : Should contain the rate of the EMC > + clock's parent clock. Why are both of these properties necessary? What is the relationship between the two? > +- clocks : Must contain an entry for each entry in clock-names. > + See ../clocks/clock-bindings.txt for details. > +- clock-names : Must include the following entries: > + - emc-parent : EMC's parent clock. Surely the clocks are a property of the EMC, and not the individual timings? > +- The following properties contain EMC timing characterization values: > + - nvidia,emc-zcal-cnt-long > + - nvidia,emc-auto-cal-interval > + - nvidia,emc-ctt-term-ctrl > + - nvidia,emc-cfg > + - nvidia,emc-cfg-2 > + - nvidia,emc-sel-dpd-ctrl > + - nvidia,emc-cfg-dig-dll > + - nvidia,emc-bgbias-ctl0 > + - nvidia,emc-auto-cal-config > + - nvidia,emc-auto-cal-config2 > + - nvidia,emc-auto-cal-config3 > + - nvidia,emc-mode-reset > + - nvidia,emc-mode-1 > + - nvidia,emc-mode-2 > + - nvidia,emc-mode-4 > +- nvidia,emc-configuration : EMC timing characterization data written to > + EMC registers. I have no idea what any of these are. Perhaps these make sense, but I cannot tell. Mark. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 10/10/2014 04:14 PM, Mark Rutland wrote: > On Fri, Oct 10, 2014 at 01:46:55PM +0100, Tomeu Vizoso wrote: >> From: Mikko Perttunen <mperttunen@nvidia.com> >> >> Add binding documentation for the nvidia,tegra124-emc device tree node. >> >> Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com> >> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> >> --- >> .../bindings/memory-controllers/tegra-emc.txt | 41 ++++++++++++++++++++++ >> 1 file changed, 41 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt >> >> diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt >> new file mode 100644 >> index 0000000..6282c6b >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt >> @@ -0,0 +1,41 @@ >> +Tegra124 SoC EMC controller >> + >> +Required properties : >> +- compatible : "nvidia,tegra124-emc". >> +- reg : Should contain 1 entry: >> + - EMC register set >> + >> +The node should contain a "timings@i" subnode for each supported RAM type >> + (see field RAM_CODE in register PMC_STRAPPING_OPT_A) >> +Required properties for "timings@i" nodes : >> +- nvidia,ram-code : Should contain the value of RAM_CODE this timing set >> + is used for. >> + >> +Each "timings@i" node should contain "timing@j" subnodes. One "timing@j" >> + node should exist for each supported EMC clock rate. > > What do the i and j correspond to? Arbitrary integers, preferably starting from 0 and increasing, similar as with regulator@i etc. Each set of `j's is independent. (Actually, these values are never used anywhere for anything). > >> +Required properties for "timing@j" nodes : >> +- clock-frequency : Should contain the memory clock rate. >> +- nvidia,parent-clock-frequency : Should contain the rate of the EMC >> + clock's parent clock. > > Why are both of these properties necessary? > > What is the relationship between the two? There is a certain set of memory timings specified by the hardware manufacturer for each board type. Each memory timing contains the memory rate, parent clock and parent clock frequency and many other values which you mention later. > >> +- clocks : Must contain an entry for each entry in clock-names. >> + See ../clocks/clock-bindings.txt for details. >> +- clock-names : Must include the following entries: >> + - emc-parent : EMC's parent clock. > > Surely the clocks are a property of the EMC, and not the individual > timings? The required parent for each timing is specified by the manufacturer. > >> +- The following properties contain EMC timing characterization values: >> + - nvidia,emc-zcal-cnt-long >> + - nvidia,emc-auto-cal-interval >> + - nvidia,emc-ctt-term-ctrl >> + - nvidia,emc-cfg >> + - nvidia,emc-cfg-2 >> + - nvidia,emc-sel-dpd-ctrl >> + - nvidia,emc-cfg-dig-dll >> + - nvidia,emc-bgbias-ctl0 >> + - nvidia,emc-auto-cal-config >> + - nvidia,emc-auto-cal-config2 >> + - nvidia,emc-auto-cal-config3 >> + - nvidia,emc-mode-reset >> + - nvidia,emc-mode-1 >> + - nvidia,emc-mode-2 >> + - nvidia,emc-mode-4 >> +- nvidia,emc-configuration : EMC timing characterization data written to >> + EMC registers. > > I have no idea what any of these are. Perhaps these make sense, but I > cannot tell. Magic values specified by the manufacturer. There is some sparse documentation in the Tegra TRM. TL;DR: The whole EMC sequence is rather magical and complicated so we stick to manufacturer-specified values rather than trying guess on our own. (Although this patch already calculates some stuff that's safe to calculate and that is manually specified by the downstream NVIDIA kernel.) > > Mark. > -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Oct 14, 2014 at 04:36:45PM +0300, Mikko Perttunen wrote: > On 10/10/2014 04:14 PM, Mark Rutland wrote: > >On Fri, Oct 10, 2014 at 01:46:55PM +0100, Tomeu Vizoso wrote: > >>From: Mikko Perttunen <mperttunen@nvidia.com> > >> > >>Add binding documentation for the nvidia,tegra124-emc device tree node. > >> > >>Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com> > >>Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> > >>--- > >> .../bindings/memory-controllers/tegra-emc.txt | 41 ++++++++++++++++++++++ > >> 1 file changed, 41 insertions(+) > >> create mode 100644 Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt > >> > >>diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt > >>new file mode 100644 > >>index 0000000..6282c6b > >>--- /dev/null > >>+++ b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt > >>@@ -0,0 +1,41 @@ > >>+Tegra124 SoC EMC controller > >>+ > >>+Required properties : > >>+- compatible : "nvidia,tegra124-emc". > >>+- reg : Should contain 1 entry: > >>+ - EMC register set > >>+ > >>+The node should contain a "timings@i" subnode for each supported RAM type > >>+ (see field RAM_CODE in register PMC_STRAPPING_OPT_A) > >>+Required properties for "timings@i" nodes : > >>+- nvidia,ram-code : Should contain the value of RAM_CODE this timing set > >>+ is used for. > >>+ > >>+Each "timings@i" node should contain "timing@j" subnodes. One "timing@j" > >>+ node should exist for each supported EMC clock rate. > > > >What do the i and j correspond to? > > Arbitrary integers, preferably starting from 0 and increasing, similar as > with regulator@i etc. Each set of `j's is independent. (Actually, these > values are never used anywhere for anything). Perhaps they should be used, then? Typically the unit-address matches the (first) value of the reg property. Perhaps in this case it would make sense to make it match the value in clock-frequency for j, and i could correspond to the value of the nvidia,ram-code property. Thierry
diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt new file mode 100644 index 0000000..6282c6b --- /dev/null +++ b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt @@ -0,0 +1,41 @@ +Tegra124 SoC EMC controller + +Required properties : +- compatible : "nvidia,tegra124-emc". +- reg : Should contain 1 entry: + - EMC register set + +The node should contain a "timings@i" subnode for each supported RAM type + (see field RAM_CODE in register PMC_STRAPPING_OPT_A) +Required properties for "timings@i" nodes : +- nvidia,ram-code : Should contain the value of RAM_CODE this timing set + is used for. + +Each "timings@i" node should contain "timing@j" subnodes. One "timing@j" + node should exist for each supported EMC clock rate. +Required properties for "timing@j" nodes : +- clock-frequency : Should contain the memory clock rate. +- nvidia,parent-clock-frequency : Should contain the rate of the EMC + clock's parent clock. +- clocks : Must contain an entry for each entry in clock-names. + See ../clocks/clock-bindings.txt for details. +- clock-names : Must include the following entries: + - emc-parent : EMC's parent clock. +- The following properties contain EMC timing characterization values: + - nvidia,emc-zcal-cnt-long + - nvidia,emc-auto-cal-interval + - nvidia,emc-ctt-term-ctrl + - nvidia,emc-cfg + - nvidia,emc-cfg-2 + - nvidia,emc-sel-dpd-ctrl + - nvidia,emc-cfg-dig-dll + - nvidia,emc-bgbias-ctl0 + - nvidia,emc-auto-cal-config + - nvidia,emc-auto-cal-config2 + - nvidia,emc-auto-cal-config3 + - nvidia,emc-mode-reset + - nvidia,emc-mode-1 + - nvidia,emc-mode-2 + - nvidia,emc-mode-4 +- nvidia,emc-configuration : EMC timing characterization data written to + EMC registers.