Re: [PATCH] memory: tegra264: Add full set of MC clients

[Sumit Gupta]

On 16/05/26 15:39, Krzysztof Kozlowski wrote:
> External email: Use caution opening links or attachments
>
>
> On Fri, May 15, 2026 at 10:49:11PM +0530, Sumit Gupta wrote:
> > Extend the Tegra264 MC dt-bindings header and tegra264_mc_clients
> > table to cover the full set of memory clients exposed by the SoC.
> > Client name is used for MC fault reporting. Clients managed by the
> > bandwidth manager in BPMP additionally carry their bpmp_id and type.
> >
> > Entries in tegra264_mc_clients[] are sorted in increasing order of
> > their client IDs, which matches the order of the override and
> > security register offsets used in previous SoCs.
> >
> > Signed-off-by: Sumit Gupta <sumitg@xxxxxxxxxx>
> > ---
> >   drivers/memory/tegra/tegra264.c              | 564 +++++++++++++++++--
> >   include/dt-bindings/memory/nvidia,tegra264.h | 287 ++++++++++
> >   2 files changed, 819 insertions(+), 32 deletions(-)
> Please run scripts/checkpatch.pl on the patches and fix reported
> warnings. After that, run also 'scripts/checkpatch.pl --strict' on the
> patches and (probably) fix more warnings. Some warnings can be ignored,
> especially from --strict run, but the code here looks like it needs a
> fix. Feel free to get in touch if the warning is not clear.
>
> Best regards,
> Krzysztof

checkpatch.pl --strict reports one warning:
  WARNING: DT binding docs and includes should be a separate patch.

Will split the nvidia,tegra264.h changes into a separate patch in v2.

Thank you,
Sumit Gupta