Re: [PATCH v2 1/2] dt-bindings: rtc: nuvoton: Add DT compatible

From: Krzysztof Kozlowski
Date: Wed Aug 09 2023 - 10:27:56 EST


On 09/08/2023 11:51, Mia Lin wrote:
> Add DT compatible "nuvoton,nct3015y" to select
>
> Signed-off-by: Mia Lin <mimi05633@xxxxxxxxx>
> ---
> Documentation/devicetree/bindings/rtc/nuvoton,nct3018y.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/rtc/nuvoton,nct3018y.yaml b/Documentation/devicetree/bindings/rtc/nuvoton,nct3018y.yaml
> index 4f9b5604acd9..67fc60fd395c 100644
> --- a/Documentation/devicetree/bindings/rtc/nuvoton,nct3018y.yaml
> +++ b/Documentation/devicetree/bindings/rtc/nuvoton,nct3018y.yaml
> @@ -15,7 +15,9 @@ maintainers:
>
> properties:
> compatible:
> - const: nuvoton,nct3018y
> + enum:
> + - nuvoton,nct3018y
> + - nuvoton,nct3015y

Responding here, but based on your changelog and driver:
Why? Why do you need compatibles to verify the reported ID? Verifying ID
does not make sense. At all.

These are compatible devices, so you could have one more compatible
using old one as fallback. And no new entry in the driver.

Best regards,
Krzysztof