On Sun, Dec 12, 2021 at 07:46:06PM +0100, Krzysztof Kozlowski wrote: > On 10/12/2021 17:47, Thierry Reding wrote: > > From: Thierry Reding <treding@xxxxxxxxxx> > > > > Document the variant of the memory controller and external memory > > controllers found on Tegra234 and add some memory client and SMMU > > stream ID definitions for use in device tree files. > > > > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > > --- > > Changes in v2: > > - conditionally set minItems for reg properties > > > > .../nvidia,tegra186-mc.yaml | 20 ++++++++++++ > > include/dt-bindings/clock/tegra234-clock.h | 9 ++++++ > > include/dt-bindings/memory/tegra234-mc.h | 32 +++++++++++++++++++ > > 3 files changed, 61 insertions(+) > > create mode 100644 include/dt-bindings/memory/tegra234-mc.h > > > > (...) > > > /** @brief output of mux controlled by CLK_RST_CONTROLLER_CLK_SOURCE_SDMMC4 */ > > diff --git a/include/dt-bindings/memory/tegra234-mc.h b/include/dt-bindings/memory/tegra234-mc.h > > new file mode 100644 > > index 000000000000..2662f70c15c6 > > --- /dev/null > > +++ b/include/dt-bindings/memory/tegra234-mc.h > > @@ -0,0 +1,32 @@ > > +/* SPDX-License-Identifier: (GPL-2.0 OR MIT) */ > > + > > +#ifndef DT_BINDINGS_MEMORY_TEGRA234_MC_H > > +#define DT_BINDINGS_MEMORY_TEGRA234_MC_H > > + > > +/* special clients */ > > +#define TEGRA234_SID_INVALID 0x00 > > +#define TEGRA234_SID_PASSTHROUGH 0x7f > > + > > + > > +/* NISO1 stream IDs */ > > +#define TEGRA234_SID_SDMMC4 0x02 > > +#define TEGRA234_SID_BPMP 0x10 > > + > > +/* > > + * memory client IDs > > + */ > > + > > +/* sdmmcd memory read client */ > > +#define TEGRA234_MEMORY_CLIENT_SDMMCRAB 0x63 > > +/* sdmmcd memory write client */ > > +#define TEGRA234_MEMORY_CLIENT_SDMMCWAB 0x67 > > +/* BPMP read client */ > > +#define TEGRA234_MEMORY_CLIENT_BPMPR 0x93 > > +/* BPMP write client */ > > +#define TEGRA234_MEMORY_CLIENT_BPMPW 0x94 > > +/* BPMPDMA read client */ > > +#define TEGRA234_MEMORY_CLIENT_BPMPDMAR 0x95 > > +/* BPMPDMA write client */ > > +#define TEGRA234_MEMORY_CLIENT_BPMPDMAW 0x96 > > + > > This header is needed by the driver and DTS patches. One of us can apply > it and provide to the other via stable-tag. I think Arnd and Olof prefer > if headers are coming with DTS and are provided to other trees, so maybe > you will take the bindings and provide a tag for me for the driver? Yeah, once you're happy with the series (I've just sent out an updated v3) I can apply this to the Tegra tree and send you a stable tag to pull into your tree. Thierry
Attachment:
signature.asc
Description: PGP signature