diff options
| author | Nicolas Boichat <drinkcat@chromium.org> | 2019-06-17 13:08:06 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2019-06-20 05:52:11 +0000 |
| commit | b16b36b94989a0d8e0f2268099f91094668c70f9 (patch) | |
| tree | ce9f912e6dbb0ed05cfb958f21e1ff67a7898292 /driver | |
| parent | bbe3abd8dbee0a91b5bfad214821768610b10371 (diff) | |
| download | chrome-ec-b16b36b94989a0d8e0f2268099f91094668c70f9.tar.gz | |
ec: Remove extraneous new line as the end of CPRINTS strings
CPRINTS already prints a new line, no need to add another one.
Spotted during boot on kukui, and then realized there are many
more instances:
""
[3.689239 Module 7 is not supported for clock disable
]
""
BRANCH=none
BUG=none
TEST=make buildall -j
TEST=`git grep CPRINTS | grep "\\\\n\""` shows nothing of
interest.
Change-Id: I4d2bbbc65a91fa56c6e6115aa5c353bfd2b384a1
Signed-off-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1660519
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'driver')
| -rw-r--r-- | driver/accelgyro_lsm6dso.c | 4 | ||||
| -rw-r--r-- | driver/led/max695x.c | 2 | ||||
| -rw-r--r-- | driver/temp_sensor/bd99992gw.c | 6 |
3 files changed, 6 insertions, 6 deletions
diff --git a/driver/accelgyro_lsm6dso.c b/driver/accelgyro_lsm6dso.c index d50e24aa17..9517712138 100644 --- a/driver/accelgyro_lsm6dso.c +++ b/driver/accelgyro_lsm6dso.c @@ -251,7 +251,7 @@ static int irq_handler(struct motion_sensor_t *s, uint32_t *event) return ret; if (fsts.len & (LSM6DSO_FIFO_DATA_OVR | LSM6DSO_FIFO_FULL)) { - CPRINTS("%s FIFO Overrun: %04x\n", s->name, fsts.len); + CPRINTS("%s FIFO Overrun: %04x", s->name, fsts.len); } if (fsts.len & LSM6DSO_FIFO_DIFF_MASK) @@ -480,7 +480,7 @@ static int init(const struct motion_sensor_t *s) err_unlock: mutex_unlock(s->mutex); - CPRINTS("%s: MS Init type:0x%X Error\n", s->name, s->type); + CPRINTS("%s: MS Init type:0x%X Error", s->name, s->type); return ret; } diff --git a/driver/led/max695x.c b/driver/led/max695x.c index a563edaf26..6f0e1b8e84 100644 --- a/driver/led/max695x.c +++ b/driver/led/max695x.c @@ -68,7 +68,7 @@ int display_7seg_write(enum seven_seg_module_display module, uint16_t data) buf[1] = 0x00; break; default: - CPRINTS("Unknown Module\n"); + CPRINTS("Unknown Module"); return EC_ERROR_UNKNOWN; } /* Segment - 1 */ diff --git a/driver/temp_sensor/bd99992gw.c b/driver/temp_sensor/bd99992gw.c index f2a48eb2d5..291ce32479 100644 --- a/driver/temp_sensor/bd99992gw.c +++ b/driver/temp_sensor/bd99992gw.c @@ -41,7 +41,7 @@ static int raw_read8(const int offset, int *data_ptr) int ret; ret = i2c_read8(I2C_PORT_THERMAL, BD99992GW_I2C_ADDR, offset, data_ptr); if (ret != EC_SUCCESS) - CPRINTS("bd99992gw read fail %d\n", ret); + CPRINTS("bd99992gw read fail %d", ret); return ret; } @@ -50,7 +50,7 @@ static int raw_write8(const int offset, int data) int ret; ret = i2c_write8(I2C_PORT_THERMAL, BD99992GW_I2C_ADDR, offset, data); if (ret != EC_SUCCESS) - CPRINTS("bd99992gw write fail %d\n", ret); + CPRINTS("bd99992gw write fail %d", ret); return ret; } @@ -136,7 +136,7 @@ int bd99992gw_get_val(int idx, int *temp_ptr) /* Make sure we found it */ if (i == ARRAY_SIZE(active_channels) || active_channels[i] != idx) { - CPRINTS("Bad ADC channel %d\n", idx); + CPRINTS("Bad ADC channel %d", idx); return EC_ERROR_INVAL; } |