From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753023Ab2DTIr5 (ORCPT ); Fri, 20 Apr 2012 04:47:57 -0400 Received: from grania.neolocation.net ([91.149.189.243]:50549 "EHLO grania.neolocation.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751736Ab2DTIrz (ORCPT ); Fri, 20 Apr 2012 04:47:55 -0400 X-Greylist: delayed 935 seconds by postgrey-1.27 at vger.kernel.org; Fri, 20 Apr 2012 04:47:54 EDT DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=tekinsoft.com; h=Received:From:To:Cc:Subject:Date:Message-Id:X-Mailer:DomainKey-Status; b=WA4e+gwo/rFPOXAbXNMFORXX9YE6zEM1pZmai9gfqwtdtmD0nLA/V3vtwrgKmQto5yRAcmXuxrqg/PHyml5of1O7lLGb3Ahxlphrb4D7KXBEswfVLTJs+Bz1R3zR0/vIfTeupOO947La1svPcdLZFCCVqHTRdpBZAQE2bT9Ijt0=; From: Yuri Matylitski To: Greg Kroah-Hartman , Johan Hovold , Preston Fick , Yuri Matylitski , Rusty Russell , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Cc: kirill@shutemov.name, Yuri Matylitski Subject: [PATCH] Fixed too small hardcoded timeout values for usb_control_msg in driver for SiliconLabs cp210x-based usb-to-serial adapters. Replaced with USB_CTRL_GET_TIMEOUT/USB_CTRL_SET_TIMEOUT. Date: Fri, 20 Apr 2012 11:32:02 +0300 Message-Id: <1334910722-7357-1-git-send-email-ym@tekinsoft.com> X-Mailer: git-send-email 1.7.0.4 DomainKey-Status: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Signed-off-by: Yuri Matylitski --- drivers/usb/serial/cp210x.c | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/usb/serial/cp210x.c b/drivers/usb/serial/cp210x.c index 0310e2d..e56066b 100644 --- a/drivers/usb/serial/cp210x.c +++ b/drivers/usb/serial/cp210x.c @@ -287,7 +287,8 @@ static int cp210x_get_config(struct usb_serial_port *port, u8 request, /* Issue the request, attempting to read 'size' bytes */ result = usb_control_msg(serial->dev, usb_rcvctrlpipe(serial->dev, 0), request, REQTYPE_DEVICE_TO_HOST, 0x0000, - port_priv->bInterfaceNumber, buf, size, 300); + port_priv->bInterfaceNumber, buf, size, + USB_CTRL_GET_TIMEOUT); /* Convert data into an array of integers */ for (i = 0; i < length; i++) @@ -340,12 +341,14 @@ static int cp210x_set_config(struct usb_serial_port *port, u8 request, result = usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0), request, REQTYPE_HOST_TO_DEVICE, 0x0000, - port_priv->bInterfaceNumber, buf, size, 300); + port_priv->bInterfaceNumber, buf, size, + USB_CTRL_SET_TIMEOUT); } else { result = usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0), request, REQTYPE_HOST_TO_DEVICE, data[0], - port_priv->bInterfaceNumber, NULL, 0, 300); + port_priv->bInterfaceNumber, NULL, 0, + USB_CTRL_SET_TIMEOUT); } kfree(buf); -- 1.7.0.4