Re: [PATCH] [media] tea5764: Fix module parameter permissions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em 11-07-2011 09:25, Fabio Belavenuto escreveu:
> Hi,
> 
> I'm the author. Sorry for my bad english, I'm from Brazil. :D
> 
> Yes, the intent of the "1" is to set the default value, in case
> compile built-in.
> 
> I like the module to be generic, decided to choose enabled by default.
> 
> Fábio
> 
> 2011/7/11 Jean Delvare <jdelvare@xxxxxxx>:
>> Hi Andy,
>>
>> On Friday 08 July 2011 12:34:38 pm Andy Walls wrote:
>>> Jean Delvare <jdelvare@xxxxxxx> wrote:
>>>> The third parameter of module_param is supposed to represent sysfs
>>>> file permissions. A value of "1" leads to the following:
>>>>
>>>> $ ls -l /sys/module/radio_tea5764/parameters/
>>>> total 0
>>>> ---------x 1 root root 4096 Jul  8 09:17 use_xtal
>>>>
>>>> I am changing it to "0" to align with the other module parameters in
>>>> this driver.
>>>>
>>>> Signed-off-by: Jean Delvare <jdelvare@xxxxxxx>
>>>> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx>
>>>> Cc: Fabio Belavenuto <belavenuto@xxxxxxxxx>
>>>> ---
>>>> drivers/media/radio/radio-tea5764.c |    2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> ---
>>>> linux-3.0-rc6.orig/drivers/media/radio/radio-tea5764.c      2011-05-20
>>>> 10:41:19.000000000 +0200
>>>> +++ linux-3.0-rc6/drivers/media/radio/radio-tea5764.c        2011-07-08
>>>> 09:15:16.000000000 +0200
>>>> @@ -596,7 +596,7 @@ MODULE_AUTHOR(DRIVER_AUTHOR);
>>>> MODULE_DESCRIPTION(DRIVER_DESC);
>>>> MODULE_LICENSE("GPL");
>>>>
>>>> -module_param(use_xtal, int, 1);
>>>> +module_param(use_xtal, int, 0);
>>>> MODULE_PARM_DESC(use_xtal, "Chip have a xtal connected in board");
>>>> module_param(radio_nr, int, 0);
>>>> MODULE_PARM_DESC(radio_nr, "video4linux device number to use");
>>>
>>> To whomever might know:
>>>
>>> Was the intent of the "1" to set the default value of the parameter?
>>
>> My guess is yes, and as a matter of fact 1 is indeed the default value
>> of use_xtal. Only the author of the code (Fabio Belavenuto) could tell
>> for sure, but he seems to be no longer involved so I wouldn't wait for
>> him.

The value there is not the default value, but the permissions. From what
I understand, the xtal frequency should be set at boot time, so setting
it to 000 seems to do the work. So, I'm applying Jean's patch.

>>
>> --
>> Jean Delvare
>> Suse L3
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux