Re: [PATCH] drivers:staging: sources for ST core
From: Alan Cox
Date: Mon Apr 26 2010 - 17:58:58 EST
> +/* function pointer pointing to either,
> + * st_kim_recv during registration to receive fw download responses
> + * st_int_recv after registration to receive proto stack responses
> + */
> +void (*st_recv) (void*, const unsigned char*, long);
You still need a context of some kind as I said before to avoid thse
globals and single device limits.
And all the other issues - no explanation of the locking etc don't seem
to have been touched either.
I don't really see how I am supposed to review this stuff when the
locking and flow control assumptions are completely opaque
Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/