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

[Giulio Benetti]

+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 <giulio.benetti@xxxxxxxxxxxxxxxx>
> > >
> > > 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 <giulio.benetti@xxxxxxxxxxxxxxxx>
> > > ---
> > >    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