diff options
| author | Wei Yongjun <weiyongjun1@huawei.com> | 2020-07-29 23:59:14 +0800 |
|---|---|---|
| committer | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2020-08-01 14:40:30 +0300 |
| commit | 066d7f32cc7c9ade76f0d7d5fa4af4015a32acea (patch) | |
| tree | 80cc5ca80f3dbbb66f36b65d0eb0e40feb407ad9 /drivers/platform | |
| parent | 61781298521eb59be6dd82f84c50e64afc77cd24 (diff) | |
platform/x86: thinkpad_acpi: Make some symbols static
The sparse tool complains as follows:
drivers/platform/x86/thinkpad_acpi.c:9636:25: warning:
symbol 'dev_attr_charge_start_threshold' was not declared. Should it be static?
drivers/platform/x86/thinkpad_acpi.c:9642:25: warning:
symbol 'dev_attr_charge_stop_threshold' was not declared. Should it be static?
Those variables are not used outside of thinkpad_acpi.c, so this
commit marks them static.
Fixes: e33929537b76 ("platform/x86: thinkpad_acpi: use standard charge control attribute names")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Diffstat (limited to 'drivers/platform')
| -rw-r--r-- | drivers/platform/x86/thinkpad_acpi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index 9eda2f84a3cf..4864a5c189d4 100644 --- a/drivers/platform/x86/thinkpad_acpi.c +++ b/drivers/platform/x86/thinkpad_acpi.c @@ -9633,13 +9633,13 @@ static ssize_t charge_control_end_threshold_store(struct device *dev, static DEVICE_ATTR_RW(charge_control_start_threshold); static DEVICE_ATTR_RW(charge_control_end_threshold); -struct device_attribute dev_attr_charge_start_threshold = __ATTR( +static struct device_attribute dev_attr_charge_start_threshold = __ATTR( charge_start_threshold, 0644, charge_control_start_threshold_show, charge_control_start_threshold_store ); -struct device_attribute dev_attr_charge_stop_threshold = __ATTR( +static struct device_attribute dev_attr_charge_stop_threshold = __ATTR( charge_stop_threshold, 0644, charge_control_end_threshold_show, |