OSDN Git Service

platform/x86: intel_cht_int33fe: make a couple of local functions static
authorColin Ian King <colin.king@canonical.com>
Wed, 11 Oct 2017 09:30:10 +0000 (10:30 +0100)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Mon, 23 Oct 2017 17:01:52 +0000 (20:01 +0300)
The functions cht_int33fe_check_for_max17047 and cht_int33fe_find_max17047
are local to the source and do not need to be in global scope, so make
them static.

Cleans up sparse warnings:
symbol 'cht_int33fe_check_for_max17047' was not declared. Should it be
static?
symbol 'cht_int33fe_find_max17047' was not declared. Should it be static?

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/platform/x86/intel_cht_int33fe.c

index 94716b2..286c620 100644 (file)
@@ -42,7 +42,7 @@ struct cht_int33fe_data {
  * These helpers are used to work around this by checking if an i2c-client
  * for the max17047 has already been registered.
  */
-int cht_int33fe_check_for_max17047(struct device *dev, void *data)
+static int cht_int33fe_check_for_max17047(struct device *dev, void *data)
 {
        struct i2c_client **max17047 = data;
        struct acpi_device *adev;
@@ -62,7 +62,7 @@ int cht_int33fe_check_for_max17047(struct device *dev, void *data)
        return 1;
 }
 
-struct i2c_client *cht_int33fe_find_max17047(void)
+static struct i2c_client *cht_int33fe_find_max17047(void)
 {
        struct i2c_client *max17047 = NULL;