On Mon, Oct 29, 2012 at 4:03 AM, gshark <gshark.jeong@xxxxxxxxx> wrote:
> 2012년 10월 29일 17:40, Axel Lin 쓴 글:
>
>> We don't need the sysfs file to be world writable or group writable.
>> This file is write-only, change it to S_IWUSR (0200).
>>
>> Signed-off-by: Axel Lin <axel.lin@xxxxxxxxxx>
>> ---
>> drivers/leds/leds-lm355x.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/leds/leds-lm355x.c b/drivers/leds/leds-lm355x.c
>> index 8f4e8e2..6bc0292 100644
>> --- a/drivers/leds/leds-lm355x.c
>> +++ b/drivers/leds/leds-lm355x.c
>> @@ -411,7 +411,7 @@ out:
>> return ret;
>> }
>> -static DEVICE_ATTR(pattern, 0666, NULL,
>> lm3556_indicator_pattern_store);
>> +static DEVICE_ATTR(pattern, S_IWUSR, NULL,
>> lm3556_indicator_pattern_store);
>> static const struct regmap_config lm355x_regmap = {
>> .reg_bits = 8,
>
>
> It's also nice to us. Thank you Alex.
> Acked-by: G.Shark Jeong<gshark.jeong@xxxxxxxxx>
>
>
Applied, thanks,
-Bryan
> --
> To unsubscribe from this list: send the line "unsubscribe linux-leds" 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-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
