mfd: arizona: Downgrade type mismatch messages to dev_warn
authorRichard Fitzgerald <rf@opensource.wolfsonmicro.com>
Fri, 2 Oct 2015 12:29:12 +0000 (13:29 +0100)
committerLee Jones <lee.jones@linaro.org>
Fri, 30 Oct 2015 17:20:27 +0000 (17:20 +0000)
If the declared codec type doesn't match the detected type
we issue a log message but carry on registering the device,
so a dev_warn() is appropriate rather than a dev_err()

Signed-off-by: Richard Fitzgerald <rf@opensource.wolfsonmicro.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/arizona-core.c

index e32d0d30e9cda0f1b513b669db635ef64bedc5df..c3f88e57fe4a16f6a7f460e067c8d1ce9816afd8 100644 (file)
@@ -1130,8 +1130,8 @@ int arizona_dev_init(struct arizona *arizona)
        case 0x5102:
                type_name = "WM5102";
                if (arizona->type != WM5102) {
-                       dev_err(arizona->dev, "WM5102 registered as %d\n",
-                               arizona->type);
+                       dev_warn(arizona->dev, "WM5102 registered as %d\n",
+                                arizona->type);
                        arizona->type = WM5102;
                }
                apply_patch = wm5102_patch;
@@ -1149,8 +1149,8 @@ int arizona_dev_init(struct arizona *arizona)
                        break;
                default:
                        type_name = "WM5110";
-                       dev_err(arizona->dev, "WM5110 registered as %d\n",
-                               arizona->type);
+                       dev_warn(arizona->dev, "WM5110 registered as %d\n",
+                                arizona->type);
                        arizona->type = WM5110;
                        break;
                }
@@ -1161,8 +1161,8 @@ int arizona_dev_init(struct arizona *arizona)
        case 0x8997:
                type_name = "WM8997";
                if (arizona->type != WM8997) {
-                       dev_err(arizona->dev, "WM8997 registered as %d\n",
-                               arizona->type);
+                       dev_warn(arizona->dev, "WM8997 registered as %d\n",
+                                arizona->type);
                        arizona->type = WM8997;
                }
                apply_patch = wm8997_patch;
@@ -1181,8 +1181,8 @@ int arizona_dev_init(struct arizona *arizona)
 
                default:
                        type_name = "WM8998";
-                       dev_err(arizona->dev, "WM8998 registered as %d\n",
-                               arizona->type);
+                       dev_warn(arizona->dev, "WM8998 registered as %d\n",
+                                arizona->type);
                        arizona->type = WM8998;
                }