2021-03-05 16:47:31

by Giulio Benetti

[permalink] [raw]
Subject: [PATCH 1/3] dt-bindings: Add Hycon Technology vendor prefix

From: Giulio Benetti <[email protected]>

Update Documentation/devicetree/bindings/vendor-prefixes.yaml to
include "hycon" as a vendor prefix for "Hycon Technology".
Company website: http://www.hycontek.com/

Signed-off-by: Giulio Benetti <[email protected]>
---
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.
"^hydis,.*":
description: Hydis Technologies
--
2.25.1


2021-03-05 17:27:58

by Jonathan Neuschäfer

[permalink] [raw]
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 <[email protected]>
>
> 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 <[email protected]>
> ---
> 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

Attachments:
(No filename) (1.24 kB)
 signature.asc (849.00 B)
 Download all attachments

2021-03-05 17:37:39

by Giulio Benetti

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: Add Hycon Technology vendor prefix

Hi Jonathan,

Il 05/03/2021 18:25, Jonathan Neuschäfer ha scritto:
> Hello,
>
> On Fri, Mar 05, 2021 at 05:38:32PM +0100, Giulio Benetti wrote:
>> From: Giulio Benetti <[email protected]>
>>
>> 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.

+1

>> Signed-off-by: Giulio Benetti <[email protected]>
>> ---
>> 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.".
>

Thank you, I've modified it and I'll send a v2 with the rest of
patchset.

Best regards
--
Giulio Benetti
CTO

MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale € 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642

2021-03-05 19:34:00

by Giulio Benetti

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: Add Hycon Technology vendor prefix

+Cc Dmitry

On 3/5/21 6:33 PM, Giulio Benetti wrote:
> Hi Jonathan,
>
> Il 05/03/2021 18:25, Jonathan Neuschäfer ha scritto:
>> Hello,
>>
>> On Fri, Mar 05, 2021 at 05:38:32PM +0100, Giulio Benetti wrote:
>>> From: Giulio Benetti <[email protected]>
>>>
>>> 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.
>
> +1
>
>>> Signed-off-by: Giulio Benetti <[email protected]>
>>> ---
>>> 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.".
>>
>
> Thank you, I've modified it and I'll send a v2 with the rest of
> patchset.
>
> Best regards
>

--
Giulio Benetti
Benetti Engineering sas