[PATCH 1/4] FB: added a structure for CPU interface to linux/fb.h

Sergei Shtylyov sshtylyov at mvista.com
Tue Nov 23 11:02:14 EST 2010


Hello.

Inki Dae wrote:

> RGB or CPU interfaces could be used as display mode but framebuffer
> framework doesn't consider CPU interface so I added this structure
> because fb_cpumode structure could be used commonly.

> Signed-off-by: Inki Dae <inki.dae at samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> ---
>  include/linux/fb.h |   12 ++++++++++++
>  1 files changed, 12 insertions(+), 0 deletions(-)

> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index 7fca3dc..9b9842c 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -1147,6 +1147,18 @@ struct fb_videomode {
>  	u32 flag;
>  };
>  
> +struct fb_cpumode {
> +	const char *name;	/* optional */
> +	u32 refresh;		/* optional */
> +	u32 xres;
> +	u32 yres;

    I think that those two fields should happily fit into 'u16' type.

> +	u32 pixclock;
> +	u32 cs_setup;
> +	u32 wr_setup;
> +	u32 wr_act;
> +	u32 wr_hold;

    Suspecting the same about the above 4 fields...

WBR, Sergei




More information about the linux-arm-kernel mailing list