[Prism54-devel] Re: [patch 2/3] drivers/net/wireless/prism54/islpci_hotplug: Use the DMA_{64, 32}BIT_MASK constants

[Luis R. Rodriguez]

On Mon, Mar 07, 2005 at 12:24:24PM -0500, Jeff Garzik wrote:
> Luis R. Rodriguez wrote:
> >On Sun, Mar 06, 2005 at 11:23:55PM +0100, domen at coderock.org wrote:
> >
> >>Use the DMA_{64,32}BIT_MASK constants from dma-mapping.h
> >>when calling pci_set_dma_mask() or pci_set_consistent_dma_mask()
> >>
> >>Signed-off-by: Tobias Klauser <tklauser at nuerscht.ch>
> >>Signed-off-by: Domen Puncer <domen at coderock.org>
> >>---
> >>
> >>
> >>kj-domen/drivers/net/wireless/prism54/islpci_hotplug.c |    2 +-
> >>1 files changed, 1 insertion(+), 1 deletion(-)
> >>
> >>diff -puN 
> >>drivers/net/wireless/prism54/islpci_hotplug.c~dma_mask-drivers_net_wireless_prism54_islpci_hotplug drivers/net/wireless/prism54/islpci_hotplug.c
> >>--- 
> >>kj/drivers/net/wireless/prism54/islpci_hotplug.c~dma_mask-drivers_net_wireless_prism54_islpci_hotplug	2005-03-05 16:12:02.000000000 +0100
> >>+++ kj-domen/drivers/net/wireless/prism54/islpci_hotplug.c	2005-03-05 
> >>16:12:02.000000000 +0100
> >>@@ -125,7 +125,7 @@ prism54_probe(struct pci_dev *pdev, cons
> >>	}
> >>
> >>	/* enable PCI DMA */
> >>-	if (pci_set_dma_mask(pdev, 0xffffffff)) {
> >>+	if (pci_set_dma_mask(pdev, DMA_32BIT_MASK)) {
> >
> >
> >Is this 2.4 backward compatible? If not we'll have to add to 2.4 compat
> >file on prism54.
> 
> Not compatible, but it is preferred to patch the file and add that 
> definition to a compat header.
> 
> For pci_module_init(), it is preferred to keep pci_module_init() rather 
> than adding compat gunk.
> 
> 	Jeff

Alright, I'll add DMA_32BIT_MASK through compat and leave
pci_module_init as is.  That said:

Patch 1 applied
Patch 2 will be applied after compat work
Patch 3 rejected per Jeff's recomendation due to 2.4 compat

	Luis

-- 
GnuPG Key fingerprint = 113F B290 C6D2 0251 4D84  A34A 6ADD 4937 E20A 525E
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://prism54.org/pipermail/prism54-devel/attachments/20050307/51d795c6/attachment.pgp