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@xxxxxxxxxxxx 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@xxxxxxxxxxx>
> >>Signed-off-by: Domen Puncer <domen@xxxxxxxxxxxx>
> >>---
> >>
> >>
> >>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
pgpD3O41a2457.pgp
Description: PGP signature
|