linux-kernel - Re: [PATCH] usb/xhci: unify parameter of xhci_msi

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <51A6034E.2080007@intel.com>
Date:	Wed, 29 May 2013 21:31:58 +0800
From:	Alex Shi <alex.shi@...el.com>
To:	Sarah Sharp <sarah.a.sharp@...ux.intel.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC:	gregkh <gregkh@...uxfoundation.org>, Felipe Balbi <balbi@...com>
Subject: Re: [PATCH] usb/xhci: unify parameter of xhci_msi_irq

On 05/28/2013 09:39 PM, Alex Shi wrote:
> On 05/24/2013 10:54 AM, Alex Shi wrote:
>>
>> Ops, I just find a old patch left on my laptop. and it still works on latest
>> Linus tree. I don't remember there is a reasonable excuse to reject this patch.
>>
>> So, anyone like to pick it up?
> 
> Ping ...

Is there someone like to give a bit comments on this patch?
>>
>> ------
>> From 6ae1b9e71f9b14be5774ae9c1b4cf57cd4e747ac Mon Sep 17 00:00:00 2001
>> From: Alex Shi <alex.shi@...el.com>
>> Date: Mon, 11 Jun 2012 15:10:18 +0800
>> Subject: [PATCH] usb/xhci: unify parameter of xhci_msi_irq
>>
>> According to Felipe and Alan's comments the second parameter of irq
>> handler should be 'void *' not a specific structure pointer.
>> So change it.
>>
>> Signed-off-by: Alex Shi <alex.shi@...el.com>
>> Acked-by: Felipe Balbi <balbi@...com>
>> ---
>>  drivers/usb/host/xhci-ring.c |    2 +-
>>  drivers/usb/host/xhci.c      |    4 ++--
>>  drivers/usb/host/xhci.h      |    2 +-
>>  3 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host/xhci-ring.c
>> index 23b4aef..cc8a52f 100644
>> --- a/drivers/usb/host/xhci-ring.c
>> +++ b/drivers/usb/host/xhci-ring.c
>> @@ -2479,7 +2479,7 @@ hw_died:
>>  	return IRQ_HANDLED;
>>  }
>>  
>> -irqreturn_t xhci_msi_irq(int irq, struct usb_hcd *hcd)
>> +irqreturn_t xhci_msi_irq(int irq, void *hcd)
>>  {
>>  	return xhci_irq(hcd);
>>  }
>> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
>> index afdc73e..f7d40c1 100644
>> --- a/drivers/usb/host/xhci.c
>> +++ b/drivers/usb/host/xhci.c
>> @@ -215,7 +215,7 @@ static int xhci_setup_msi(struct xhci_hcd *xhci)
>>  		return ret;
>>  	}
>>  
>> -	ret = request_irq(pdev->irq, (irq_handler_t)xhci_msi_irq,
>> +	ret = request_irq(pdev->irq, xhci_msi_irq,
>>  				0, "xhci_hcd", xhci_to_hcd(xhci));
>>  	if (ret) {
>>  		xhci_dbg(xhci, "disable MSI interrupt\n");
>> @@ -287,7 +287,7 @@ static int xhci_setup_msix(struct xhci_hcd *xhci)
>>  
>>  	for (i = 0; i < xhci->msix_count; i++) {
>>  		ret = request_irq(xhci->msix_entries[i].vector,
>> -				(irq_handler_t)xhci_msi_irq,
>> +				xhci_msi_irq,
>>  				0, "xhci_hcd", xhci_to_hcd(xhci));
>>  		if (ret)
>>  			goto disable_msix;
>> diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h
>> index de3d6e3..737ef54 100644
>> --- a/drivers/usb/host/xhci.h
>> +++ b/drivers/usb/host/xhci.h
>> @@ -1710,7 +1710,7 @@ int xhci_resume(struct xhci_hcd *xhci, bool hibernated);
>>  
>>  int xhci_get_frame(struct usb_hcd *hcd);
>>  irqreturn_t xhci_irq(struct usb_hcd *hcd);
>> -irqreturn_t xhci_msi_irq(int irq, struct usb_hcd *hcd);
>> +irqreturn_t xhci_msi_irq(int irq, void *hcd);
>>  int xhci_alloc_dev(struct usb_hcd *hcd, struct usb_device *udev);
>>  void xhci_free_dev(struct usb_hcd *hcd, struct usb_device *udev);
>>  int xhci_alloc_tt_info(struct xhci_hcd *xhci,
>>
> 
> 


-- 
Thanks
    Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/