Hi,
On Wed, Jan 11, 2006 at 08:42:24AM +0200, Denis Vlasenko wrote:
> On Tuesday 10 January 2006 17:11, Andreas Mohr wrote:
> > Hi all,
> >
> > this patch adds support for configopt parsing for ACX100 EEPROM version v5
> > (please report whether ACX100 v5 now works, preferrably with logs showing
> > configopt activity).
> >
> > Also, we now only start another radio recalibration interval every 5 seconds
> > on ACX111 instead of every second (in case of radio recalibration FAILURES,
> > that is). 5 recalibration attempts every second in case of repeated failure
> > is waaaaay too much, both for undistorted card operation and syslog.
>
> -unsigned int acx_debug = L_ASSOC|L_INIT;
> +unsigned int acx_debug __read_mostly = L_ASSOC|L_INIT;
>
> I think __read_mostly is not worth the compat trouble it currently will
> require
> (#ifdef LINUX_VERSION_CODE...).
OK, and it will add ~ 200 bytes to the driver anyway, so...
> + }
> + else
> + {
>
> Changing this to "} else {".
Hmpf, again.
> Patch applied, thanks!
You're too fast ;)
> BTW, currently we have
>
> priv->scan_probe_delay = 200;
> /* reported to break scanning: priv->scan_probe_delay =
> priv->cfgopt_probe_delay; */
>
> Do I need to uncomment that?
With the sanity check in parse_configopt, I think we should assign the
sanitized EEPROM values now. But OTOH this most likely decreases reliability
of our driver since you never know what exactly the EEPROM had in mind...
Maybe the only thing to be changed here is the comment:
/* we once had priv->scan_probe_delay = priv->cfgopt_probe_delay; but since
those values are "random" EEPROM values, this introduces another unknown
variable to driver reliability, so use a predictable hard-coded init value
instead for now. */
Andreas Mohr
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
