On Wed September 15 2010 12:28:36 Jonathan Guerin wrote: > >> Should all of the init be changed to use these functions, along with > >> #define init values? > > > > i think it would be best if we could remove the initvals alltoghether and > > use the function to set the ack timeout. but i'm not sure if we need to > > have them initialized to some value at the beginning. in this case can > > we just override them later using the function. or just update the > > initvals numerically (but no defines). > > If there are no initvals, changing the mode won't init them to default > values, will it? Or are you saying that values should not be > initialised until the mode is set? that's what i'm not sure about... but ath5k_hw_set_ack_timeout() could be called from reset so it applies the changes when the mode is set. i think that should be fine, but it needs to be tested -- and i think the same applies to a lot of the initvals. > I've been a bit scared to post here until I had more information > rather than a generic "uhh guys, something's not working". It's been > causing me massive headaches with my work as I'm relying on the spec > for timing values, but finding that the driver is doing something > much, much different. I'm more than happy to post these things, but > I'm not a good enough programmer to just give you patches without > discussing the fact that I may just not be understanding what's really > happening. > Thanks for the help, it's greatly appreciated! same here :) the thing is, like bob said, that we don't really know much about what the HW expects in some of these registers, so unless you can prove your changes to be correct - e.g. by measurements which fit the theoretical model better, or by better thruput, it's hard to say if they are correct or not. so before you worry about how to make a correct patch, just go ahead and change those initvals to what you believe they should be. if it improves something, let's talk about how to correctly fix it. bruno -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
