power: pmic: Correct debug/error output
authorSimon Glass <sjg@chromium.org>
Sun, 18 Nov 2018 15:14:28 +0000 (08:14 -0700)
committerSimon Glass <sjg@chromium.org>
Thu, 29 Nov 2018 16:30:05 +0000 (09:30 -0700)
There is a newline missing from quite a few printf() strings in these pmic
files. Fix them.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Lukasz Majewski <lukma@denx.de>
18 files changed:
drivers/power/pmic/act8846.c
drivers/power/pmic/as3722.c
drivers/power/pmic/as3722_gpio.c
drivers/power/pmic/lp873x.c
drivers/power/pmic/lp87565.c
drivers/power/pmic/max77686.c
drivers/power/pmic/max8997.c
drivers/power/pmic/max8998.c
drivers/power/pmic/mc34708.c
drivers/power/pmic/palmas.c
drivers/power/pmic/pfuze100.c
drivers/power/pmic/rk8xx.c
drivers/power/pmic/rn5t567.c
drivers/power/pmic/s2mps11.c
drivers/power/pmic/s5m8767.c
drivers/power/pmic/sandbox.c
drivers/power/pmic/stpmu1.c
drivers/power/pmic/tps65090.c

index b0c759c647c0d85aa0f76f4ad8fb4fdb488d466f..186fa907e18bc07731c591896d40626c54ba7428 100644 (file)
@@ -50,7 +50,7 @@ static int act8846_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
+               debug("%s: %s regulators subnode not found!\n", __func__,
                      dev->name);
                return -ENXIO;
        }
index 63df6133549acb3b7ae660aaba2e73be0a9abc49..54adcbf50f5515ade7af71612bf7608cb462360e 100644 (file)
@@ -45,14 +45,14 @@ static int as3722_read_id(struct udevice *dev, uint *idp, uint *revisionp)
 
        ret = pmic_reg_read(dev, AS3722_ASIC_ID1);
        if (ret < 0) {
-               pr_err("failed to read ID1 register: %d", ret);
+               pr_err("failed to read ID1 register: %d\n", ret);
                return ret;
        }
        *idp = ret;
 
        ret = pmic_reg_read(dev, AS3722_ASIC_ID2);
        if (ret < 0) {
-               pr_err("failed to read ID2 register: %d", ret);
+               pr_err("failed to read ID2 register: %d\n", ret);
                return ret;
        }
        *revisionp = ret;
@@ -70,7 +70,7 @@ int as3722_sd_set_voltage(struct udevice *dev, unsigned int sd, u8 value)
 
        ret = pmic_reg_write(dev, AS3722_SD_VOLTAGE(sd), value);
        if (ret < 0) {
-               pr_err("failed to write SD%u voltage register: %d", sd, ret);
+               pr_err("failed to write SD%u voltage register: %d\n", sd, ret);
                return ret;
        }
 
@@ -86,8 +86,8 @@ int as3722_ldo_set_voltage(struct udevice *dev, unsigned int ldo, u8 value)
 
        ret = pmic_reg_write(dev, AS3722_LDO_VOLTAGE(ldo), value);
        if (ret < 0) {
-               pr_err("failed to write LDO%u voltage register: %d", ldo,
-                     ret);
+               pr_err("failed to write LDO%u voltage register: %d\n", ldo,
+                      ret);
                return ret;
        }
 
@@ -101,12 +101,12 @@ static int as3722_probe(struct udevice *dev)
 
        ret = as3722_read_id(dev, &id, &revision);
        if (ret < 0) {
-               pr_err("failed to read ID: %d", ret);
+               pr_err("failed to read ID: %d\n", ret);
                return ret;
        }
 
        if (id != AS3722_DEVICE_ID) {
-               pr_err("unknown device");
+               pr_err("unknown device\n");
                return -ENOENT;
        }
 
