Re: [Linux-fbdev-devel] [PATCH 10/15] viafb: make module parameters visible in sysfs

[Krzysztof H. <krz...@po...>]

On Thu, 21 May 2009 04:28:48 +0800
Harald Welte <Har...@vi...> wrote:

> From: Harald Welte <la...@gn...>
> 
> Signed-off-by: Harald Welte <Har...@vi...>
> ---

Acked-by: Krzysztof Helt <krz...@wp...>

>  drivers/video/via/viafbdev.c |   45 +++++++++++++++++++++--------------------
>  1 files changed, 23 insertions(+), 22 deletions(-)
> 
> diff --git a/drivers/video/via/viafbdev.c b/drivers/video/via/viafbdev.c
> index 00f58c8..980eab8 100644
> --- a/drivers/video/via/viafbdev.c
> +++ b/drivers/video/via/viafbdev.c
> @@ -20,6 +20,7 @@
>   */
>  
>  #include <linux/module.h>
> +#include <linux/stat.h>
>  #define _MASTER_FILE
>  
>  #include "global.h"
> @@ -2570,82 +2571,82 @@ module_init(viafb_init);
>  module_exit(viafb_exit);
>  
>  #ifdef MODULE
> -module_param(viafb_memsize, int, 0);
> +module_param(viafb_memsize, int, S_IRUSR);
>  
> -module_param(viafb_mode, charp, 0);
> +module_param(viafb_mode, charp, S_IRUSR);
>  MODULE_PARM_DESC(viafb_mode, "Set resolution (default=640x480)");
>  
> -module_param(viafb_mode1, charp, 0);
> +module_param(viafb_mode1, charp, S_IRUSR);
>  MODULE_PARM_DESC(viafb_mode1, "Set resolution (default=640x480)");
>  
> -module_param(viafb_bpp, int, 0);
> +module_param(viafb_bpp, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_bpp, "Set color depth (default=32bpp)");
>  
> -module_param(viafb_bpp1, int, 0);
> +module_param(viafb_bpp1, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_bpp1, "Set color depth (default=32bpp)");
>  
> -module_param(viafb_refresh, int, 0);
> +module_param(viafb_refresh, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_refresh,
>  	"Set CRT viafb_refresh rate (default = 60)");
>  
> -module_param(viafb_refresh1, int, 0);
> +module_param(viafb_refresh1, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_refresh1,
>  	"Set CRT refresh rate (default = 60)");
>  
> -module_param(viafb_lcd_panel_id, int, 0);
> +module_param(viafb_lcd_panel_id, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_lcd_panel_id,
>  	"Set Flat Panel type(Default=1024x768)");
>  
> -module_param(viafb_lcd_dsp_method, int, 0);
> +module_param(viafb_lcd_dsp_method, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_lcd_dsp_method,
>  	"Set Flat Panel display scaling method.(Default=Expandsion)");
>  
> -module_param(viafb_SAMM_ON, int, 0);
> +module_param(viafb_SAMM_ON, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_SAMM_ON,
>  	"Turn on/off flag of SAMM(Default=OFF)");
>  
> -module_param(viafb_accel, int, 0);
> +module_param(viafb_accel, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_accel,
>  	"Set 2D Hardware Acceleration.(Default = OFF)");
>  
> -module_param(viafb_active_dev, charp, 0);
> +module_param(viafb_active_dev, charp, S_IRUSR);
>  MODULE_PARM_DESC(viafb_active_dev, "Specify active devices.");
>  
> -module_param(viafb_display_hardware_layout, int, 0);
> +module_param(viafb_display_hardware_layout, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_display_hardware_layout,
>  	"Display Hardware Layout (LCD Only, DVI Only...,etc)");
>  
> -module_param(viafb_second_size, int, 0);
> +module_param(viafb_second_size, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_second_size,
>  	"Set secondary device memory size");
>  
> -module_param(viafb_dual_fb, int, 0);
> +module_param(viafb_dual_fb, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_dual_fb,
>  	"Turn on/off flag of dual framebuffer devices.(Default = OFF)");
>  
> -module_param(viafb_platform_epia_dvi, int, 0);
> +module_param(viafb_platform_epia_dvi, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_platform_epia_dvi,
>  	"Turn on/off flag of DVI devices on EPIA board.(Default = OFF)");
>  
> -module_param(viafb_device_lcd_dualedge, int, 0);
> +module_param(viafb_device_lcd_dualedge, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_device_lcd_dualedge,
>  	"Turn on/off flag of dual edge panel.(Default = OFF)");
>  
> -module_param(viafb_bus_width, int, 0);
> +module_param(viafb_bus_width, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_bus_width,
>  	"Set bus width of panel.(Default = 12)");
>  
> -module_param(viafb_lcd_mode, int, 0);
> +module_param(viafb_lcd_mode, int, S_IRUSR);
>  MODULE_PARM_DESC(viafb_lcd_mode,
>  	"Set Flat Panel mode(Default=OPENLDI)");
>  
> -module_param(viafb_video_dev, charp, 0);
> +module_param(viafb_video_dev, charp, S_IRUSR);
>  MODULE_PARM_DESC(viafb_video_dev, "Specify video devices.");
>  
> -module_param(viafb_lcd_port, charp, 0);
> +module_param(viafb_lcd_port, charp, S_IRUSR);
>  MODULE_PARM_DESC(viafb_lcd_port, "Specify LCD output port.");
>  
> -module_param(viafb_dvi_port, charp, 0);
> +module_param(viafb_dvi_port, charp, S_IRUSR);
>  MODULE_PARM_DESC(viafb_dvi_port, "Specify DVI output port.");
>  
>  MODULE_LICENSE("GPL");
> -- 
> 1.6.2.4
> 
> 
> ------------------------------------------------------------------------------
> Crystal Reports - New Free Runtime and 30 Day Trial
> Check out the new simplified licensing option that enables 
> unlimited royalty-free distribution of the report engine 
> for externally facing server and web deployment. 
> http://p.sf.net/sfu/businessobjects
> _______________________________________________
> Linux-fbdev-devel mailing list
> Lin...@li...
> https://lists.sourceforge.net/lists/listinfo/linux-fbdev-devel
> 

 	
----------------------------------------------------------------------
Sprawdz pogode na dzis!
Kliknij >>> http://link.interia.pl/f217d