[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260107-deft-mouflon-of-shopping-baaece@quoll>
Date: Wed, 7 Jan 2026 08:38:22 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Ram Kumar Dwivedi <ram.dwivedi@....qualcomm.com>
Cc: mani@...nel.org, alim.akhtar@...sung.com, avri.altman@....com,
bvanassche@....org, robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
James.Bottomley@...senpartnership.com, martin.petersen@...cle.com, linux-arm-msm@...r.kernel.org,
linux-scsi@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Anjana Hari <anjana.hari@....qualcomm.com>
Subject: Re: [PATCH V4 2/4] dt-bindings: ufs: Document bindings for SA8255P
UFS Host Controller
On Tue, Jan 06, 2026 at 07:10:06PM +0530, Ram Kumar Dwivedi wrote:
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
> +
> + ufshc@...4000 {
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.
Best regards,
Krzysztof
Powered by blists - more mailing lists