diff mbox series

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

Message ID 20250206131300.1295111-2-yelangyan@huaqin.corp-partner.google.com (mailing list archive)
State New, archived
Headers show
Series support for csot-pna957qt1-1 MIPI-DSI panel | expand

Commit Message

Langyan Ye Feb. 6, 2025, 1:12 p.m. UTC
Add "csot" to the Devicetree Vendor Prefix Registry.

Signed-off-by: Langyan Ye <yelangyan@huaqin.corp-partner.google.com>
---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Doug Anderson Feb. 6, 2025, 5:12 p.m. UTC | #1
Hi,

On Thu, Feb 6, 2025 at 5:13 AM Langyan Ye
<yelangyan@huaqin.corp-partner.google.com> wrote:
>
> Add "csot" to the Devicetree Vendor Prefix Registry.
>
> Signed-off-by: Langyan Ye <yelangyan@huaqin.corp-partner.google.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 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@chromium.org>

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

-Doug
Conor Dooley Feb. 6, 2025, 6:13 p.m. UTC | #2
On Thu, Feb 06, 2025 at 09:12:45AM -0800, Doug Anderson wrote:
> Hi,
> 
> On Thu, Feb 6, 2025 at 5:13 AM Langyan Ye
> <yelangyan@huaqin.corp-partner.google.com> wrote:
> >
> > Add "csot" to the Devicetree Vendor Prefix Registry.
> >
> > Signed-off-by: Langyan Ye <yelangyan@huaqin.corp-partner.google.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 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@chromium.org>
> 
> I would assume this will go through the DT tree, not drm-misc. If this
> is wrong then someone should shout.

idk, probably the whole series should go together via drm-misc.
%subject can change if there's a resubmission, otherwise
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Doug Anderson Feb. 6, 2025, 7:21 p.m. UTC | #3
Hi,

On Thu, Feb 6, 2025 at 10:13 AM Conor Dooley <conor@kernel.org> wrote:
>
> On Thu, Feb 06, 2025 at 09:12:45AM -0800, Doug Anderson wrote:
> > Hi,
> >
> > On Thu, Feb 6, 2025 at 5:13 AM Langyan Ye
> > <yelangyan@huaqin.corp-partner.google.com> wrote:
> > >
> > > Add "csot" to the Devicetree Vendor Prefix Registry.
> > >
> > > Signed-off-by: Langyan Ye <yelangyan@huaqin.corp-partner.google.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 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@chromium.org>
> >
> > I would assume this will go through the DT tree, not drm-misc. If this
> > is wrong then someone should shout.
>
> idk, probably the whole series should go together via drm-misc.
> %subject can change if there's a resubmission, otherwise
> Acked-by: Conor Dooley <conor.dooley@microchip.com>

Ah, I'm fine landing it with drm-misc. I'll land it tomorrow unless
there are objections. I'll adjust the subject when landing.

-Doug
Doug Anderson Feb. 7, 2025, 5:58 p.m. UTC | #4
Hi,

On Thu, Feb 6, 2025 at 11:21 AM Doug Anderson <dianders@chromium.org> wrote:
>
> Hi,
>
> On Thu, Feb 6, 2025 at 10:13 AM Conor Dooley <conor@kernel.org> wrote:
> >
> > On Thu, Feb 06, 2025 at 09:12:45AM -0800, Doug Anderson wrote:
> > > Hi,
> > >
> > > On Thu, Feb 6, 2025 at 5:13 AM Langyan Ye
> > > <yelangyan@huaqin.corp-partner.google.com> wrote:
> > > >
> > > > Add "csot" to the Devicetree Vendor Prefix Registry.
> > > >
> > > > Signed-off-by: Langyan Ye <yelangyan@huaqin.corp-partner.google.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 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@chromium.org>
> > >
> > > I would assume this will go through the DT tree, not drm-misc. If this
> > > is wrong then someone should shout.
> >
> > idk, probably the whole series should go together via drm-misc.
> > %subject can change if there's a resubmission, otherwise
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
>
> Ah, I'm fine landing it with drm-misc. I'll land it tomorrow unless
> there are objections. I'll adjust the subject when landing.

Pushed to drm-misc-next with Conor's Ack after adjusting the subject:

[1/3] dt-bindings: vendor-prefixes: add csot
      commit: 75e80af7d62e0dde006c23aec67b9ec8037324db
diff mbox series

Patch

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
   "^csq,.*":
     description: Shenzen Chuangsiqi Technology Co.,Ltd.
   "^ctera,.*":