From: Colin Ian King <colin.k...@canonical.com>

The const array 'names' can be made static, saves populating it on
the stack and will make it read-only.

Signed-off-by: Colin Ian King <colin.k...@canonical.com>
---
 drivers/hwmon/tmp421.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/hwmon/tmp421.c b/drivers/hwmon/tmp421.c
index e36399213324..8844c9565d2a 100644
--- a/drivers/hwmon/tmp421.c
+++ b/drivers/hwmon/tmp421.c
@@ -226,8 +226,10 @@ static int tmp421_detect(struct i2c_client *client,
 {
        enum chips kind;
        struct i2c_adapter *adapter = client->adapter;
-       const char * const names[] = { "TMP421", "TMP422", "TMP423",
-                                      "TMP441", "TMP442" };
+       static const char * const names[] = {
+               "TMP421", "TMP422", "TMP423",
+               "TMP441", "TMP442"
+       };
        int addr = client->addr;
        u8 reg;
 
-- 
2.17.1

Reply via email to