Re: [PATCH v5 1/8] dt-bindings: serial: describe SA8255p

From: Praveen Talari
Date: Thu May 08 2025 - 01:46:30 EST


Hi Krzysztof

Thank you for your patience. I consider your inputs as valuable learning.

On 5/6/2025 11:53 PM, Krzysztof Kozlowski wrote:
On 06/05/2025 20:02, Praveen Talari wrote:
+
+properties:
+ compatible:
+ enum:
+ - qcom,sa8255p-geni-uart
+ - qcom,sa8255p-geni-debug-uart
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ minItems: 1
Nothing changed here, this should be dropped based on previous discussion.

You sent this v5 on 8:02 PM of my time. *THEN* you responded to my
comment at v4 at 8:05 PM. That's the way to waste everyone's time.

I do not understand why interrupt is optional for a new, complete device
description.

On this platform, there is no use case of waking up UART, so we consider the  wake up IRQ as optional.

Thanks,

Praveen


Best regards,
Krzysztof