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

[Sumit Gupta <sumitg@nvidia.com>]

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@nvidia.com>
>> ---
>>   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