Re: [PATCH v1 3/5] dt-bindings: memory: Add Tegra114 memory client IDs
From: Thierry Reding
Date: Fri Mar 27 2026 - 19:56:33 EST
On Tue, Feb 17, 2026 at 08:22:24AM +0100, Krzysztof Kozlowski wrote:
> On 26/01/2026 20:07, Svyatoslav Ryhel wrote:
> > Each memory client has unique hardware ID, add these IDs.
> >
> > Signed-off-by: Svyatoslav Ryhel <clamor95@xxxxxxxxx>
> > Acked-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> > Reviewed-by: Mikko Perttunen <mperttunen@xxxxxxxxxx>
> > ---
> > include/dt-bindings/memory/tegra114-mc.h | 67 ++++++++++++++++++++++++
>
> This is never a separate commit. Squash with the binding.
You have previously requested that bindings and driver changes be
applied together. If this header file is applied to your memory tree it
means I cannot apply the corresponding DT changes until a release later
because the defines are part of the header included in the DT bindings
patch.
Seems a bit suboptimal. Do you have any good ideas on how to solve that
particular issue? The only one that comes to mind is for you to pick up
the DT changes as well, though that obviously runs a greater risk of
causing merge conflicts down the road.
Thierry
Attachment:
signature.asc
Description: PGP signature