Re: [EXTERNAL] Re: [PATCH v2 2/4] dt-bindings: cp110: Document the reset controller

[Krzysztof Kozlowski <krzk@kernel.org>]

On 22/02/2025 22:02, Wilson Ding wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk@kernel.org>
>> Sent: Friday, February 21, 2025 12:47 AM
>> To: Wilson Ding <dingwei@marvell.com>
>> 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
>> <salee@marvell.com>; Geethasowjanya Akula <gakula@marvell.com>
>> 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:
>>> Add reset-controller sub-node into the system controller node, and
>>> document the supported reset lines.
>>>
>>> Signed-off-by: Wilson Ding <dingwei@marvell.com>
>>> ---
>>>  .../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