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

From: Anshul Dalal <anshulusr@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org,
	devicetree@vger.kernel.org
Cc: Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Jonathan Cameron <jic23@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Shuah Khan <skhan@linuxfoundation.org>,
	linux-kernel-mentees@lists.linuxfoundation.org
Subject: Re: [PATCH 1/3] dt-bindings: vendor-prefixes: add asair
Date: Wed, 8 Nov 2023 06:02:52 +0530	[thread overview]
Message-ID: <f66cab34-266d-4e9f-b53f-26a2948f863f@gmail.com> (raw)
In-Reply-To: <87762a47-9caf-435a-9b73-397384660b11@linaro.org>

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
> 