[PATCH 2/10] ps3: AV Settings Driver

Christoph Hellwig hch at lst.de
Fri Feb 9 07:30:14 EST 2007


On Thu, Feb 08, 2007 at 02:58:31PM +0100, Geert Uytterhoeven wrote:
> Add the PS3 AV Settings Driver.
> 
> The AV Settings driver is used to control Audio and Video settings.
> It communicates with the policy manager through the virtual uart.

please don't use kernel_thread for new code, but rather kthread_create/
ktread_run.  And check the possible errors it can return :)

Using the kthread infrastructure also means you'll have to redo your
synchronisation scheme as it doesn't work very well with this use
of semaphores.  But that's a good thing as these useages are not
easily parseable for most kernel hackers despite beeing tought in
OS101 :)




More information about the Linuxppc-dev mailing list