linux-kernel - Re: [PATCH 3/4] dt-bindings: mfd: Add bindings for Samsung SysMgr

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <313e2dc6-e8cf-f6c9-20e2-d7c766b4ecee@kernel.org>
Date:   Wed, 13 Jul 2022 09:14:11 +0200
From:   Krzysztof Kozlowski <krzk@...nel.org>
To:     dj76.yang@...sung.com,
        "jesper.nilsson@...s.com" <jesper.nilsson@...s.com>,
        "lars.persson@...s.com" <lars.persson@...s.com>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
        "lee.jones@...aro.org" <lee.jones@...aro.org>
Cc:     "javierm@...hat.com" <javierm@...hat.com>,
        "bjorn.andersson@...aro.org" <bjorn.andersson@...aro.org>,
        "linux-arm-kernel@...s.com" <linux-arm-kernel@...s.com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Moon-Ki Jun <moonki.jun@...sung.com>,
        Sang Min Kim <hypmean.kim@...sung.com>,
        Wangseok Lee <wangseok.lee@...sung.com>
Subject: Re: [PATCH 3/4] dt-bindings: mfd: Add bindings for Samsung SysMgr

On 13/07/2022 06:56, Dongjin Yang wrote:
> Add an devicetree binding for Samsung system manager.
> This driver is used for SoCs produced by Samsung Foundry to provide
> Samsung sysmgr API. The read/write request of sysmgr is delivered to
> Samsung secure monitor call.

Nope. Second patch with the same - you need to describe the hardware or
underlying firmware, not the driver. "This driver" is not acceptable for
bindings.

> 
> Signed-off-by: Dongjin Yang <dj76.yang@...sung.com>
> ---
>  .../devicetree/bindings/mfd/samsung,sys-mgr.yaml   | 42 ++++++++++++++++++++++
>  MAINTAINERS                                        |  1 +
>  2 files changed, 43 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/samsung,sys-mgr.yaml
> 
> diff --git a/Documentation/devicetree/bindings/mfd/samsung,sys-mgr.yaml b/Documentation/devicetree/bindings/mfd/samsung,sys-mgr.yaml
> new file mode 100644
> index 000000000000..83b9d73a5420
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/samsung,sys-mgr.yaml
> @@ -0,0 +1,42 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/firmware/samsung,sys-mgr.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Samsung System Manager
> +
> +maintainers:
> +  - Jesper Nilsson <jesper.nilsson@...s.com>

How is this related to Axis platforms?

> +
> +description: |
> +  The file documents device tree bindings of Samsung system manager.

Useless description. Describe the hardw

> +
> +properties:
> +  compatible:
> +    enum:
> +      - samsung,sys-mgr
> +      - samsung,sys-mgr-smccc

Sorry, this does not look like hardware. It could be description of
underlying firmware, but you would need to justify it. Look at Qualcomm
SCM for example. Or ARM SCMI, SCPI.


Best regards,
Krzysztof