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

From: Ferass El Hafidi <funderscore@postmarketos.org>
To: linux-amlogic@lists.infradead.org,
	Alberto Robazza <robazza.alberto@proton.me>,
	robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org,
	neil.armstrong@linaro.org, khilman@baylibre.com
Cc: jbrunet@baylibre.com, martin.blumenstingl@googlemail.com,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-amlogic@lists.infradead.org,
	Alberto Robazza <robazzalberto@gmail.com>
Subject: Re: [PATCH v4 1/3] dt-bindings: vendor-prefixes: add Orbbec
Date: Wed, 29 Jul 2026 16:16:49 +0000	[thread overview]
Message-ID: <tiy37g.1ypi45yn8qibx@postmarketos.org> (raw)
In-Reply-To: <20260729141234.65037-2-robazzalberto@gmail.com>

Hi Alberto,

On Wed, 29 Jul 2026 14:12, Alberto Robazza <robazzalberto@gmail.com> 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@gmail.com>
>---
> 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@lists.infradead.org
>http://lists.infradead.org/mailman/listinfo/linux-amlogic