Thread (13 messages) 13 messages, 5 authors, 2026-01-13

Re: [PATCH V4 2/4] dt-bindings: ufs: Document bindings for SA8255P UFS Host Controller

From: Ram Kumar Dwivedi <hidden>
Date: 2026-01-13 07:25:22
Also in: linux-arm-msm, linux-scsi, lkml


On 07-Jan-26 1:08 PM, Krzysztof Kozlowski wrote:
On Tue, Jan 06, 2026 at 07:10:06PM +0530, Ram Kumar Dwivedi wrote:
quoted
+unevaluatedProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+        ufshc@1d84000 {
Completely messed indentation.

Why did we ask to drop the unnecessary soc node? To make it simpler.

Even if you do not believe that code should be simpler, you should
immediately spot the odd indentation.

For such trivialities you cannot get this patch merged. It's third
revision which you send carelessly.
Hi Krzysztof,

Apologies for the oversight. I missed correcting the indentation after removing the 'soc' node wrapper.
I will ensure the example is correctly indented in the next version.

Thanks,
Ram

Best regards,
Krzysztof
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help