[3/5] dt-bindings: vendor-prefixes: Add TouchNetix AS

Commit Message

Marco Felsch Nov. 19, 2024, 10:33 p.m. UTC

From: Kamel Bouhara <[e-mail]>

Add vendor prefix for TouchNetix AS (https://www.touchnetix.com/products/).

Signed-off-by: Kamel Bouhara <[e-mail]>
Acked-by: Krzysztof Kozlowski <[e-mail]>
Reviewed-by: Krzysztof Kozlowski <[e-mail]>
Signed-off-by: Marco Felsch <[e-mail]>
---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Krzysztof Kozlowski Nov. 20, 2024, 7:50 a.m. UTC | #1

On Tue, Nov 19, 2024 at 11:33:52PM +0100, Marco Felsch wrote:
> From: Kamel Bouhara <[e-mail]>
> 
> Add vendor prefix for TouchNetix AS (https://www.touchnetix.com/products/).
> 
> Signed-off-by: Kamel Bouhara <[e-mail]>
> Acked-by: Krzysztof Kozlowski <[e-mail]>
> Reviewed-by: Krzysztof Kozlowski <[e-mail]>

I think this did not happen (only Ack).

Best regards,
Krzysztof

Marco Felsch Nov. 20, 2024, 8:15 a.m. UTC | #2

On 24-11-20, Krzysztof Kozlowski wrote:
> On Tue, Nov 19, 2024 at 11:33:52PM +0100, Marco Felsch wrote:
> > From: Kamel Bouhara <[e-mail]>
> > 
> > Add vendor prefix for TouchNetix AS (https://www.touchnetix.com/products/).
> > 
> > Signed-off-by: Kamel Bouhara <[e-mail]>
> > Acked-by: Krzysztof Kozlowski <[e-mail]>
> > Reviewed-by: Krzysztof Kozlowski <[e-mail]>
> 
> I think this did not happen (only Ack).

Huh.. I used b4 to retrieve the latest Bootlin version. According [1] it
was already present :/ I can drop it if you want.

[1] https://lore.kernel.org/all/[e-mail]/

Regards,
  Marco

> 
> Best regards,
> Krzysztof
> 
>

Krzysztof Kozlowski Nov. 21, 2024, 7:47 a.m. UTC | #3

On 20/11/2024 09:15, Marco Felsch wrote:
> On 24-11-20, Krzysztof Kozlowski wrote:
>> On Tue, Nov 19, 2024 at 11:33:52PM +0100, Marco Felsch wrote:
>>> From: Kamel Bouhara <[e-mail]>
>>>
>>> Add vendor prefix for TouchNetix AS (https://www.touchnetix.com/products/).
>>>
>>> Signed-off-by: Kamel Bouhara <[e-mail]>
>>> Acked-by: Krzysztof Kozlowski <[e-mail]>
>>> Reviewed-by: Krzysztof Kozlowski <[e-mail]>
>>
>> I think this did not happen (only Ack).
> 
> Huh.. I used b4 to retrieve the latest Bootlin version. According [1] it
> was already present :/ I can drop it if you want.
> 
> [1] https://lore.kernel.org/all/[e-mail]/
> 
Hm? This points to entirely different patch!

Best regards,
Krzysztof

Kamel BOUHARA Nov. 21, 2024, 8:09 a.m. UTC | #4

Le 2024-11-21 08:47, Krzysztof Kozlowski a écrit :
> On 20/11/2024 09:15, Marco Felsch wrote:
>> On 24-11-20, Krzysztof Kozlowski wrote:
>>> On Tue, Nov 19, 2024 at 11:33:52PM +0100, Marco Felsch wrote:
>>>> From: Kamel Bouhara <[e-mail]>
>>>> 
>>>> Add vendor prefix for TouchNetix AS 
>>>> (https://www.touchnetix.com/products/).
>>>> 
>>>> Signed-off-by: Kamel Bouhara <[e-mail]>
>>>> Acked-by: Krzysztof Kozlowski <[e-mail]>
>>>> Reviewed-by: Krzysztof Kozlowski <[e-mail]>
>>> 
>>> I think this did not happen (only Ack).
>> 
>> Huh.. I used b4 to retrieve the latest Bootlin version. According [1] 
>> it
>> was already present :/ I can drop it if you want.
>> 
>> [1] 
>> https://lore.kernel.org/all/[e-mail]/
>> 
> Hm? This points to entirely different patch!
> 

Hello Marco,

I think the last review from Krzystof is on v7 [1] ?

[1]: 
https://lore.kernel.org/linux-kernel/[e-mail]/

Regards,
--
Kamel Bouhara, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com

Marco Felsch Nov. 21, 2024, 8:45 a.m. UTC | #5

On 24-11-21, Kamel BOUHARA wrote:
> Le 2024-11-21 08:47, Krzysztof Kozlowski a écrit :
> > On 20/11/2024 09:15, Marco Felsch wrote:
> > > On 24-11-20, Krzysztof Kozlowski wrote:
> > > > On Tue, Nov 19, 2024 at 11:33:52PM +0100, Marco Felsch wrote:
> > > > > From: Kamel Bouhara <[e-mail]>
> > > > > 
> > > > > Add vendor prefix for TouchNetix AS
> > > > > (https://www.touchnetix.com/products/).
> > > > > 
> > > > > Signed-off-by: Kamel Bouhara <[e-mail]>
> > > > > Acked-by: Krzysztof Kozlowski <[e-mail]>
> > > > > Reviewed-by: Krzysztof Kozlowski <[e-mail]>
> > > > 
> > > > I think this did not happen (only Ack).
> > > 
> > > Huh.. I used b4 to retrieve the latest Bootlin version. According
> > > [1] it
> > > was already present :/ I can drop it if you want.
> > > 
> > > [1] https://lore.kernel.org/all/[e-mail]/
> > > 
> > Hm? This points to entirely different patch!
> > 
> 
> Hello Marco,
> 
> I think the last review from Krzystof is on v7 [1] ?
> 
> [1]: https://lore.kernel.org/linux-kernel/[e-mail]/

Yeah Krzysztof is right, I pointed to the wrong patch, but still I
didn't added the reviewed-by tag but b4 did. I ran it again and here it
is:

$ b4 am https://lore.kernel.org/all/[e-mail]/
Grabbing thread from lore.kernel.org/all/[e-mail]/t.mbox.gz
Analyzing 9 messages in the thread
Looking for additional code-review trailers on lore.kernel.org
Analyzing 137 code-review messages
Checking attestation on all messages, may take a moment...
---
  ✓ [PATCH v16 1/3] dt-bindings: vendor-prefixes: Add TouchNetix AS
    + Reviewed-by: Krzysztof Kozlowski <[e-mail]> (✗ DKIM/linaro.org)
  ✓ [PATCH v16 2/3] dt-bindings: input: Add TouchNetix axiom touchscreen
  ✓ [PATCH v16 3/3] Input: Add TouchNetix axiom i2c touchscreen driver
  ---
  ✓ Signed: DKIM/bootlin.com
---
Total patches: 3

Krzysztof I'm sorry, shall I drop it from the next version?

Regards,
  Marco

Krzysztof Kozlowski Nov. 21, 2024, 10 a.m. UTC | #6

On 21/11/2024 09:45, Marco Felsch wrote:
> On 24-11-21, Kamel BOUHARA wrote:
>> Le 2024-11-21 08:47, Krzysztof Kozlowski a écrit :
>>> On 20/11/2024 09:15, Marco Felsch wrote:
>>>> On 24-11-20, Krzysztof Kozlowski wrote:
>>>>> On Tue, Nov 19, 2024 at 11:33:52PM +0100, Marco Felsch wrote:
>>>>>> From: Kamel Bouhara <[e-mail]>
>>>>>>
>>>>>> Add vendor prefix for TouchNetix AS
>>>>>> (https://www.touchnetix.com/products/).
>>>>>>
>>>>>> Signed-off-by: Kamel Bouhara <[e-mail]>
>>>>>> Acked-by: Krzysztof Kozlowski <[e-mail]>
>>>>>> Reviewed-by: Krzysztof Kozlowski <[e-mail]>
>>>>>
>>>>> I think this did not happen (only Ack).
>>>>
>>>> Huh.. I used b4 to retrieve the latest Bootlin version. According
>>>> [1] it
>>>> was already present :/ I can drop it if you want.
>>>>
>>>> [1] https://lore.kernel.org/all/[e-mail]/
>>>>
>>> Hm? This points to entirely different patch!
>>>
>>
>> Hello Marco,
>>
>> I think the last review from Krzystof is on v7 [1] ?
>>
>> [1]: https://lore.kernel.org/linux-kernel/[e-mail]/
> 
> Yeah Krzysztof is right, I pointed to the wrong patch, but still I
> didn't added the reviewed-by tag but b4 did. I ran it again and here it
> is:


I know, it happens sometimes. I also experienced it.

> 
> $ b4 am https://lore.kernel.org/all/[e-mail]/
> Grabbing thread from lore.kernel.org/all/[e-mail]/t.mbox.gz
> Analyzing 9 messages in the thread
> Looking for additional code-review trailers on lore.kernel.org
> Analyzing 137 code-review messages
> Checking attestation on all messages, may take a moment...
> ---
>   ✓ [PATCH v16 1/3] dt-bindings: vendor-prefixes: Add TouchNetix AS
>     + Reviewed-by: Krzysztof Kozlowski <[e-mail]> (✗ DKIM/linaro.org)
>   ✓ [PATCH v16 2/3] dt-bindings: input: Add TouchNetix axiom touchscreen
>   ✓ [PATCH v16 3/3] Input: Add TouchNetix axiom i2c touchscreen driver
>   ---
>   ✓ Signed: DKIM/bootlin.com
> ---
> Total patches: 3
> 
> Krzysztof I'm sorry, shall I drop it from the next version?

Yeah, please drop from this patch and keep only Ack here.

Best regards,
Krzysztof

Patch

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
index b320a39de7fe..c40170c0c387 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1515,6 +1515,8 @@  patternProperties:
     description: Toradex AG
   "^toshiba,.*":
     description: Toshiba Corporation
+  "^touchnetix,.*":
+    description: TouchNetix AS
   "^toumaz,.*":
     description: Toumaz
   "^tpk,.*":