diff mbox series

[1/3] dt-bindings: vendor-prefixes: add asair

Message ID 20231107173100.62715-1-anshulusr@gmail.com (mailing list archive)
State Changes Requested
Headers show
Series [1/3] dt-bindings: vendor-prefixes: add asair | expand

Commit Message

Anshul Dalal Nov. 7, 2023, 5:30 p.m. UTC
Aosong Electronic Co., LTD. is a supplier for MEMS sensors such as AHT20
temperature and humidity sensor under the name Asair

Signed-off-by: Anshul Dalal <anshulusr@gmail.com>
---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Krzysztof Kozlowski Nov. 7, 2023, 5:46 p.m. UTC | #1
On 07/11/2023 18:30, Anshul Dalal wrote:
> Aosong Electronic Co., LTD. is a supplier for MEMS sensors such as AHT20
> temperature and humidity sensor under the name Asair

The name of the company is Aosong Electronic or Asair? What does Asair
stands for? Sensors name? Then it is not a vendor prefix.

> 
> Signed-off-by: Anshul Dalal <anshulusr@gmail.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 573578db9509..df3204f9dda6 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -143,6 +143,8 @@ patternProperties:
>      description: Artesyn Embedded Technologies Inc.
>    "^asahi-kasei,.*":
>      description: Asahi Kasei Corp.
> +  "^asair,.*":
> +    description: Aosong Electronic Co., Ltd.
>    "^asc,.*":
>      description: All Sensors Corporation
>    "^asix,.*":

Best regards,
Krzysztof
Anshul Dalal Nov. 8, 2023, 12:32 a.m. UTC | #2
On 11/7/23 23:16, Krzysztof Kozlowski wrote:
> On 07/11/2023 18:30, Anshul Dalal wrote:
>> Aosong Electronic Co., LTD. is a supplier for MEMS sensors such as AHT20
>> temperature and humidity sensor under the name Asair
> 
> The name of the company is Aosong Electronic or Asair? What does Asair
> stands for? Sensors name? Then it is not a vendor prefix.
> 

Asair is the brand name under which Aosong Electronics sells their
products. I thought Asair would be the better choice since
all their products use that as the manufacturer name instead of Aosong
as well as all the retail suppliers of their sensors.

>>
>> Signed-off-by: Anshul Dalal <anshulusr@gmail.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 573578db9509..df3204f9dda6 100644
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>> @@ -143,6 +143,8 @@ patternProperties:
>>      description: Artesyn Embedded Technologies Inc.
>>    "^asahi-kasei,.*":
>>      description: Asahi Kasei Corp.
>> +  "^asair,.*":
>> +    description: Aosong Electronic Co., Ltd.
>>    "^asc,.*":
>>      description: All Sensors Corporation
>>    "^asix,.*":
> 
> Best regards,
> Krzysztof
>
Krzysztof Kozlowski Nov. 8, 2023, 8:23 a.m. UTC | #3
On 08/11/2023 01:32, Anshul Dalal wrote:
> On 11/7/23 23:16, Krzysztof Kozlowski wrote:
>> On 07/11/2023 18:30, Anshul Dalal wrote:
>>> Aosong Electronic Co., LTD. is a supplier for MEMS sensors such as AHT20
>>> temperature and humidity sensor under the name Asair
>>
>> The name of the company is Aosong Electronic or Asair? What does Asair
>> stands for? Sensors name? Then it is not a vendor prefix.
>>
> 
> Asair is the brand name under which Aosong Electronics sells their
> products. I thought Asair would be the better choice since
> all their products use that as the manufacturer name instead of Aosong
> as well as all the retail suppliers of their sensors.

What if Aosong sells new product, e.g. power supply, under name Aosong.
Would you create new prefix? One vendor should have one prefix, so in
that case all other products not sold under Asair should be with "asair".

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
index 573578db9509..df3204f9dda6 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -143,6 +143,8 @@  patternProperties:
     description: Artesyn Embedded Technologies Inc.
   "^asahi-kasei,.*":
     description: Asahi Kasei Corp.
+  "^asair,.*":
+    description: Aosong Electronic Co., Ltd.
   "^asc,.*":
     description: All Sensors Corporation
   "^asix,.*":