Skip to content

Commit 5320207

Browse files
committed
soundwire: cadence_master: make frame index trace more readable
The existing code uses i as the frame index which is 0 based. And "bad frame %d/%d", i, num_frames where num_frames is 1 based. Change the frame index to i + 1 to align with num_frames and more readable. Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
1 parent e1e5f5d commit 5320207

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

drivers/soundwire/cadence_master.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2499,14 +2499,14 @@ int sdw_cdns_check_write_response(struct device *dev, u8 *dma_buffer,
24992499
ret = check_frame_start(header, counter);
25002500
if (ret < 0) {
25012501
dev_err(dev, "%s: bad frame %d/%d start header %x\n",
2502-
__func__, i, num_frames, header);
2502+
__func__, i + 1, num_frames, header);
25032503
return ret;
25042504
}
25052505

25062506
ret = check_frame_end(footer);
25072507
if (ret < 0) {
25082508
dev_err(dev, "%s: bad frame %d/%d end footer %x\n",
2509-
__func__, i, num_frames, footer);
2509+
__func__, i + 1, num_frames, footer);
25102510
return ret;
25112511
}
25122512

@@ -2577,7 +2577,7 @@ int sdw_cdns_check_read_response(struct device *dev, u8 *dma_buffer, int dma_buf
25772577
ret = check_frame_start(header, counter);
25782578
if (ret < 0) {
25792579
dev_err(dev, "%s: bad frame %d/%d start header %x\n",
2580-
__func__, i, num_frames, header);
2580+
__func__, i + 1, num_frames, header);
25812581
return ret;
25822582
}
25832583

@@ -2592,7 +2592,7 @@ int sdw_cdns_check_read_response(struct device *dev, u8 *dma_buffer, int dma_buf
25922592

25932593
if (crc != expected_crc) {
25942594
dev_err(dev, "%s: bad frame %d/%d crc %#x expected %#x\n",
2595-
__func__, i, num_frames, crc, expected_crc);
2595+
__func__, i + 1, num_frames, crc, expected_crc);
25962596
return -EIO;
25972597
}
25982598

@@ -2603,7 +2603,7 @@ int sdw_cdns_check_read_response(struct device *dev, u8 *dma_buffer, int dma_buf
26032603
ret = check_frame_end(footer);
26042604
if (ret < 0) {
26052605
dev_err(dev, "%s: bad frame %d/%d end footer %x\n",
2606-
__func__, i, num_frames, footer);
2606+
__func__, i + 1, num_frames, footer);
26072607
return ret;
26082608
}
26092609

0 commit comments

Comments
 (0)