Re: [PATCH v3 07/19] dt-bindings: memory: factorise LPDDR channel binding into memory channel

[Clement LE GOFFIC <clement.legoffic@foss.st.com>]

On 7/22/25 23:58, Julius Werner wrote:
>> +  purpose of this node is to overall memory topology of the system, including the
> 
> nit: Might take the opportunity to fix the typo here (missing words:
> "is to describe the overall memory topology").

Yes true.

> 
>>     - Julius Werner <jwerner@chromium.org>
> 
> Why remove me? (Although I'm also not really sure why I'm maintainer
> for this file and Krzysztof for all the others, tbh.)

I didn't remove you. It is just the minus of the maintainer list :-)

> 
>>   examples:
>>     - |
> 
> I think that's a load-bearing pipe character you're removing here?

Didn't remove either. There are spaces before so it it is not the git 
minus char.

> 
>> -    lpddr-channel0 {
>> +    memory-channel0 {
> 
> Just to double-check, the name of this node doesn't really mean
> anything and isn't directly interpreted by the kernel, right? I'm fine
> with changing the example here to fit better with the new expanded
> scope of the schema, but we have existing firmware that generates
> nodes with the `lpddr-channel0` name, I want to make sure that it
> won't break from making changes here.

Oh ok didn't know about that and didn't find it though.
For now no pattern has been defined for the node name so it shouldn't 
break anything.

Best regards,
Clément