Thread (28 messages) 28 messages, 9 authors, 2006-08-28

Re: [PATCH 2.6.17 2/9] NetXen: Hardware access routines

From: Amit S. Kale <hidden>
Date: 2006-08-28 09:52:22

On Monday 21 August 2006 19:33, Stephen Hemminger wrote:
On Mon, 21 Aug 2006 13:57:23 +0530

"Amit S. Kale" [off-list ref] wrote:
quoted
We can certainly create a table for all error messages. It'll hurt
readability of code in many of the other places where printks are used to
indicate some hardware error.
-Amit
My suggestion was intended as an way to handle multiple driver versions
all using the same firmware or vice versa. By locking the firmware and
driver version together you might make maintenance more difficult.
Ah, I had missed that completely in your first email. Thanks for your 
suggestion.

The NetXen firmware will most probably keep changing. It's hardware is 
flexible enough, so the firmware changes will possibly be varied in nature. 
Thinking about this further, it seems we should coalesce firmware dependent 
code into a few isolated functions. While this may be difficult, we should do 
it anyway. Hopefully future changes will not cause these efforts to go waste.
-Amit
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help