lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20241104205040adb479a7@mail.local>
Date: Mon, 4 Nov 2024 21:50:40 +0100
From: Alexandre Belloni <alexandre.belloni@...tlin.com>
To: CL Wang <cl634@...estech.com>
Cc: robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
	linux-rtc@...r.kernel.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, tim609@...estech.com
Subject: Re: [PATCH V3 1/1] dt-bindings: rtc: add atcrtc100 Real Time Clock

On 04/11/2024 22:40:53+0800, CL Wang wrote:
> Document devicetree bindings for the Andes atcrtc100 Real Time Clock.
> 
> Signed-off-by: CL Wang <cl634@...estech.com>
> ---
>  .../bindings/rtc/andestech,atcrtc100.yaml     | 44 +++++++++++++++++++
>  MAINTAINERS                                   |  1 +
>  2 files changed, 45 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/rtc/andestech,atcrtc100.yaml
> 
> diff --git a/Documentation/devicetree/bindings/rtc/andestech,atcrtc100.yaml b/Documentation/devicetree/bindings/rtc/andestech,atcrtc100.yaml
> new file mode 100644
> index 000000000000..cf99cff76734
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/rtc/andestech,atcrtc100.yaml
> @@ -0,0 +1,44 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/rtc/andestech,atcrtc100.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Andes ATCRTC100 Real-Time Clock
> +
> +maintainers:
> +  - CL Wang <cl634@...estech.com>
> +
> +allOf:
> +  - $ref: rtc.yaml#
> +
> +properties:
> +  compatible:
> +    enum:
> +      - andestech,atcrtc100
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    items:
> +      - description: Periodic timekeeping interrupt
> +      - description: RTC alarm interrupt
> +
> +  wakeup-source: true

The driver misuses this property. wakeup-source is mutually exclusive
with the alarm interrupt. The driver requires both interrupts to be
present so the property is useless (which is expected as the interrupt
lines are always connected to the CPU)

> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    rtc@...00000 {
> +        compatible = "andestech,atcrtc100";
> +        reg = <0xf0300000 0x100>;
> +        interrupts = <1>, <2>;
> +        wakeup-source;
> +    };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 10342c0fa599..372d7ea53c98 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3357,6 +3357,7 @@ F:	include/linux/mfd/atc260x/*
>  ATCRTC100 RTC DRIVER
>  M:	CL Wang <cl634@...estech.com>
>  S:	Supported
> +F:	Documentation/devicetree/bindings/rtc/andestech,atcrtc100.yaml
>  F:	drivers/rtc/rtc-atcrtc100.c
>  
>  ATHEROS 71XX/9XXX GPIO DRIVER
> -- 
> 2.34.1
> 

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