On Mon, 21 Feb 2005, Russell King wrote:
>
> In cs.c, alloc_io_space(), find the line:
>
> if (*base & ~(align-1)) {
>
> delete the ~ and rebuild. This may resolve your problem.
Unlikely. The code is too broken for words.
First off, setting align to zero makes no sense. An alignment cannot be
zero, although some other parts of the system seem to consider a zero
alignment to be the same as a maximum one (ie nonstatic_find_io_region()).
Something like the appended has at least a snowballs chance in hell of
being correct, although considering the amount of confusion in that single
function, I'm not very optimistic.
In particular, the initialization of "align" still has a case where it
sets alignment to 0:
align = (*base) ? (lines ? 1<<lines : 0) : 1;
(ie *base != 0 and lines == 0), and that one is _guaranteed_ to cause a
warning just a line later when we do a (corrected)
if (*base & (align-1))
and then set align to 1 again.
IOW, the whole function just does not make any sense. It didn't make sense
before, it doesn't make sense after it's partly fixed. Somebody who
understands what it is actually supposed to _do_, please explain it..
Linus
--
--- 1.124/drivers/pcmcia/cs.c 2005-01-25 13:50:25 -08:00
+++ edited/drivers/pcmcia/cs.c 2005-02-21 12:13:52 -08:00
@@ -748,14 +748,14 @@
if (*base) {
cs_dbg(s, 0, "odd IO request: num %#x align %#lx\n",
num, align);
- align = 0;
+ align = 1;
} else
while (align && (align < num)) align <<= 1;
}
- if (*base & ~(align-1)) {
+ if (*base & (align-1)) {
cs_dbg(s, 0, "odd IO request: base %#x align %#lx\n",
*base, align);
- align = 0;
+ align = 1;
}
if ((s->features & SS_CAP_STATIC_MAP) && s->io_offset) {
*base = s->io_offset | (*base & 0x0fff);
@@ -766,7 +766,7 @@
potential conflicts, just the most obvious ones. */
for (i = 0; i < MAX_IO_WIN; i++)
if ((s->io[i].NumPorts != 0) &&
- ((s->io[i].BasePort & (align-1)) == *base))
+ ((s->io[i].BasePort & ~(align-1)) == *base))
return 1;
for (i = 0; i < MAX_IO_WIN; i++) {
if (s->io[i].NumPorts == 0) {
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
