From: Heikki Krogerus <heikki.krogerus@linux.intel.com>
To: Amber Kao <amber.kao@ite.com.tw>
Cc: Jeson Yang <jeson.yang@ite.com.tw>,
Yaode Fang <Yaode.Fang@ite.com.tw>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
linux-usb@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Bling Chiang <Bling.Chiang@ite.com.tw>,
Eric Su <Eric.Su@ite.com.tw>, Doreen Lin <doreen.lin@ite.com.tw>
Subject: Re: [PATCH v7 0/2] usb: typec: ucsi: Add ITE IT8851/IT8853 support
Date: Wed, 12 Aug 2026 11:23:49 +0200 [thread overview]
Message-ID: <anw7pY2vvJAsHiYT@black.igk.intel.com> (raw)
In-Reply-To: <20260807-ucsi-itepd-v7-0-998ac11ed0ef@ite.com.tw>
On Fri, Aug 07, 2026 at 06:17:44PM +0800, Amber Kao wrote:
> This series adds Device Tree bindings and minimal UCSI driver support
> for the ITE IT8851/IT8853 USB Type-C Power Delivery controller over I2C.
>
> Changes in v7:
> - dt-bindings: define 'reg' for the connector@[01] nodes, since a
> unit-address requires a 'reg' property.
> - dt-bindings: dropped Krzysztof's Reviewed-by tag due to the above
> change.
> - ucsi_itepd: use guard(mutex) in itepd_process_event(),
> ucsi_itepd_read_cci() and ucsi_itepd_read_message_in().
> - ucsi_itepd: kept request_threaded_irq() instead of
> devm_request_threaded_irq(). ucsi_destroy() is not devres-managed and
> runs inside remove(), while a devm IRQ is only released after remove()
> has returned; an interrupt in that window would call
> ucsi_notify_common() on an already destroyed ucsi instance.
Why are you answering questions here in the changelog instead of the
thread where they were originally asked?
Changelog should really only list the things that were change.
Thanks,
--
heikki
prev parent reply other threads:[~2026-08-12 9:23 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-07 10:17 [PATCH v7 0/2] usb: typec: ucsi: Add ITE IT8851/IT8853 support Amber Kao
2026-08-07 10:17 ` [PATCH v7 1/2] dt-bindings: usb: Add ITE IT8851/IT8853 Type-C PD controllers Amber Kao
2026-08-07 10:17 ` [PATCH v7 2/2] usb: typec: ucsi: Add ITE IT8851/IT8853 Type-C PD controller driver Amber Kao
2026-08-12 9:24 ` Heikki Krogerus
2026-08-12 9:23 ` Heikki Krogerus [this message]
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=anw7pY2vvJAsHiYT@black.igk.intel.com \
--to=heikki.krogerus@linux.intel.com \
--cc=Bling.Chiang@ite.com.tw \
--cc=Eric.Su@ite.com.tw \
--cc=Yaode.Fang@ite.com.tw \
--cc=amber.kao@ite.com.tw \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=doreen.lin@ite.com.tw \
--cc=gregkh@linuxfoundation.org \
--cc=jeson.yang@ite.com.tw \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=robh@kernel.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