Re: [PATCH v4 1/3] dt-bindings: vendor-prefixes: add Orbbec

[Robazza Alberto]

Thank you for the feedback. I didn't ignored, indeed the suggestion was implemented in v4. My bad!

Regards,
Alberto

On Wednesday, July 29th, 2026 at 18:31, Ferass El Hafidi <funderscore@xxxxxxxxxxxxxxxx> wrote:

> Hi Alberto,
> 
> On Wed, 29 Jul 2026 14:12, Alberto Robazza <robazzalberto@xxxxxxxxx> wrote:
> >Add the "orbbec" vendor prefix for Orbbec Inc., a manufacturer of
> >3D depth cameras and embedded vision boards.
> >
> >Signed-off-by: Alberto Robazza <robazzalberto@xxxxxxxxx>
> >---
> > Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> 
> You seem to have missed Krzysztof's Acked-by tag in this patch:
> https://lore.kernel.org/all/20260729-unbreakable-observant-clam-ca4f6d@quoll/
> 
> Reviewers will typically give out Acked-by or Reviewed-by tags to your
> patch to signal that the patch looks good to them (typically the former tag)
> or that they have reviewed it in great detail and approve it (typically the
> latter tag). These tags need to be picked up and added to your patch.
> 
> If you use b4 to send the patches, you can run `b4 trailers -u` each
> time before sending a new patchset revision to the mailing list to get
> all Acked-by and Reviewed-by tags reviewers will give to your patch.
> If you don't use b4, you should be able to add them manually still with
> git commands.
> 
> If you don't have them added to your patch, it may sound like you
> ignored their review. Reviewing takes effort and is time-consuming,
> so always make sure you have all of them added.
> 
> Also see: https://docs.kernel.org/process/submitting-patches.html
> 
> Best regards,
> Ferass
> 
> >diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> >index 396044f36..a42899bd2 100644
> >--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> >+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> >@@ -1251,6 +1251,8 @@ patternProperties:
> >     description: Option NV
> >   "^oranth,.*":
> >     description: Shenzhen Oranth Technology Co., Ltd.
> >+  "^orbbec,.*":
> >+    description: Orbbec Inc.
> >   "^ORCL,.*":
> >     description: Oracle Corporation
> >   "^orisetech,.*":
> >--
> >2.55.0
> >
> >
> >_______________________________________________
> >linux-amlogic mailing list
> >linux-amlogic@xxxxxxxxxxxxxxxxxxx
> >http://lists.infradead.org/mailman/listinfo/linux-amlogic
>