From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751439AbdHaMvr (ORCPT ); Thu, 31 Aug 2017 08:51:47 -0400 Received: from esa5.microchip.iphmx.com ([216.71.150.166]:18708 "EHLO esa5.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750942AbdHaMvp (ORCPT ); Thu, 31 Aug 2017 08:51:45 -0400 X-IronPort-AV: E=Sophos;i="5.41,453,1498546800"; d="scan'208";a="4315438" From: Nicolas Ferre To: Alexandre Belloni , , , CC: Boris BREZILLON , Quentin Schulz , Ludovic Desroches , , , Nicolas Ferre , "stable # v4 . 4+" Subject: [PATCH RESEND new email address] usb: gadget: udc: atmel: set vbus irqflags explicitly Date: Thu, 31 Aug 2017 14:51:40 +0200 Message-ID: <20170831125140.21600-1-nicolas.ferre@microchip.com> X-Mailer: git-send-email 2.9.0 In-Reply-To: <20170831124820.21423-1-nicolas.ferre@microchip.com> References: <20170831124820.21423-1-nicolas.ferre@microchip.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The driver triggers actions on both edges of the vbus signal. The former PIO controller was triggering IRQs on both falling and rising edges by default. Newer PIO controller don't, so it's better to set it explicitly to IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING. Without this patch we may trigger the connection with host but only on some bouncing signal conditions and thus lose connecting events. Signed-off-by: Nicolas Ferre Cc: stable # v4.4+ --- drivers/usb/gadget/udc/atmel_usba_udc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/usb/gadget/udc/atmel_usba_udc.c b/drivers/usb/gadget/udc/atmel_usba_udc.c index 98d71400f8a1..a884c022df7a 100644 --- a/drivers/usb/gadget/udc/atmel_usba_udc.c +++ b/drivers/usb/gadget/udc/atmel_usba_udc.c @@ -29,6 +29,8 @@ #include #include "atmel_usba_udc.h" +#define USBA_VBUS_IRQFLAGS (IRQF_ONESHOT \ + | IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING) #ifdef CONFIG_USB_GADGET_DEBUG_FS #include @@ -2361,7 +2363,7 @@ static int usba_udc_probe(struct platform_device *pdev) IRQ_NOAUTOEN); ret = devm_request_threaded_irq(&pdev->dev, gpio_to_irq(udc->vbus_pin), NULL, - usba_vbus_irq_thread, IRQF_ONESHOT, + usba_vbus_irq_thread, USBA_VBUS_IRQFLAGS, "atmel_usba_udc", udc); if (ret) { udc->vbus_pin = -ENODEV; -- 2.9.0