index 36f4fbfa4704fafe35e75352aed70fef6d429349..96943bc1ad5d82f185efcbea5fcf3f1b3f2792eb 100644 (file)
@@ -25,7 +25,7 @@ int as3722_gpio_configure(struct udevice *pmic, unsigned int gpio,
 
        err = pmic_reg_write(pmic, AS3722_GPIO_CONTROL(gpio), value);
        if (err) {
-               pr_err("failed to configure GPIO#%u: %d", gpio, err);
+               pr_err("failed to configure GPIO#%u: %d\n", gpio, err);
                return err;
        }
 
@@ -45,7 +45,7 @@ static int as3722_gpio_set_value(struct udevice *dev, unsigned int gpio,
 
        err = pmic_reg_read(pmic, AS3722_GPIO_SIGNAL_OUT);
        if (err < 0) {
-               pr_err("failed to read GPIO signal out register: %d", err);
+               pr_err("failed to read GPIO signal out register: %d\n", err);
                return err;
        }
        value = err;
@@ -60,7 +60,7 @@ static int as3722_gpio_set_value(struct udevice *dev, unsigned int gpio,
 
        err = pmic_reg_write(pmic, AS3722_GPIO_SIGNAL_OUT, value);
        if (err) {
-               pr_err("failed to set GPIO#%u %s: %d", gpio, l, err);
+               pr_err("failed to set GPIO#%u %s: %d\n", gpio, l, err);
                return err;
        }
 
@@ -83,13 +83,14 @@ int as3722_gpio_direction_output(struct udevice *dev, unsigned int gpio,
 
        err = pmic_reg_write(pmic, AS3722_GPIO_CONTROL(gpio), value);
        if (err) {
-               pr_err("failed to configure GPIO#%u as output: %d", gpio, err);
+               pr_err("failed to configure GPIO#%u as output: %d\n", gpio,
+                      err);
                return err;
        }
 
        err = as3722_gpio_set_value(pmic, gpio, value);
        if (err < 0) {
-               pr_err("failed to set GPIO#%u high: %d", gpio, err);
+               pr_err("failed to set GPIO#%u high: %d\n", gpio, err);
                return err;
        }
 
index 432ad4cecf62404bf80fa893e1e5eec1dbb5f987..4ae4043b943727d209bb1fb29bfef98664f6a720 100644 (file)
@@ -24,7 +24,7 @@ static int lp873x_write(struct udevice *dev, uint reg, const uint8_t *buff,
                          int len)
 {
        if (dm_i2c_write(dev, reg, buff, len)) {
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -34,7 +34,7 @@ static int lp873x_write(struct udevice *dev, uint reg, const uint8_t *buff,
 static int lp873x_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 {
        if (dm_i2c_read(dev, reg, buff, len)) {
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -48,7 +48,7 @@ static int lp873x_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
+               debug("%s: %s regulators subnode not found!\n", __func__,
                      dev->name);
                return -ENXIO;
        }
index 450dbb8a780537a3c1c23b19ba19722644c6dcc0..3e5fc608d28b23768b26206ebbd4dc0a45c95070 100644 (file)
@@ -26,7 +26,7 @@ static int lp87565_write(struct udevice *dev, uint reg, const uint8_t *buff,
 
        ret = dm_i2c_write(dev, reg, buff, len);
        if (ret)
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
@@ -37,7 +37,7 @@ static int lp87565_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buff, len);
        if (ret)
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
@@ -49,7 +49,7 @@ static int lp87565_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
+               debug("%s: %s regulators subnode not found!\n", __func__,
                      dev->name);
                return -ENXIO;
        }
index 834713af28655bf01ecb841b449e1072c4f4898b..8e3a8cf870bae55d7c9aae33d674763104ec886d 100644 (file)
@@ -28,7 +28,7 @@ static int max77686_write(struct udevice *dev, uint reg, const uint8_t *buff,
                          int len)
 {
        if (dm_i2c_write(dev, reg, buff, len)) {
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -38,7 +38,7 @@ static int max77686_write(struct udevice *dev, uint reg, const uint8_t *buff,
 static int max77686_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 {
        if (dm_i2c_read(dev, reg, buff, len)) {
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -52,8 +52,8 @@ static int max77686_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "voltage-regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
-                                                            dev->name);
+               debug("%s: %s regulators subnode not found!\n", __func__,
+                     dev->name);
                return -ENXIO;
        }
 
index 0dcdbad5832f82180aebd79223b04f9fc465f9b0..dbae155fb3425cb3d77a27f60cc9c1e8372b8287 100644 (file)
@@ -23,7 +23,7 @@ static int max8997_write(struct udevice *dev, uint reg, const uint8_t *buff,
 
        ret = dm_i2c_write(dev, reg, buff, len);
        if (ret)
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
@@ -34,7 +34,7 @@ static int max8997_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buff, len);
        if (ret)
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
index f571add6e6a375279b50db36cd0ab3fa1fbb2fdf..f58d9f2d74c1d186f2db43f87291dad258b1ebf1 100644 (file)
@@ -23,7 +23,7 @@ static int max8998_write(struct udevice *dev, uint reg, const uint8_t *buff,
 
        ret = dm_i2c_write(dev, reg, buff, len);
        if (ret)
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
@@ -34,7 +34,7 @@ static int max8998_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buff, len);
        if (ret)
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
index 2b2fc72a472e524555cbc95ca5225f99a89f7d6c..66253a4a43b0f7652d290afd577098a338bc4cd3 100644 (file)
@@ -38,7 +38,7 @@ static int mc34708_write(struct udevice *dev, uint reg, const u8 *buff,
 
        ret = dm_i2c_write(dev, reg, buf, len);
        if (ret)
-               printf("write error to device: %p register: %#x!", dev, reg);
+               printf("write error to device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
@@ -53,7 +53,7 @@ static int mc34708_read(struct udevice *dev, uint reg, u8 *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buf, len);
        if (ret)
-               printf("read error from device: %p register: %#x!", dev, reg);
+               printf("read error from device: %p register: %#x!\n", dev, reg);
 
        buff[0] = buf[2];
        buff[1] = buf[1];
index 250a5d34bbf5ca366b8449f1b7ef7882f1f24fda..36be119b6cff84e030b4066d073984a03305ac11 100644 (file)
@@ -24,7 +24,7 @@ static int palmas_write(struct udevice *dev, uint reg, const uint8_t *buff,
                          int len)
 {
        if (dm_i2c_write(dev, reg, buff, len)) {
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -34,7 +34,7 @@ static int palmas_write(struct udevice *dev, uint reg, const uint8_t *buff,
 static int palmas_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 {
        if (dm_i2c_read(dev, reg, buff, len)) {
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -60,14 +60,14 @@ static int palmas_bind(struct udevice *dev)
        }
 
        if (!ofnode_valid(pmic_node)) {
-               debug("%s: %s pmic subnode not found!", __func__, dev->name);
+               debug("%s: %s pmic subnode not found!\n", __func__, dev->name);
                return -ENXIO;
        }
 
        regulators_node = ofnode_find_subnode(pmic_node, "regulators");
 
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s reg subnode not found!", __func__, dev->name);
+               debug("%s: %s reg subnode not found!\n", __func__, dev->name);
                return -ENXIO;
        }
 
index 8a5a8996b432879fcf3acb58611706bd2c693ed4..6cf5f35f0f2c6e1af15030075941d5811558bbf5 100644 (file)
@@ -31,7 +31,7 @@ static int pfuze100_write(struct udevice *dev, uint reg, const uint8_t *buff,
                          int len)
 {
        if (dm_i2c_write(dev, reg, buff, len)) {
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -41,7 +41,7 @@ static int pfuze100_write(struct udevice *dev, uint reg, const uint8_t *buff,
 static int pfuze100_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 {
        if (dm_i2c_read(dev, reg, buff, len)) {
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -55,7 +55,7 @@ static int pfuze100_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
+               debug("%s: %s regulators subnode not found!\n", __func__,
                      dev->name);
                return -ENXIO;
        }
index c60dfff5bfd693c9876fafe182a4b4f96d62e62e..25c339ab12cc5c2ba4f2f23e2ae62146365b11db 100644 (file)
@@ -29,7 +29,7 @@ static int rk8xx_write(struct udevice *dev, uint reg, const uint8_t *buff,
 
        ret = dm_i2c_write(dev, reg, buff, len);
        if (ret) {
-               debug("write error to device: %p register: %#x!", dev, reg);
+               debug("write error to device: %p register: %#x!\n", dev, reg);
                return ret;
        }
 
@@ -42,7 +42,7 @@ static int rk8xx_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buff, len);
        if (ret) {
-               debug("read error from device: %p register: %#x!", dev, reg);
+               debug("read error from device: %p register: %#x!\n", dev, reg);
                return ret;
        }
 
@@ -57,7 +57,7 @@ static int rk8xx_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
+               debug("%s: %s regulators subnode not found!\n", __func__,
                      dev->name);
                return -ENXIO;
        }
index c3be3fed4a05696df143be4437d6b6e6927364f4..f238396d368301365b0646e8dfae2faeb2060f79 100644 (file)
@@ -24,7 +24,7 @@ static int rn5t567_write(struct udevice *dev, uint reg, const uint8_t *buff,
 
        ret = dm_i2c_write(dev, reg, buff, len);
        if (ret) {
-               debug("write error to device: %p register: %#x!", dev, reg);
+               debug("write error to device: %p register: %#x!\n", dev, reg);
                return ret;
        }
 
@@ -37,7 +37,7 @@ static int rn5t567_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buff, len);
        if (ret) {
-               debug("read error from device: %p register: %#x!", dev, reg);
+               debug("read error from device: %p register: %#x!\n", dev, reg);
                return ret;
        }
 
index e45d4bc6e16ef8d6c344994eec0e265eee5268d4..f2aab6c457012f2387a753591216a4162033af6b 100644 (file)
@@ -30,7 +30,7 @@ static int s2mps11_write(struct udevice *dev, uint reg, const uint8_t *buff,
 
        ret = dm_i2c_write(dev, reg, buff, len);
        if (ret)
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
@@ -41,7 +41,7 @@ static int s2mps11_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buff, len);
        if (ret)
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
 
        return ret;
 }
@@ -53,8 +53,8 @@ static int s2mps11_probe(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "voltage-regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
-                                                            dev->name);
+               debug("%s: %s regulators subnode not found!\n", __func__,
+                     dev->name);
                return -ENXIO;
        }
 
index 54e44ce8645bb86bda57722d4caccb1b82a90145..b5ddd4930e3146d100c5bd26ba0df2e1aaeeb633 100644 (file)
@@ -27,7 +27,7 @@ static int s5m8767_write(struct udevice *dev, uint reg, const uint8_t *buff,
                          int len)
 {
        if (dm_i2c_write(dev, reg, buff, len)) {
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -37,7 +37,7 @@ static int s5m8767_write(struct udevice *dev, uint reg, const uint8_t *buff,
 static int s5m8767_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 {
        if (dm_i2c_read(dev, reg, buff, len)) {
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -56,7 +56,7 @@ static int s5m8767_bind(struct udevice *dev)
 
        node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
+               debug("%s: %s regulators subnode not found!\n", __func__,
                      dev->name);
                return -ENXIO;
        }
index 64e2f276dabdfa27f81a97ad694a4fb45d16c183..d7870915de8c3192c942ad0ea5c40a9e1feb5ebd 100644 (file)
@@ -28,7 +28,7 @@ static int sandbox_pmic_write(struct udevice *dev, uint reg,
                              const uint8_t *buff, int len)
 {
        if (dm_i2c_write(dev, reg, buff, len)) {
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -39,7 +39,7 @@ static int sandbox_pmic_read(struct udevice *dev, uint reg,
                             uint8_t *buff, int len)
 {
        if (dm_i2c_read(dev, reg, buff, len)) {
-               pr_err("read error from device: %p register: %#x!", dev, reg);
+               pr_err("read error from device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
index 82351b66137050ec0f836e67e415d293672f3f5b..47af0123328eed795d0d957f5e8d49e441ebd69f 100644 (file)
@@ -61,7 +61,7 @@ static int stpmu1_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(regulators_node)) {
-               dev_dbg(dev, "regulators subnode not found!");
+               dev_dbg(dev, "regulators subnode not found!\n");
                return -ENXIO;
        }
        dev_dbg(dev, "found regulators subnode\n");
index 1d3bf00b568673d3f8da1162279e2d8d9a160feb..5b1d19f3e0b70e6d57786cec2658c4825655853b 100644 (file)
@@ -26,7 +26,7 @@ static int tps65090_write(struct udevice *dev, uint reg, const uint8_t *buff,
                          int len)
 {
        if (dm_i2c_write(dev, reg, buff, len)) {
-               pr_err("write error to device: %p register: %#x!", dev, reg);
+               pr_err("write error to device: %p register: %#x!\n", dev, reg);
                return -EIO;
        }
 
@@ -39,8 +39,8 @@ static int tps65090_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
 
        ret = dm_i2c_read(dev, reg, buff, len);
        if (ret) {
-               pr_err("read error %d from device: %p register: %#x!", ret, dev,
-                     reg);
+               pr_err("read error %d from device: %p register: %#x!\n", ret,
+                      dev, reg);
                return -EIO;
        }
 
@@ -54,7 +54,7 @@ static int tps65090_bind(struct udevice *dev)
 
        regulators_node = dev_read_subnode(dev, "regulators");
        if (!ofnode_valid(regulators_node)) {
-               debug("%s: %s regulators subnode not found!", __func__,
+               debug("%s: %s regulators subnode not found!\n", __func__,
                      dev->name);
                return -ENXIO;
        }