* [PATCH v2] dt-bindings: display: verisilicon,dc: make resets non-optional
@ 2026-07-02 5:48 Icenowy Zheng
2026-07-02 16:06 ` Rob Herring (Arm)
0 siblings, 1 reply; 2+ messages in thread
From: Icenowy Zheng @ 2026-07-02 5:48 UTC (permalink / raw)
To: Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
Simona Vetter, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Joey Lu
Cc: dri-devel, devicetree, linux-kernel, Icenowy Zheng, Conor Dooley
The only variant supported by the binding now, the DC8200 controller on
T-Head TH1520, requires three reset lines.
Add the reset properties to the required list to clarify this. The
in-tree th1520.dtsi file isn't affected by this change because the reset
lines are already populated there.
The driver still considers the resets optional, so if another variant
with no reset line appears, only the binding needs tweaking.
Signed-off-by: Icenowy Zheng <zhengxingda@iscas.ac.cn>
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
---
Changes in v2:
- Fixed wrong reset line count of TH1520 in the commit message (Thanks
to Sashiko).
- Added Conor's R-b.
Documentation/devicetree/bindings/display/verisilicon,dc.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/verisilicon,dc.yaml b/Documentation/devicetree/bindings/display/verisilicon,dc.yaml
index 9dc35ab973f20..919a900122012 100644
--- a/Documentation/devicetree/bindings/display/verisilicon,dc.yaml
+++ b/Documentation/devicetree/bindings/display/verisilicon,dc.yaml
@@ -75,6 +75,8 @@ required:
- interrupts
- clocks
- clock-names
+ - resets
+ - reset-names
- ports
additionalProperties: false
--
2.52.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] dt-bindings: display: verisilicon,dc: make resets non-optional
2026-07-02 5:48 [PATCH v2] dt-bindings: display: verisilicon,dc: make resets non-optional Icenowy Zheng
@ 2026-07-02 16:06 ` Rob Herring (Arm)
0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring (Arm) @ 2026-07-02 16:06 UTC (permalink / raw)
To: Icenowy Zheng
Cc: devicetree, Joey Lu, Conor Dooley, Krzysztof Kozlowski, dri-devel,
linux-kernel, Simona Vetter, Conor Dooley, Maarten Lankhorst,
David Airlie, Maxime Ripard, Thomas Zimmermann
On Thu, 02 Jul 2026 13:48:04 +0800, Icenowy Zheng wrote:
> The only variant supported by the binding now, the DC8200 controller on
> T-Head TH1520, requires three reset lines.
>
> Add the reset properties to the required list to clarify this. The
> in-tree th1520.dtsi file isn't affected by this change because the reset
> lines are already populated there.
>
> The driver still considers the resets optional, so if another variant
> with no reset line appears, only the binding needs tweaking.
>
> Signed-off-by: Icenowy Zheng <zhengxingda@iscas.ac.cn>
> Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
> ---
> Changes in v2:
> - Fixed wrong reset line count of TH1520 in the commit message (Thanks
> to Sashiko).
> - Added Conor's R-b.
>
> Documentation/devicetree/bindings/display/verisilicon,dc.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-07-02 16:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-02 5:48 [PATCH v2] dt-bindings: display: verisilicon,dc: make resets non-optional Icenowy Zheng
2026-07-02 16:06 ` Rob Herring (Arm)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox