Re: [PATCH] dt-bindings: vendor-prefixes: add Huawei

From: Stanislav Jakubek
Date: Mon Jan 10 2022 - 07:13:27 EST


On Mon, Jan 10, 2022 at 09:55:43AM +0000, Jonathan Cameron wrote:
> On Sun, 9 Jan 2022 14:23:11 +0100
> Stanislav Jakubek <stano.jakubek@xxxxxxxxx> wrote:
>
> > Add vendor prefix for Huawei (https://www.huawei.com/en/)
>
> Patch description should say why... Is there an existing binding
> using a huawei vendor prefix?

Not sure about bindings, but there are device trees using the huawei
vendor prefix. I'm aware of at least these two:
/arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts
/arch/arm64/boot/dts/qcom/msm8994-angler-rev-101.dts

Similar situation with the other vendor prefix patches I've sent yesterday.

Should I send a v2 with an updated patch description for this and
the other patches?

Thanks,

Stanislav

>
> Thanks,
>
> Jonathan
>
>
> >
> > Signed-off-by: Stanislav Jakubek <stano.jakubek@xxxxxxxxx>
> > ---
> > 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 a13d6a19c2b4..18ffa2d7379f 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > @@ -515,6 +515,8 @@ patternProperties:
> > description: HannStar Display Co.
> > "^holtek,.*":
> > description: Holtek Semiconductor, Inc.
> > + "^huawei,.*":
> > + description: Huawei Technologies Co., Ltd.
> > "^hugsun,.*":
> > description: Shenzhen Hugsun Technology Co. Ltd.
> > "^hwacom,.*":
>