Re: [PATCH v2] dt-bindings: memory: tegra: Add external memory controller binding for Tegra210

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/30/19 5:56 AM, Rob Herring wrote:
On Fri, Apr 12, 2019 at 04:08:55PM +0800, Joseph Lo wrote:
Add the binding document for the external memory controller (EMC) which
communicates with external LPDDR4 devices. It includes the bindings of
the EMC node and the EMC table of different rates.

To support high rates for LPDDR4, the EMC table must be trained before
it can be used for runtime clock switching. It has been done by firmware
and merged the training data to the table that the kernel can share the
result. So the bindings are used for both kernel and firmware.

Based on the work of Peter De Schrijver <pdeschrijver@xxxxxxxxxx>.

Signed-off-by: Joseph Lo <josephl@xxxxxxxxxx>
---
This patch splits from the original patch set that supports EMC scaling
with binding document and drivers. Because the binding would be shared
by both firmware and kernel. We want to settle this first. Then we can
fix the kernel and firmware to support the same.

Changes in v2:
* only use "tegra210" string in compatible string and remove the legacy
   "tegra21" string.
* clock-frequency -> fix the unit from kilohertz to hertz
* add "interrupts" property
* s/nvidia,emc-min-mv/nvidia,emc-min-millivolt/
* s/nvidia,gk20a-min-mv/nvidia,gk20a-min-millivolt/
* s/nvidia,source/clock-names/
* fix lots of properties that use underline to hyphen
* s/nvidia,emc-clock-latency-change/nvidia,emc-clock-latency-microsecond/
* add more information in the property descriptions
---
  .../nvidia,tegra210-emc.txt                   | 614 ++++++++++++++++++
  1 file changed, 614 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.txt

diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.txt b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.txt
new file mode 100644
index 000000000000..318239c3c295
--- /dev/null
+++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra210-emc.txt
@@ -0,0 +1,614 @@
+NVIDIA Tegra210 SoC EMC (external memory controller)
+====================================================
+
+Required properties :
+- compatible : should be "nvidia,tegra210-emc".
+- reg : physical base address and length of the controller's registers.
+- clocks : phandles of the possible source clocks
+- clock-names : names of the possible source clocks
+- interrupts : Should contain the EMC general interrupt
+- #address-cells : should be 1
+- #size-cells : should be 0
+- nvidia,memory-controller : phandle of the memory controller.
+- nvidia,use-ram-code : boolean, indicates whether we should use RAM_CODE in
+		        the register to find matching emc-table nodes
+
...
+- nvidia,ptfv : a 12 word array of control data for periodic training
+- nvidia,emc-registers :
+- nvidia,emc-shadow-regs-ca-train :
+- nvidia,emc-shadow-regs-quse-train :
+- nvidia,emc-shadow-regs-rdwr-train :
+  a 221 word array of the following registers (See TRM 18.10.2 for register
+  descriptions)

I think this dumping of register values should not be in DT. I think the
result here will be a lot of duplication of data. How many of the
registers' values vary between 2 frequencies, for example?

We have bindings already for DDR that use timing values (see
bindings/lpddr2/lpddr2.txt). There's one for LPDDR3 being added. This
data is similar to the SPD data which is used in DIMMs. If SPD data is
enough information for any DIMM to work on any PC, then that should be
sufficient for embedded uses too.

Hi Rob,

Thanks for the review. After some internal discussions, we decide to choose another approach. Instead of these EMC table bindings in the DT, we think that would be easier to pass the binary blob of the EMC table. Because the timing/settings in the EMC table could be different depends on vendors and devices (lpddr2/3/4), unify binding may not fit for each vendor.

For Tegra210, lpddr4 is the only SDRAM devices we support, it's more complicated than lpddr2/3. And the rate above 800MHz must be trained before it can be used, it's done by firmware, so the table also includes these training data. Just want to describe the table could have many private settings.

So we want to go through the EMC table with binary blob, it makes the DT binding easier for review and control in the driver. Will re-work the series to support this approach.

reserved-memory {
    #address-cells = <...>;
    #size-cells = <...>;
    ranges;

    emc_table: emc-table@.... {
        compatible = ...;
        reg = <...>;
    };
};

external-memory-controller@... {
    ...
    memory-region = <&emc_table>;
};

Thanks,
Joseph



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux