[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <12ee900f-ed9d-4339-ba96-87c22c5ecbcf@linaro.org>
Date: Thu, 21 Nov 2024 11:00:45 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Marco Felsch <m.felsch@...gutronix.de>,
Kamel BOUHARA <kamel.bouhara@...tlin.com>
Cc: Luis Chamberlain <mcgrof@...nel.org>, Russ Weight
<russ.weight@...ux.dev>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
"Rafael J. Wysocki" <rafael@...nel.org>,
Andrew Morton <akpm@...ux-foundation.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Dmitry Torokhov <dmitry.torokhov@...il.com>,
Marco Felsch <kernel@...gutronix.de>, Henrik Rydberg <rydberg@...math.org>,
Danilo Krummrich <dakr@...hat.com>, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org, linux-input@...r.kernel.org
Subject: Re: [PATCH 3/5] dt-bindings: vendor-prefixes: Add TouchNetix AS
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 <kamel.bouhara@...tlin.com>
>>>>>>
>>>>>> Add vendor prefix for TouchNetix AS
>>>>>> (https://www.touchnetix.com/products/).
>>>>>>
>>>>>> Signed-off-by: Kamel Bouhara <kamel.bouhara@...tlin.com>
>>>>>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>>>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>>>>>
>>>>> 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/20240703142520.207066-3-kamel.bouhara@bootlin.com/
>>>>
>>> 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/821ce1d4-bc15-4764-bbe0-315c57e8536e@linaro.org/
>
> 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/20240703142520.207066-3-kamel.bouhara@bootlin.com/
> Grabbing thread from lore.kernel.org/all/20240703142520.207066-3-kamel.bouhara@...tlin.com/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 <krzysztof.kozlowski@...aro.org> (✗ 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
Powered by blists - more mailing lists