Thread (12 messages) flat view 12 messages, 4 authors, 1d ago

Re: [PATCH 1/3] dt-bindings: vendor-prefixes: Add vendor prefix for Teclast

From: sashiko-bot@kernel.org
Date: 2026-09-14 21:11:39
Also in: linux-sunxi

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The newly added vendor prefix `^teclast,.*` is inserted out of alphabetical order.
--

commit 5eadfadaf4aefd64a40c08bf27f4fc8d4fc5a0a6
Author: Andre Przywara [off-list ref]

dt-bindings: vendor-prefixes: Add vendor prefix for Teclast

This commit adds a new vendor prefix for Teclast to the device tree
vendor-prefixes.yaml file. Teclast is a manufacturer of tablets, laptops,
and mini PCs.

Link: https://www.teclast.com [1]
quoted hunk ↗ jump to hunk
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
index ba20029693734..6da733d465a31 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1722,6 +1722,8 @@ patternProperties:
     description: Thundercomm Technology Co., Ltd.
   "^thwc,.*":
     description: Shenzhen Tong Heng Wei Chuang Technology Co., Ltd.
+  "^teclast,.*":
+    description: Teclast
[Severity: Low]
Is this newly added vendor prefix inserted out of alphabetical order?

In the patternProperties section of vendor-prefixes.yaml, there is a
requirement to keep the list in alphabetical order. Alphabetically, the
entry "^teclast,.*" should be placed after "^techwell,.*" and before
"^teejet,.*", rather than after "^thwc,.*".
   "^ti,.*":
     description: Texas Instruments
   "^tianma,.*":
-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260914210115.28151-1-andre.przywara@arm.com?part=1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help