Subject: Re: [PATCH 2/10] ps3: AV Settings Driver Library

From: Geoff Levand <[email-masked]>
Date: Wed, 31 Jan 2007 08:34:56 -0800

Geert Uytterhoeven wrote:
> Add the PS3 AV Settings Driver Library.
> 
> The AV Settings driver is used to control Audio and Video settings.
> It communicates with the policy manager through the virtual uart.
> 
...  
> +config PS3_PS3AV
> +	tristate "PS3 AV settings driver library"
> +	select PS3_VUART
> +	default y
> +	help
> +	  Include support for the PS3 Platform AV Settings driver library.

Sorry for not saying something before, but I really don't like calling this
a library.  It is modeled as a driver, shows up as a device/driver pair under
/sys/bus/ps3_vuart/, and services a vuart port device.  It should also be fixed
to work as a lodable module.  I think calling it a driver much better
communicates how it fits into the device model.

-Geoff