Jiri Slaby wrote:
>Rolf Eike Beer napsal(a):
>>Am Mittwoch, 20. Juli 2005 12:40 schrieb Jiri Slaby:
>>>Rolf Eike Beer napsal(a):
>>>>Your patch to arch/sparc64/kernel/ebus.c is broken, the removed and added
>>>>parts do not match in behaviour.
>>>
>>>I can't still see the difference.
>>
>>diff --git a/arch/sparc64/kernel/ebus.c b/arch/sparc64/kernel/ebus.c
>>--- a/arch/sparc64/kernel/ebus.c
>>+++ b/arch/sparc64/kernel/ebus.c
>>@@ -527,8 +527,15 @@ static struct pci_dev *find_next_ebus(st
>> {
>> struct pci_dev *pdev = start;
>>
>>- do {
>>- pdev = pci_find_device(PCI_VENDOR_ID_SUN, PCI_ANY_ID, pdev);
>>+ while (pdev = pci_get_device(PCI_VENDOR_ID_SUN, PCI_ANY_ID, pdev))
>>+ if (pdev->device == PCI_DEVICE_ID_SUN_EBUS ||
>>+ pdev->device == PCI_DEVICE_ID_SUN_RIO_EBUS)
>>+ break;
>>+ *is_rio_p = !!(pdev && (pdev->device == PCI_DEVICE_ID_SUN_RIO_EBUS));
>>+
>>+/* do { // BEFORE \/ AFTER ^
This looks like some sed command went wrong. And I missed that you were
starting a comment here.
>Is there any difference? I don't see any, but... The reading of diff
>file in this case is not the best, maybe...
Yes, that was the problem. I would prefer if you could just remove the code
instead of commenting it out. This would have made this clearer.
If my editor doesn't fool me you are using spaces for indentation of the
while. There has to be a tab.
Question to the sparc folks: is it really needed to preserve the order of the
ebusses? Or would it be possible to do
pdev = pci_find_device(PCI_VENDOR_ID_SUN, PCI_DEVICE_ID_SUN_EBUS)
first and if this returns NULL start again with
pdev = pci_find_device(PCI_VENDOR_ID_SUN, PCI_DEVICE_ID_SUN_RIO_EBUS)
?
Eike
Attachment:
pgpY9FVngfP0E.pgp
Description: PGP signature
- Follow-Ups:
- Re: [PATCH] pci_find_device --> pci_get_device
- From: Jiri Slaby <[email protected]>
- Re: [PATCH] pci_find_device --> pci_get_device
- References:
- [PATCH] pci_find_device --> pci_get_device
- From: Jiri Slaby <[email protected]>
- Re: [PATCH] pci_find_device --> pci_get_device
- From: Rolf Eike Beer <[email protected]>
- Re: [PATCH] pci_find_device --> pci_get_device
- From: Jiri Slaby <[email protected]>
- [PATCH] pci_find_device --> pci_get_device
- Prev by Date: Re: [patch] I-pipe 2.6.12-v0.9-02
- Next by Date: Re: [PATCH] Remove Comtrol mail address from MAINTAINERS
- Previous by thread: Re: [PATCH] pci_find_device --> pci_get_device
- Next by thread: Re: [PATCH] pci_find_device --> pci_get_device
- Index(es):
