Re: [ath5k-devel] AR5418 works with ath9k

[Pavel Roskin <proski@gnu.org>]

On Wed, 2008-07-23 at 05:13 -0700, Luis R. Rodriguez wrote:
> > Maybe MacBook Pro's card has different eeprom's contents? Anyway the
> > card now works fine, and it also appears quite stable! ;-)
> > Thank you very much for your work!
> 
> I believe your last patch may have been "Use skb_end_pointer() ..." if
> so then this is why you had this issue. Give the git tree a shot:
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/ath9k.git

Luis, I'm afraid that code is still wrong.  It used to be min(), but now
it's essentially max().  You wrote that there was an error in my patch
and I agreed, but we were talking of different errors, I'm afraid.

Since you have touched the code after me and you know more about the
hardware, I would prefer that you fix it and recheck everything once
again.

Here's the patch.  I don't feel comfortable about describing things I
don't fully understand.

diff --git a/ksrc/hw.c b/ksrc/hw.c
index 5c8127e..e0a5464 100644
--- a/ksrc/hw.c
+++ b/ksrc/hw.c
@@ -852,7 +852,7 @@ static inline enum hal_status ath9k_hw_check_eeprom(struct ath_hal *ah)
 	else
 		el = ahp->ah_eeprom.baseEepHeader.length;
 
-	if (el < sizeof(struct ar5416_eeprom))
+	if (el > sizeof(struct ar5416_eeprom))
 		el = sizeof(struct ar5416_eeprom) / sizeof(u_int16_t);
 	else
 		el = el / sizeof(u_int16_t);


-- 
Regards,
Pavel Roskin