Thread (14 messages) 14 messages, 2 authors, 2012-08-31

[PATCH net-next 0/12] qlcnic: patches for new adapter - Qlogic 83XX CNA

flat view
STALE5160d SUPERSEDED

From: Sony Chacko <hidden>
Date: 2012-08-31 06:45:55

Revision v1 of 13 in this series.

Revisions (13)
  1. v1
  2. v1 current
  3. v1
  4. v1
  5. v1
  6. v1
  7. v1
  8. v1
  9. v2 [diff vs current]
  10. v3 [diff vs current]
  11. v4 [diff vs current]
  12. v5 [diff vs current]
  13. v6 [diff vs current]
From: Sony Chacko <redacted>
From: David Miller [mailto:davem@davemloft.net]
Firstly, I'm not going to let you create arbitrary new sysfs
and debugfs crap for facilities that are already supported by
the kernel via other interfaces.
For example, providing a facility to read and write the PCI BARs
is completely pointless.  Use the PCI config space access APIs for
this if you want to do this from userspace.
The patches are also much more verbose than they need to be.
When you move an operation to the new hwops, keep the existing function
name but make it an inline function that invokes the hwop.
That way you won't need hundreds of lines in your patch that look like this:
-	qlcnic_clear_lb_mode(adapter);
+	ahw->hw_ops->clear_loopback(adapter, mode);
Instead you'd have:
static inline void qlcnic_clear_lb_mode(struct qlcnic_adapter *adapter,
						u8 mode) {
struct qlcnic_hardware_context *ahw = &adapter->ahw;
ahw->hw_ops->clear_loopback(adapter, mode); }
I have made the changes as you suggested above.
Please apply the updated pacth series to net-next.

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