On Fri, Jul 31, 2026 at 11:39:52AM +0200, Krzysztof Kozlowski wrote: > On 31/07/2026 11:37, Krzysztof Kozlowski wrote: > > On Thu, Jul 30, 2026 at 09:34:09AM -0700, Laurentiu Mihalcea wrote: > >> From: Laurentiu Mihalcea <[email protected]> > >> > >> The carveout region names are derived based on the DT node names. Because > >> of this, the DT node names are ABI, which is not supposed to happen. > >> > >> Fix this by documenting an additional, optional property: > >> "memory-region-names". This way, the software will have a way to build the > >> carveout region names without relying on the DT node names. > > > > That's not the point of xxx-names property. It only serves as a helper > > to get the resource. > > > >> > >> Signed-off-by: Laurentiu Mihalcea <[email protected]> > >> --- > >> .../devicetree/bindings/remoteproc/fsl,imx-rproc.yaml | 4 ++++ > >> 1 file changed, 4 insertions(+) > >> > >> diff --git > >> a/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml > >> b/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml > >> index c18f71b64889..8e3e6676a95e 100644 > >> --- a/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml > >> +++ b/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml > >> @@ -62,6 +62,10 @@ properties: > >> minItems: 1 > >> maxItems: 32 > >> > >> + memory-region-names: > >> + minItems: 1 > >> + maxItems: 32 > > > > This cannot be flexible and unspecific. Why "yellow submarine" would be > > a correct name? > > > > And I already SAID that at v2, so you just keep pushing this without > implementing feedback. > > NAK
Thanks you review these patch and I already sent out suggestion, could you wait for me check it firstly to off load your work. Frank > > Best regards, > Krzysztof >
