On 22/02/2025 22:02, Wilson Ding wrote:

quoted

-----Original Message-----
From: Krzysztof Kozlowski <krzk@kernel.org>
Sent: Friday, February 21, 2025 12:47 AM
To: Wilson Ding <redacted>
Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; linux-arm-
kernel@lists.infradead.org; andrew@lunn.ch; gregory.clement@bootlin.com;
sebastian.hesselbarth@gmail.com; robh@kernel.org; krzk+dt@kernel.org;
conor+dt@kernel.org; p.zabel@pengutronix.de; Sanghoon Lee
[off-list ref]; Geethasowjanya Akula [off-list ref]
Subject: [EXTERNAL] Re: [PATCH v2 2/4] dt-bindings: cp110: Document the
reset controller

On Thu, Feb 20, 2025 at 03: 25: 25PM -0800, Wilson Ding wrote: > Add reset-
controller sub-node into the system controller node, and > document the
supported reset lines. > > Signed-off-by: Wilson Ding
<dingwei@ marvell. com> > 
On Thu, Feb 20, 2025 at 03:25:25PM -0800, Wilson Ding wrote:

quoted

Add reset-controller sub-node into the system controller node, and
document the supported reset lines.

Signed-off-by: Wilson Ding <redacted>
---
 .../arm/marvell/cp110-system-controller.txt   | 43 +++++++++++++++++++

NAK.

You already got feedback that TXT bindings are not accepted.

I am not trying to use this TXT for dt-binding. But as you can see this file
describes all the sub-nodes details, such as mpps, clocks information. So
that I add the reset line info here. If this is not the right place, where shall
I put this information. I wonder if you are asking to convert this txt into
yaml.

You just duplicated the YAML binding in TXT. If you wanted to reference
the child, you would say something that there is a reset device in
binding foo.yaml.

But yeah, ideally this entire binding should be also converted to DT
Schema / YAML.

Best regards,
Krzysztof