linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marco Felsch <m.felsch@pengutronix.de>
To: Kamel BOUHARA <kamel.bouhara@bootlin.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Luis Chamberlain <mcgrof@kernel.org>,
	Russ Weight <russ.weight@linux.dev>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Marco Felsch <kernel@pengutronix.de>,
	Henrik Rydberg <rydberg@bitmath.org>,
	Danilo Krummrich <dakr@redhat.com>,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	linux-input@vger.kernel.org
Subject: Re: [PATCH 3/5] dt-bindings: vendor-prefixes: Add TouchNetix AS
Date: Thu, 21 Nov 2024 09:45:49 +0100	[thread overview]
Message-ID: <20241121084549.4bzebpuusjreqiik@pengutronix.de> (raw)
In-Reply-To: <3c072ad8e4a15cd23a195855f8e68bfa@bootlin.com>

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@bootlin.com>
> > > > > 
> > > > > Add vendor prefix for TouchNetix AS
> > > > > (https://www.touchnetix.com/products/).
> > > > > 
> > > > > Signed-off-by: Kamel Bouhara <kamel.bouhara@bootlin.com>
> > > > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.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:

$ 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@bootlin.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@linaro.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?

Regards,
  Marco

  reply	other threads:[~2024-11-21  8:46 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-19 22:33 [PATCH 0/5] Input: Add support for TouchNetix aXiom touchscreen Marco Felsch
2024-11-19 22:33 ` [PATCH 1/5] firmware_loader: expand firmware error codes with skip error Marco Felsch
2024-11-19 22:33 ` [PATCH 2/5] firmware_loader: add support to handle FW_UPLOAD_ERR_SKIP Marco Felsch
2024-11-20 16:50   ` Russ Weight
2024-11-20 17:30     ` Marco Felsch
2024-11-20 18:56       ` Russ Weight
2024-11-21  8:41         ` Marco Felsch
2024-11-19 22:33 ` [PATCH 3/5] dt-bindings: vendor-prefixes: Add TouchNetix AS Marco Felsch
2024-11-20  7:50   ` Krzysztof Kozlowski
2024-11-20  8:15     ` Marco Felsch
2024-11-21  7:47       ` Krzysztof Kozlowski
2024-11-21  8:09         ` Kamel BOUHARA
2024-11-21  8:45           ` Marco Felsch [this message]
2024-11-21 10:00             ` Krzysztof Kozlowski
2024-11-19 22:33 ` [PATCH 4/5] dt-bindings: input: Add TouchNetix axiom touchscreen Marco Felsch
2024-11-19 22:33 ` [PATCH 5/5] Input: Add TouchNetix aXiom I2C Touchscreen support Marco Felsch
2024-12-02  9:44   ` Marco Felsch
2024-12-02  9:56     ` Greg Kroah-Hartman
2024-12-02 10:02       ` Marco Felsch
2024-11-21  8:13 ` [PATCH 0/5] Input: Add support for TouchNetix aXiom touchscreen Kamel BOUHARA

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20241121084549.4bzebpuusjreqiik@pengutronix.de \
    --to=m.felsch@pengutronix.de \
    --cc=akpm@linux-foundation.org \
    --cc=conor+dt@kernel.org \
    --cc=dakr@redhat.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=kamel.bouhara@bootlin.com \
    --cc=kernel@pengutronix.de \
    --cc=krzk+dt@kernel.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mcgrof@kernel.org \
    --cc=rafael@kernel.org \
    --cc=robh@kernel.org \
    --cc=russ.weight@linux.dev \
    --cc=rydberg@bitmath.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).