linux-kernel - Re: [PATCH 1/3] dt-bindings: Add Hycon Technology vendor prefix

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <YEJpogUgOHHPFUQ+@latitude>
Date:   Fri, 5 Mar 2021 18:25:54 +0100
From:   Jonathan Neuschäfer <j.ne@...teo.net>
To:     Giulio Benetti <giulio.benetti@...ettiengineering.com>
Cc:     Rob Herring <robh+dt@...nel.org>,
        Giulio Benetti <giulio.benetti@...ronovasrl.com>,
        Rob Herring <robh@...nel.org>, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] dt-bindings: Add Hycon Technology vendor prefix

Hello,

On Fri, Mar 05, 2021 at 05:38:32PM +0100, Giulio Benetti wrote:
> From: Giulio Benetti <giulio.benetti@...ronovasrl.com>
> 
> Update Documentation/devicetree/bindings/vendor-prefixes.yaml to
> include "hycon" as a vendor prefix for "Hycon Technology".
> Company website: http://www.hycontek.com/

As far as I can see, https would be possible here.

> Signed-off-by: Giulio Benetti <giulio.benetti@...ronovasrl.com>
> ---
>  Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> index a1312637d6ff..51b00aa96dff 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -488,6 +488,8 @@ patternProperties:
>    "^hugsun,.*":
>      description: Shenzhen Hugsun Technology Co. Ltd.
>    "^hwacom,.*":
> +    description: Hycon Technology Corp.
> +  "^hycon,.*":
>      description: HwaCom Systems Inc.

This is the wrong way around. You declared "hwacom," to mean "Hycon
Technology Corp.", and "hycon," to mean "HwaCom Systems Inc.".


Best regards,
Jonathan Neuschäfer

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)