Re: [PATCH v2 1/3] dt-bindings: vendor: add csot

From: Doug Anderson
Date: Thu Feb 06 2025 - 12:13:10 EST


Hi,

On Thu, Feb 6, 2025 at 5:13 AM Langyan Ye
<yelangyan@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:
>
> Add "csot" to the Devicetree Vendor Prefix Registry.
>
> Signed-off-by: Langyan Ye <yelangyan@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
> ---
> 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 42d14899d584..375f1f7c79ef 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -336,6 +336,8 @@ patternProperties:
> description: Crystalfontz America, Inc.
> "^csky,.*":
> description: Hangzhou C-SKY Microsystems Co., Ltd
> + "^csot,.*":
> + description: Guangzhou China Star Optoelectronics Technology Co., Ltd

Doing a `git log` on
`Documentation/devicetree/bindings/vendor-prefixes.yaml` shows that
most patches use the subject prefix `dt-bindings: vendor-prefixes`,
not `dt-bindings: vendor`. If device-tree folks care about this
difference and they don't want to fix it when applying, they might
request you to send a new version.

In any case, that's fairly minor so I'm OK with:

Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>

I would assume this will go through the DT tree, not drm-misc. If this
is wrong then someone should shout.

-Doug