Re: [PATCH] [SPI] build as module and fix priority inversion problem

[David Brownell]

On Monday 05 December 2005 12:26 pm, Mark Underwood wrote:
> Sorry, made a bit of a mess with the last e-mail :(. Trying again :) 
>  
> This patch fix's the possible priority inversion that vitaly pointed out

That part looks OK to me, though I don't have time to test it just.
It's the change to "use kmalloc if we must".

I'd like to see you split that part out so it can be merged into the
next MM kernel.  (I don't know when that is, but I'd guess RSN.)


> and allows the driver to be built as a module. 

That part won't work right.  Remember, the spi_board_info setup
must be safe to call from arch_init() code; that support needs
to be statically linked.  The full patch for this would split
those parts into a separate file, and expose some internal hooks
to the module.  (Like maybe the struct boardinfo type, and the
board_list of instances, and the lock protecting that list.)

- Dave
-
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/