Re: [PATCH v2 0/4] memory: tegra210-emc: Support Device Tree EMC Tables

[Thierry Reding]

On Thu, May 08, 2025 at 01:07:37AM -0500, Aaron Kling via B4 Relay wrote:
> Signed-off-by: Aaron Kling <webgeek1234@xxxxxxxxx>
> ---
> Changes in v2:
> - Add patch to describe the emc table bindings
> - Add patch to allow a fallback compatible on the tegra210 emc device to
>   match firmware expectations
> - Add a patch to include the baseline emc tables on p2180
> - Link to v1: https://lore.kernel.org/r/20250430-tegra210-emc-dt-v1-1-99896fa69341@xxxxxxxxx
> 
> ---
> Aaron Kling (4):
>       dt-bindings: memory-controllers: Describe Tegra210 EMC Tables
>       dt-bindings: memory-controllers: tegra210: Allow fallback compatible
>       arm64: tegra: Add EMC timings to P2180
>       memory: tegra210-emc: Support Device Tree EMC Tables
> 
>  .../nvidia,tegra21-emc-table.yaml                  |  1692 +
>  .../memory-controllers/nvidia,tegra210-emc.yaml    |    44 +-
>  arch/arm64/boot/dts/nvidia/tegra210-p2180-emc.dtsi | 49749 +++++++++++++++++++
>  arch/arm64/boot/dts/nvidia/tegra210-p2180.dtsi     |     1 +
>  drivers/memory/tegra/tegra210-emc-core.c           |   246 +-
>  5 files changed, 51721 insertions(+), 11 deletions(-)

We've had discussions about this in the past, and I don't think this is
going to go anywhere. Device tree maintainers have repeatedly said that
they won't accept this kind of binding, which is, admittedly, a bit non-
sensical. 50,000 lines of DT for EMC tables is just crazy.

The existing binary table bindings were created to avoid the need for
this. I don't know how easy this is to achieve for all bootloaders, but
the expectation was that these tables should be passed in their native
format.

Thierry

Attachment: signature.asc
Description: PGP signature