All of lore.kernel.org
 help / color / mirror / Atom feed
From: Phil Edworthy <phil.edworthy@renesas.com>
To: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Cc: Biju Das <biju.das.jz@bp.renesas.com>,
	Aswath Govindraju <a-govindraju@ti.com>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>
Subject: RE: usb: typec: driver for TI HD3SS3220 USB Type-C DRP port controller question
Date: Tue, 21 Jun 2022 08:28:07 +0000	[thread overview]
Message-ID: <TYYPR01MB7086A377DAC14011066C239BF5B39@TYYPR01MB7086.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <YrFx8RFYgF6RWckE@kuha.fi.intel.com>

Hi Heikki,

On 21 June 2022 08:23 Heikki Krogerus wrote:
> On Mon, Jun 20, 2022 at 09:04:10AM +0000, Phil Edworthy wrote:
> > Hi Heikki,
> >
> > I have a board that uses the TI HD3SS3220 USB Type-C DRP port
> > controller [1], however it doesn't have the interrupt line
> > connected. How would you recommend I handle this? Should I get
> > the driver to poll the i2c registers?
> >
> > [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/usb/typec/hd3ss3220.c?h=v5.19-rc3
> 
> If you have some other way of detecting connections on your system,
> then we probable could consider some kind of notifier, but as a
> general solution for the lack of interrupt, I think polling is the
> only way.
Thanks for your comments.

I wonder if this is a problem that affects quite a few drivers, where
a generic polling solution can be used instead of interrupts. This
would require a driver-specific function that can detect when an
interrupt would have occurred, e.g. by reading a register via i2c.
This is already done like that for ethernet PHY interrupts.

> I'll add the list, and Aswath and Roger. Aswath and Roger were looking
> at the same problem with the tps6598x (tipd) driver.

Thanks
Phil

  reply	other threads:[~2022-06-21  8:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <TYYPR01MB70867A4B8CEBD6450A239D45F5B09@TYYPR01MB7086.jpnprd01.prod.outlook.com>
2022-06-21  7:23 ` usb: typec: driver for TI HD3SS3220 USB Type-C DRP port controller question Heikki Krogerus
2022-06-21  8:28   ` Phil Edworthy [this message]
2022-06-21  8:40     ` Biju Das
2022-06-21  8:43       ` Phil Edworthy
2022-06-21  9:04         ` Biju Das

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=TYYPR01MB7086A377DAC14011066C239BF5B39@TYYPR01MB7086.jpnprd01.prod.outlook.com \
    --to=phil.edworthy@renesas.com \
    --cc=a-govindraju@ti.com \
    --cc=biju.das.jz@bp.renesas.com \
    --cc=heikki.krogerus@linux.intel.com \
    --cc=linux-usb@vger.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.