[PATCHv2] netxen: fix pci bar mapping

Subsystems: networking drivers, the rest

3 messages, 3 authors, 2009-10-12 · open the first message on its own page

[PATCHv2] netxen: fix pci bar mapping

From: Dhananjay Phadke <hidden>
Date: 2009-10-12 06:14:27

Use phys_addr_t data type for pci resource address
instead of unsigned long. This fixes mapping of
pci resources where resource addresses can be larger
than word size (e.g. PAE).

Signed-off-by: Dhananjay Phadke <redacted>
---
 drivers/net/netxen/netxen_nic_main.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/net/netxen/netxen_nic_main.c b/drivers/net/netxen/netxen_nic_main.c
index 9b9eab1..7fc15e9 100644
--- a/drivers/net/netxen/netxen_nic_main.c
+++ b/drivers/net/netxen/netxen_nic_main.c
@@ -595,7 +595,8 @@ netxen_setup_pci_map(struct netxen_adapter *adapter)
 	void __iomem *mem_ptr2 = NULL;
 	void __iomem *db_ptr = NULL;
 
-	unsigned long mem_base, mem_len, db_base, db_len = 0, pci_len0 = 0;
+	resource_size_t mem_base, db_base;
+	unsigned long mem_len, db_len = 0, pci_len0 = 0;
 
 	struct pci_dev *pdev = adapter->pdev;
 	int pci_func = adapter->ahw.pci_func;
-- 
1.6.0.2

Re: [PATCHv2] netxen: fix pci bar mapping

From: David Miller <davem@davemloft.net>
Date: 2009-10-12 06:22:07

From: Dhananjay Phadke <redacted>
Date: Sun, 11 Oct 2009 23:13:51 -0700
Use phys_addr_t data type for pci resource address
instead of unsigned long. This fixes mapping of
pci resources where resource addresses can be larger
than word size (e.g. PAE).

Signed-off-by: Dhananjay Phadke <redacted>
Lazy, you didn't update the commit log message to match the
change you made to the patch.

I think I'll just ignore your patches for a while until
you start to be a little bit less careless.

Thanks.

Re: [PATCHv2] netxen: fix pci bar mapping

From: Dhananjay Phadke <hidden>
Date: 2009-10-12 06:31:06

Huh, okay I was impatient. I will make one more attempt to pass the test.

Thanks.

David Miller wrote:
I think I'll just ignore your patches for a while until
you start to be a little bit less careless.

Thanks.
  

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help