On Wed, 12 Aug 2009 02:15:02 +0200
Francisco Jerez <[email protected]> wrote:

> 
> Signed-off-by: Francisco Jerez <[email protected]>
> ---
>  drivers/gpu/drm/nouveau/nouveau_connector.c |  198 
> +++++++++++++--------------
>  drivers/gpu/drm/nouveau/nv50_sor.c          |   14 --
>  2 files changed, 97 insertions(+), 115 deletions(-)
> 
> diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c 
> b/drivers/gpu/drm/nouveau/nouveau_connector.c
> index 54d5e58..d49282e 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_connector.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_connector.c
> @@ -34,8 +34,10 @@
>  #include "nouveau_connector.h"
>  #include "nouveau_hw.h"
>  
> +#define get_slave_funcs(nv_encoder) 
> (to_encoder_slave(to_drm_encoder(nv_encoder))->slave_funcs)

Again, add type safety, please. It works also as documentation.

> +
>  static struct nouveau_encoder *
> -nouveau_connector_encoder_get(struct drm_connector *connector, int type)
> +find_encoder_by_type(struct drm_connector *connector, int type)
>  {
>       struct drm_device *dev = connector->dev;
>       struct nouveau_encoder *nv_encoder;

-- 
Pekka Paalanen
http://www.iki.fi/pq/
_______________________________________________
Nouveau mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/nouveau

Reply via email to