Thread (37 messages) 37 messages, 5 authors, 2025-07-28

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

From: Clement LE GOFFIC <hidden>
Date: 2025-07-23 07:57:32
Also in: linux-clk, linux-devicetree, linux-doc, linux-perf-users, lkml

On 7/22/25 23:58, Julius Werner wrote:
quoted
+  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.
quoted
    - Julius Werner [off-list ref]
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 :-)
quoted
  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.
quoted
-    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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help