mirrors/linux-stable
1
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git synced 2024-10-06 00:39:48 +00:00
Code Issues Releases Wiki Activity

ptp: ptp_clockmatrix: Clean-up dev_*() messages.

Browse source 
Code clean-up.

* Remove unnecessary \n termination from dev_*() messages.
* Remove 'char *fmt' to define strings to stay within 80 column
  limit.  Not needed since coding guidelines increased to
  100 columns limit.
  Keeping format in place allows static code checkers to
  validate the arguments.
* Tighten up vertical spacing.

Signed-off-by: Vincent Cheng <vincent.cheng.xh@renesas.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Vincent Cheng 2021-02-17 00:42:15 -05:00 committed by David S. Miller
parent 10c270cf25
commit 1c49d3e947
1 changed files with 43 additions and 79 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

122
drivers/ptp/ptp_clockmatrix.c
View file

@ -160,7 +160,6 @@ static int idtcm_xfer_read(struct idtcm *idtcm,
struct i2c_client *client = idtcm->client; struct i2c_client *client = idtcm->client;
struct i2c_msg msg[2]; struct i2c_msg msg[2];
int cnt; int cnt;
char *fmt = "i2c_transfer failed at %d in %s, at addr: %04X!\n";
msg[0].addr = client->addr; msg[0].addr = client->addr;
msg[0].flags = 0; msg[0].flags = 0;
@ -176,14 +175,12 @@ static int idtcm_xfer_read(struct idtcm *idtcm,
if (cnt < 0) { if (cnt < 0) {
dev_err(&client->dev, dev_err(&client->dev,
fmt, "i2c_transfer failed at %d in %s, at addr: %04x!",
__LINE__, __LINE__, __func__, regaddr);
__func__,
regaddr);
return cnt; return cnt;
} else if (cnt != 2) { } else if (cnt != 2) {
dev_err(&client->dev, dev_err(&client->dev,
"i2c_transfer sent only %d of %d messages\n", cnt, 2); "i2c_transfer sent only %d of %d messages", cnt, 2);
return -EIO; return -EIO;
} }
@ -199,7 +196,6 @@ static int idtcm_xfer_write(struct idtcm *idtcm,
/* we add 1 byte for device register */ /* we add 1 byte for device register */
u8 msg[IDTCM_MAX_WRITE_COUNT + 1]; u8 msg[IDTCM_MAX_WRITE_COUNT + 1];
int cnt; int cnt;
char *fmt = "i2c_master_send failed at %d in %s, at addr: %04X!\n";
if (count > IDTCM_MAX_WRITE_COUNT) if (count > IDTCM_MAX_WRITE_COUNT)
return -EINVAL; return -EINVAL;
@ -211,10 +207,8 @@ static int idtcm_xfer_write(struct idtcm *idtcm,
if (cnt < 0) { if (cnt < 0) {
dev_err(&client->dev, dev_err(&client->dev,
fmt, "i2c_master_send failed at %d in %s, at addr: %04x!",
__LINE__, __LINE__, __func__, regaddr);
__func__,
regaddr);
return cnt; return cnt;
} }
@ -238,7 +232,7 @@ static int idtcm_page_offset(struct idtcm *idtcm, u8 val)
if (err) { if (err) {
idtcm->page_offset = 0xff; idtcm->page_offset = 0xff;
dev_err(&idtcm->client->dev, "failed to set page offset\n"); dev_err(&idtcm->client->dev, "failed to set page offset");
} else { } else {
idtcm->page_offset = val; idtcm->page_offset = val;
} }
@ -330,7 +324,7 @@ static int wait_for_boot_status_ready(struct idtcm *idtcm)
} while (i); } while (i);
dev_warn(&idtcm->client->dev, "%s timed out\n", __func__); dev_warn(&idtcm->client->dev, "%s timed out", __func__);
return -EBUSY; return -EBUSY;
} }
@ -811,7 +805,7 @@ static int _idtcm_set_dpll_scsr_tod(struct idtcm_channel *channel,
if (++count > 20) { if (++count > 20) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Timed out waiting for the write counter\n"); "Timed out waiting for the write counter");
return -EIO; return -EIO;
} }
} }
@ -877,7 +871,7 @@ static int _idtcm_settime_deprecated(struct idtcm_channel *channel,
if (err) { if (err) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"%s: Set HW ToD failed\n", __func__); "%s: Set HW ToD failed", __func__);
return err; return err;
} }
@ -1079,14 +1073,14 @@ static int idtcm_state_machine_reset(struct idtcm *idtcm)
if (status == 0xA0) { if (status == 0xA0) {
dev_dbg(&idtcm->client->dev, dev_dbg(&idtcm->client->dev,
"SM_RESET completed in %d ms\n", "SM_RESET completed in %d ms", i * 100);
i * 100);
break; break;
} }
} }
if (!status) if (!status)
dev_err(&idtcm->client->dev, "Timed out waiting for CM_RESET to complete\n"); dev_err(&idtcm->client->dev,
"Timed out waiting for CM_RESET to complete");
} }
return err; return err;
@ -1182,12 +1176,12 @@ static int set_pll_output_mask(struct idtcm *idtcm, u16 addr, u8 val)
static int set_tod_ptp_pll(struct idtcm *idtcm, u8 index, u8 pll) static int set_tod_ptp_pll(struct idtcm *idtcm, u8 index, u8 pll)
{ {
if (index >= MAX_TOD) { if (index >= MAX_TOD) {
dev_err(&idtcm->client->dev, "ToD%d not supported\n", index); dev_err(&idtcm->client->dev, "ToD%d not supported", index);
return -EINVAL; return -EINVAL;
} }
if (pll >= MAX_PLL) { if (pll >= MAX_PLL) {
dev_err(&idtcm->client->dev, "Pll%d not supported\n", pll); dev_err(&idtcm->client->dev, "Pll%d not supported", pll);
return -EINVAL; return -EINVAL;
} }
@ -1205,8 +1199,7 @@ static int check_and_set_masks(struct idtcm *idtcm,
switch (regaddr) { switch (regaddr) {
case TOD_MASK_ADDR: case TOD_MASK_ADDR:
if ((val & 0xf0) || !(val & 0x0f)) { if ((val & 0xf0) || !(val & 0x0f)) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev, "Invalid TOD mask 0x%02x", val);
"Invalid TOD mask 0x%hhx\n", val);
err = -EINVAL; err = -EINVAL;
} else { } else {
idtcm->tod_mask = val; idtcm->tod_mask = val;
@ -1237,14 +1230,14 @@ static void display_pll_and_masks(struct idtcm *idtcm)
u8 i; u8 i;
u8 mask; u8 mask;
dev_dbg(&idtcm->client->dev, "tod_mask = 0x%02x\n", idtcm->tod_mask); dev_dbg(&idtcm->client->dev, "tod_mask = 0x%02x", idtcm->tod_mask);
for (i = 0; i < MAX_TOD; i++) { for (i = 0; i < MAX_TOD; i++) {
mask = 1 << i; mask = 1 << i;
if (mask & idtcm->tod_mask) if (mask & idtcm->tod_mask)
dev_dbg(&idtcm->client->dev, dev_dbg(&idtcm->client->dev,
"TOD%d pll = %d output_mask = 0x%04x\n", "TOD%d pll = %d output_mask = 0x%04x",
i, idtcm->channel[i].pll, i, idtcm->channel[i].pll,
idtcm->channel[i].output_mask); idtcm->channel[i].output_mask);
} }
@ -1265,19 +1258,16 @@ static int idtcm_load_firmware(struct idtcm *idtcm,
if (firmware) /* module parameter */ if (firmware) /* module parameter */
snprintf(fname, sizeof(fname), "%s", firmware); snprintf(fname, sizeof(fname), "%s", firmware);
dev_dbg(&idtcm->client->dev, "requesting firmware '%s'\n", fname); dev_dbg(&idtcm->client->dev, "requesting firmware '%s'", fname);
err = request_firmware(&fw, fname, dev); err = request_firmware(&fw, fname, dev);
if (err) { if (err) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
return err; return err;
} }
dev_dbg(&idtcm->client->dev, "firmware size %zu bytes\n", fw->size); dev_dbg(&idtcm->client->dev, "firmware size %zu bytes", fw->size);
rec = (struct idtcm_fwrc *) fw->data; rec = (struct idtcm_fwrc *) fw->data;
@ -1288,7 +1278,7 @@ static int idtcm_load_firmware(struct idtcm *idtcm,
if (rec->reserved) { if (rec->reserved) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"bad firmware, reserved field non-zero\n"); "bad firmware, reserved field non-zero");
err = -EINVAL; err = -EINVAL;
} else { } else {
regaddr = rec->hiaddr << 8; regaddr = rec->hiaddr << 8;
@ -1559,10 +1549,8 @@ static int idtcm_gettime(struct ptp_clock_info *ptp, struct timespec64 *ts)
err = _idtcm_gettime(channel, ts); err = _idtcm_gettime(channel, ts);
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev, "Failed at line %d in %s!",
"Failed at line %d in func %s!\n", __LINE__, __func__);
__LINE__,
__func__);
mutex_unlock(&idtcm->reg_lock); mutex_unlock(&idtcm->reg_lock);
@ -1583,9 +1571,7 @@ static int idtcm_settime_deprecated(struct ptp_clock_info *ptp,
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
mutex_unlock(&idtcm->reg_lock); mutex_unlock(&idtcm->reg_lock);
@ -1606,9 +1592,7 @@ static int idtcm_settime(struct ptp_clock_info *ptp,
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
mutex_unlock(&idtcm->reg_lock); mutex_unlock(&idtcm->reg_lock);
@ -1628,9 +1612,7 @@ static int idtcm_adjtime_deprecated(struct ptp_clock_info *ptp, s64 delta)
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
mutex_unlock(&idtcm->reg_lock); mutex_unlock(&idtcm->reg_lock);
@ -1650,9 +1632,7 @@ static int idtcm_adjtime(struct ptp_clock_info *ptp, s64 delta)
err = idtcm_do_phase_pull_in(channel, delta, 0); err = idtcm_do_phase_pull_in(channel, delta, 0);
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
return err; return err;
} }
@ -1670,9 +1650,7 @@ static int idtcm_adjtime(struct ptp_clock_info *ptp, s64 delta)
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
mutex_unlock(&idtcm->reg_lock); mutex_unlock(&idtcm->reg_lock);
@ -1694,9 +1672,7 @@ static int idtcm_adjphase(struct ptp_clock_info *ptp, s32 delta)
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
mutex_unlock(&idtcm->reg_lock); mutex_unlock(&idtcm->reg_lock);
@ -1718,9 +1694,7 @@ static int idtcm_adjfine(struct ptp_clock_info *ptp, long scaled_ppm)
if (err) if (err)
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
mutex_unlock(&idtcm->reg_lock); mutex_unlock(&idtcm->reg_lock);
@ -1741,9 +1715,8 @@ static int idtcm_enable(struct ptp_clock_info *ptp,
err = idtcm_perout_enable(channel, false, &rq->perout); err = idtcm_perout_enable(channel, false, &rq->perout);
if (err) if (err)
dev_err(&channel->idtcm->client->dev, dev_err(&channel->idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!",
__LINE__, __LINE__, __func__);
__func__);
return err; return err;
} }
@ -1755,9 +1728,7 @@ static int idtcm_enable(struct ptp_clock_info *ptp,
err = idtcm_perout_enable(channel, true, &rq->perout); err = idtcm_perout_enable(channel, true, &rq->perout);
if (err) if (err)
dev_err(&channel->idtcm->client->dev, dev_err(&channel->idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
return err; return err;
default: default:
break; break;
@ -2025,7 +1996,6 @@ static void idtcm_set_version_info(struct idtcm *idtcm)
u16 product_id; u16 product_id;
u8 hw_rev_id; u8 hw_rev_id;
u8 config_select; u8 config_select;
char *fmt = "%d.%d.%d, Id: 0x%04x HW Rev: %d OTP Config Select: %d\n";
idtcm_read_major_release(idtcm, &major); idtcm_read_major_release(idtcm, &major);
idtcm_read_minor_release(idtcm, &minor); idtcm_read_minor_release(idtcm, &minor);
@ -2044,7 +2014,9 @@ static void idtcm_set_version_info(struct idtcm *idtcm)
else else
idtcm->deprecated = 1; idtcm->deprecated = 1;
dev_info(&idtcm->client->dev, fmt, major, minor, hotfix, dev_info(&idtcm->client->dev,
"%d.%d.%d, Id: 0x%04x HW Rev: %d OTP Config Select: %d",
major, minor, hotfix,
product_id, hw_rev_id, config_select); product_id, hw_rev_id, config_select);
} }
@ -2203,9 +2175,7 @@ static int idtcm_enable_channel(struct idtcm *idtcm, u32 index)
err = idtcm_enable_tod_sync(channel); err = idtcm_enable_tod_sync(channel);
if (err) { if (err) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
return err; return err;
} }
} }
@ -2214,16 +2184,14 @@ static int idtcm_enable_channel(struct idtcm *idtcm, u32 index)
err = idtcm_get_pll_mode(channel, &channel->pll_mode); err = idtcm_get_pll_mode(channel, &channel->pll_mode);
if (err) { if (err) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Error: %s - Unable to read pll mode\n", __func__); "Error: %s - Unable to read pll mode", __func__);
return err; return err;
} }
err = idtcm_enable_tod(channel); err = idtcm_enable_tod(channel);
if (err) { if (err) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"Failed at line %d in func %s!\n", "Failed at line %d in %s!", __LINE__, __func__);
__LINE__,
__func__);
return err; return err;
} }
@ -2238,7 +2206,7 @@ static int idtcm_enable_channel(struct idtcm *idtcm, u32 index)
if (!channel->ptp_clock) if (!channel->ptp_clock)
return -ENOTSUPP; return -ENOTSUPP;
dev_info(&idtcm->client->dev, "PLL%d registered as ptp%d\n", dev_info(&idtcm->client->dev, "PLL%d registered as ptp%d",
index, channel->ptp_clock->index); index, channel->ptp_clock->index);
return 0; return 0;
@ -2279,7 +2247,6 @@ static int idtcm_probe(struct i2c_client *client,
struct idtcm *idtcm; struct idtcm *idtcm;
int err; int err;
u8 i; u8 i;
char *fmt = "Failed at %d in line %s with channel output %d!\n";
/* Unused for now */ /* Unused for now */
(void)id; (void)id;
@ -2304,7 +2271,7 @@ static int idtcm_probe(struct i2c_client *client,
if (err) if (err)
dev_warn(&idtcm->client->dev, dev_warn(&idtcm->client->dev,
"loading firmware failed with %d\n", err); "loading firmware failed with %d", err);
wait_for_chip_ready(idtcm); wait_for_chip_ready(idtcm);
@ -2314,17 +2281,14 @@ static int idtcm_probe(struct i2c_client *client,
err = idtcm_enable_channel(idtcm, i); err = idtcm_enable_channel(idtcm, i);
if (err) { if (err) {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
fmt, "idtcm_enable_channel %d failed!", i);
__LINE__,
__func__,
i);
break; break;
} }
} }
} }
} else { } else {
dev_err(&idtcm->client->dev, dev_err(&idtcm->client->dev,
"no PLLs flagged as PHCs, nothing to do\n"); "no PLLs flagged as PHCs, nothing to do");
err = -ENODEV; err = -ENODEV;
} }