lkml.org 
[lkml]   [2021]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] powercap: Replace symbolic permissions with octal permissions
On Thu, Jun 24, 2021 at 10:44 AM Jinchao Wang <wjc@cdjrlc.com> wrote:
>
> Resolve following checkpatch issue,
> Replace symbolic permissions with octal permissions

I don't see much value in this replacement and checkpatch complaining
about the existing code base is not something to worry about in
general.

> Signed-off-by: Jinchao Wang <wjc@cdjrlc.com>
> ---
> drivers/powercap/powercap_sys.c | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/powercap/powercap_sys.c b/drivers/powercap/powercap_sys.c
> index f0654a932b37..e706548c59bb 100644
> --- a/drivers/powercap/powercap_sys.c
> +++ b/drivers/powercap/powercap_sys.c
> @@ -223,46 +223,46 @@ static int seed_constraint_attributes(void)
>
> for (i = 0; i < MAX_CONSTRAINTS_PER_ZONE; ++i) {
> ret = create_constraint_attribute(i, "power_limit_uw",
> - S_IWUSR | S_IRUGO,
> + 0644,
> &constraint_attrs[i].power_limit_attr,
> show_constraint_power_limit_uw,
> store_constraint_power_limit_uw);
> if (ret)
> goto err_alloc;
> ret = create_constraint_attribute(i, "time_window_us",
> - S_IWUSR | S_IRUGO,
> + 0644,
> &constraint_attrs[i].time_window_attr,
> show_constraint_time_window_us,
> store_constraint_time_window_us);
> if (ret)
> goto err_alloc;
> - ret = create_constraint_attribute(i, "name", S_IRUGO,
> + ret = create_constraint_attribute(i, "name", 0444,
> &constraint_attrs[i].name_attr,
> show_constraint_name,
> NULL);
> if (ret)
> goto err_alloc;
> - ret = create_constraint_attribute(i, "max_power_uw", S_IRUGO,
> + ret = create_constraint_attribute(i, "max_power_uw", 0444,
> &constraint_attrs[i].max_power_attr,
> show_constraint_max_power_uw,
> NULL);
> if (ret)
> goto err_alloc;
> - ret = create_constraint_attribute(i, "min_power_uw", S_IRUGO,
> + ret = create_constraint_attribute(i, "min_power_uw", 0444,
> &constraint_attrs[i].min_power_attr,
> show_constraint_min_power_uw,
> NULL);
> if (ret)
> goto err_alloc;
> ret = create_constraint_attribute(i, "max_time_window_us",
> - S_IRUGO,
> + 0444,
> &constraint_attrs[i].max_time_window_attr,
> show_constraint_max_time_window_us,
> NULL);
> if (ret)
> goto err_alloc;
> ret = create_constraint_attribute(i, "min_time_window_us",
> - S_IRUGO,
> + 0444,
> &constraint_attrs[i].min_time_window_attr,
> show_constraint_min_time_window_us,
> NULL);
> @@ -366,9 +366,9 @@ static void create_power_zone_common_attributes(
> &dev_attr_max_energy_range_uj.attr;
> if (power_zone->ops->get_energy_uj) {
> if (power_zone->ops->reset_energy_uj)
> - dev_attr_energy_uj.attr.mode = S_IWUSR | S_IRUSR;
> + dev_attr_energy_uj.attr.mode = 0600;
> else
> - dev_attr_energy_uj.attr.mode = S_IRUSR;
> + dev_attr_energy_uj.attr.mode = 0400;
> power_zone->zone_dev_attrs[count++] =
> &dev_attr_energy_uj.attr;
> }
> --
> 2.31.1
>
>
>

\
 
 \ /
  Last update: 2021-06-30 19:42    [W:0.051 / U:0.236 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site