Re: [PATCH] usb: typec: tcpci: Request IRQ with IRQF_SHARED

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/7/22 23:16, Xu Yang wrote:
Under resource constraints, this interrupt may use other interrupt line
or this interrupt line may be shared with other devices as long as they
meet the sharing requirements. Besides, This irq flag will not cause other
side effect if tcpci driver is the only user. So a kindly wish to add this
flag.

The last sentence is not appropriate for a commit description.


Signed-off-by: Xu Yang <xu.yang_2@xxxxxxx>
---
  drivers/usb/typec/tcpm/tcpci.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/typec/tcpm/tcpci.c b/drivers/usb/typec/tcpm/tcpci.c
index fe781a38dc82..223a1de4fb1d 100644
--- a/drivers/usb/typec/tcpm/tcpci.c
+++ b/drivers/usb/typec/tcpm/tcpci.c
@@ -838,7 +838,7 @@ static int tcpci_probe(struct i2c_client *client)
err = devm_request_threaded_irq(&client->dev, client->irq, NULL,
  					_tcpci_irq,
-					IRQF_ONESHOT | IRQF_TRIGGER_LOW,
+					IRQF_SHARED | IRQF_ONESHOT | IRQF_TRIGGER_LOW,
  					dev_name(&client->dev), chip);
  	if (err < 0) {
  		tcpci_unregister_port(chip->tcpci);

I don't think this is sufficient. The interrupt handler always returns
IRQ_HANDLED, even if the interrupt status was 0 and the handler did not do
anything. It should return IRQ_NONE in that case.

Guenter




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux