[PATCH 3/7] i3c: mipi-i3c-hci: Quieten initialization messages
Frank Li
Frank.li at nxp.com
Tue Dec 9 08:25:41 PST 2025
On Tue, Dec 09, 2025 at 01:51:00PM +0200, Adrian Hunter wrote:
> The copious initialization messages are at most useful only for debugging.
> Change them from dev_info() or dev_notice() to dev_dbg().
>
> Signed-off-by: Adrian Hunter <adrian.hunter at intel.com>
> ---
If have chance, move this to debug fs.
Reviewed-by: Frank Li <Frank.Li at nxp.com>
> drivers/i3c/master/mipi-i3c-hci/core.c | 18 +++++-----
> drivers/i3c/master/mipi-i3c-hci/dma.c | 4 +--
> drivers/i3c/master/mipi-i3c-hci/ext_caps.c | 39 +++++++++++-----------
> drivers/i3c/master/mipi-i3c-hci/pio.c | 8 ++---
> 4 files changed, 34 insertions(+), 35 deletions(-)
>
> diff --git a/drivers/i3c/master/mipi-i3c-hci/core.c b/drivers/i3c/master/mipi-i3c-hci/core.c
> index 211321f73e02..07fb91a12593 100644
> --- a/drivers/i3c/master/mipi-i3c-hci/core.c
> +++ b/drivers/i3c/master/mipi-i3c-hci/core.c
> @@ -597,8 +597,8 @@ static int i3c_hci_init(struct i3c_hci *hci)
> hci->DAT_entry_size = FIELD_GET(DAT_ENTRY_SIZE, regval) ? 0 : 8;
> if (size_in_dwords)
> hci->DAT_entries = 4 * hci->DAT_entries / hci->DAT_entry_size;
> - dev_info(&hci->master.dev, "DAT: %u %u-bytes entries at offset %#x\n",
> - hci->DAT_entries, hci->DAT_entry_size, offset);
> + dev_dbg(&hci->master.dev, "DAT: %u %u-bytes entries at offset %#x\n",
> + hci->DAT_entries, hci->DAT_entry_size, offset);
>
> regval = reg_read(DCT_SECTION);
> offset = FIELD_GET(DCT_TABLE_OFFSET, regval);
> @@ -607,23 +607,23 @@ static int i3c_hci_init(struct i3c_hci *hci)
> hci->DCT_entry_size = FIELD_GET(DCT_ENTRY_SIZE, regval) ? 0 : 16;
> if (size_in_dwords)
> hci->DCT_entries = 4 * hci->DCT_entries / hci->DCT_entry_size;
> - dev_info(&hci->master.dev, "DCT: %u %u-bytes entries at offset %#x\n",
> - hci->DCT_entries, hci->DCT_entry_size, offset);
> + dev_dbg(&hci->master.dev, "DCT: %u %u-bytes entries at offset %#x\n",
> + hci->DCT_entries, hci->DCT_entry_size, offset);
>
> regval = reg_read(RING_HEADERS_SECTION);
> offset = FIELD_GET(RING_HEADERS_OFFSET, regval);
> hci->RHS_regs = offset ? hci->base_regs + offset : NULL;
> - dev_info(&hci->master.dev, "Ring Headers at offset %#x\n", offset);
> + dev_dbg(&hci->master.dev, "Ring Headers at offset %#x\n", offset);
>
> regval = reg_read(PIO_SECTION);
> offset = FIELD_GET(PIO_REGS_OFFSET, regval);
> hci->PIO_regs = offset ? hci->base_regs + offset : NULL;
> - dev_info(&hci->master.dev, "PIO section at offset %#x\n", offset);
> + dev_dbg(&hci->master.dev, "PIO section at offset %#x\n", offset);
>
> regval = reg_read(EXT_CAPS_SECTION);
> offset = FIELD_GET(EXT_CAPS_OFFSET, regval);
> hci->EXTCAPS_regs = offset ? hci->base_regs + offset : NULL;
> - dev_info(&hci->master.dev, "Extended Caps at offset %#x\n", offset);
> + dev_dbg(&hci->master.dev, "Extended Caps at offset %#x\n", offset);
>
> ret = i3c_hci_parse_ext_caps(hci);
> if (ret)
> @@ -705,7 +705,7 @@ static int i3c_hci_init(struct i3c_hci *hci)
> ret = -EIO;
> } else {
> hci->io = &mipi_i3c_hci_dma;
> - dev_info(&hci->master.dev, "Using DMA\n");
> + dev_dbg(&hci->master.dev, "Using DMA\n");
> }
> }
>
> @@ -717,7 +717,7 @@ static int i3c_hci_init(struct i3c_hci *hci)
> ret = -EIO;
> } else {
> hci->io = &mipi_i3c_hci_pio;
> - dev_info(&hci->master.dev, "Using PIO\n");
> + dev_dbg(&hci->master.dev, "Using PIO\n");
> }
> }
>
> diff --git a/drivers/i3c/master/mipi-i3c-hci/dma.c b/drivers/i3c/master/mipi-i3c-hci/dma.c
> index f20db2899989..0f6bbe184e85 100644
> --- a/drivers/i3c/master/mipi-i3c-hci/dma.c
> +++ b/drivers/i3c/master/mipi-i3c-hci/dma.c
> @@ -212,7 +212,7 @@ static int hci_dma_init(struct i3c_hci *hci)
>
> regval = rhs_reg_read(CONTROL);
> nr_rings = FIELD_GET(MAX_HEADER_COUNT_CAP, regval);
> - dev_info(&hci->master.dev, "%d DMA rings available\n", nr_rings);
> + dev_dbg(&hci->master.dev, "%d DMA rings available\n", nr_rings);
> if (unlikely(nr_rings > 8)) {
> dev_err(&hci->master.dev, "number of rings should be <= 8\n");
> nr_rings = 8;
> @@ -232,7 +232,7 @@ static int hci_dma_init(struct i3c_hci *hci)
> for (i = 0; i < rings->total; i++) {
> u32 offset = rhs_reg_read(RHn_OFFSET(i));
>
> - dev_info(&hci->master.dev, "Ring %d at offset %#x\n", i, offset);
> + dev_dbg(&hci->master.dev, "Ring %d at offset %#x\n", i, offset);
> ret = -EINVAL;
> if (!offset)
> goto err_out;
> diff --git a/drivers/i3c/master/mipi-i3c-hci/ext_caps.c b/drivers/i3c/master/mipi-i3c-hci/ext_caps.c
> index 024bccf23fd0..79c6b52df6e7 100644
> --- a/drivers/i3c/master/mipi-i3c-hci/ext_caps.c
> +++ b/drivers/i3c/master/mipi-i3c-hci/ext_caps.c
> @@ -26,9 +26,9 @@ static int hci_extcap_hardware_id(struct i3c_hci *hci, void __iomem *base)
> hci->vendor_version_id = readl(base + 0x08);
> hci->vendor_product_id = readl(base + 0x0c);
>
> - dev_info(&hci->master.dev, "vendor MIPI ID: %#x\n", hci->vendor_mipi_id);
> - dev_info(&hci->master.dev, "vendor version ID: %#x\n", hci->vendor_version_id);
> - dev_info(&hci->master.dev, "vendor product ID: %#x\n", hci->vendor_product_id);
> + dev_dbg(&hci->master.dev, "vendor MIPI ID: %#x\n", hci->vendor_mipi_id);
> + dev_dbg(&hci->master.dev, "vendor version ID: %#x\n", hci->vendor_version_id);
> + dev_dbg(&hci->master.dev, "vendor product ID: %#x\n", hci->vendor_product_id);
>
> /* ought to go in a table if this grows too much */
> switch (hci->vendor_mipi_id) {
> @@ -48,7 +48,7 @@ static int hci_extcap_master_config(struct i3c_hci *hci, void __iomem *base)
> static const char * const functionality[] = {
> "(unknown)", "master only", "target only",
> "primary/secondary master" };
> - dev_info(&hci->master.dev, "operation mode: %s\n", functionality[operation_mode]);
> + dev_dbg(&hci->master.dev, "operation mode: %s\n", functionality[operation_mode]);
> if (operation_mode & 0x1)
> return 0;
> dev_err(&hci->master.dev, "only master mode is currently supported\n");
> @@ -60,7 +60,7 @@ static int hci_extcap_multi_bus(struct i3c_hci *hci, void __iomem *base)
> u32 bus_instance = readl(base + 0x04);
> unsigned int count = FIELD_GET(GENMASK(3, 0), bus_instance);
>
> - dev_info(&hci->master.dev, "%d bus instances\n", count);
> + dev_dbg(&hci->master.dev, "%d bus instances\n", count);
> return 0;
> }
>
> @@ -70,7 +70,7 @@ static int hci_extcap_xfer_modes(struct i3c_hci *hci, void __iomem *base)
> u32 entries = FIELD_GET(CAP_HEADER_LENGTH, header) - 1;
> unsigned int index;
>
> - dev_info(&hci->master.dev, "transfer mode table has %d entries\n",
> + dev_dbg(&hci->master.dev, "transfer mode table has %d entries\n",
> entries);
> base += 4; /* skip header */
> for (index = 0; index < entries; index++) {
> @@ -94,7 +94,7 @@ static int hci_extcap_xfer_rates(struct i3c_hci *hci, void __iomem *base)
>
> base += 4; /* skip header */
>
> - dev_info(&hci->master.dev, "available data rates:\n");
> + dev_dbg(&hci->master.dev, "available data rates:\n");
> for (index = 0; index < entries; index++) {
> rate_entry = readl(base);
> dev_dbg(&hci->master.dev, "entry %d: 0x%08x",
> @@ -102,7 +102,7 @@ static int hci_extcap_xfer_rates(struct i3c_hci *hci, void __iomem *base)
> rate = FIELD_GET(XFERRATE_ACTUAL_RATE_KHZ, rate_entry);
> rate_id = FIELD_GET(XFERRATE_RATE_ID, rate_entry);
> mode_id = FIELD_GET(XFERRATE_MODE_ID, rate_entry);
> - dev_info(&hci->master.dev, "rate %d for %s = %d kHz\n",
> + dev_dbg(&hci->master.dev, "rate %d for %s = %d kHz\n",
> rate_id,
> mode_id == XFERRATE_MODE_I3C ? "I3C" :
> mode_id == XFERRATE_MODE_I2C ? "I2C" :
> @@ -121,7 +121,7 @@ static int hci_extcap_auto_command(struct i3c_hci *hci, void __iomem *base)
> u32 autocmd_ext_config = readl(base + 0x08);
> unsigned int count = FIELD_GET(GENMASK(3, 0), autocmd_ext_config);
>
> - dev_info(&hci->master.dev, "%d/%d active auto-command entries\n",
> + dev_dbg(&hci->master.dev, "%d/%d active auto-command entries\n",
> count, max_count);
> /* remember auto-command register location for later use */
> hci->AUTOCMD_regs = base;
> @@ -130,46 +130,46 @@ static int hci_extcap_auto_command(struct i3c_hci *hci, void __iomem *base)
>
> static int hci_extcap_debug(struct i3c_hci *hci, void __iomem *base)
> {
> - dev_info(&hci->master.dev, "debug registers present\n");
> + dev_dbg(&hci->master.dev, "debug registers present\n");
> hci->DEBUG_regs = base;
> return 0;
> }
>
> static int hci_extcap_scheduled_cmd(struct i3c_hci *hci, void __iomem *base)
> {
> - dev_info(&hci->master.dev, "scheduled commands available\n");
> + dev_dbg(&hci->master.dev, "scheduled commands available\n");
> /* hci->schedcmd_regs = base; */
> return 0;
> }
>
> static int hci_extcap_non_curr_master(struct i3c_hci *hci, void __iomem *base)
> {
> - dev_info(&hci->master.dev, "Non-Current Master support available\n");
> + dev_dbg(&hci->master.dev, "Non-Current Master support available\n");
> /* hci->NCM_regs = base; */
> return 0;
> }
>
> static int hci_extcap_ccc_resp_conf(struct i3c_hci *hci, void __iomem *base)
> {
> - dev_info(&hci->master.dev, "CCC Response Configuration available\n");
> + dev_dbg(&hci->master.dev, "CCC Response Configuration available\n");
> return 0;
> }
>
> static int hci_extcap_global_DAT(struct i3c_hci *hci, void __iomem *base)
> {
> - dev_info(&hci->master.dev, "Global DAT available\n");
> + dev_dbg(&hci->master.dev, "Global DAT available\n");
> return 0;
> }
>
> static int hci_extcap_multilane(struct i3c_hci *hci, void __iomem *base)
> {
> - dev_info(&hci->master.dev, "Master Multi-Lane support available\n");
> + dev_dbg(&hci->master.dev, "Master Multi-Lane support available\n");
> return 0;
> }
>
> static int hci_extcap_ncm_multilane(struct i3c_hci *hci, void __iomem *base)
> {
> - dev_info(&hci->master.dev, "NCM Multi-Lane support available\n");
> + dev_dbg(&hci->master.dev, "NCM Multi-Lane support available\n");
> return 0;
> }
>
> @@ -202,7 +202,7 @@ static const struct hci_ext_caps ext_capabilities[] = {
> static int hci_extcap_vendor_NXP(struct i3c_hci *hci, void __iomem *base)
> {
> hci->vendor_data = (__force void *)base;
> - dev_info(&hci->master.dev, "Build Date Info = %#x\n", readl(base + 1*4));
> + dev_dbg(&hci->master.dev, "Build Date Info = %#x\n", readl(base + 1*4));
> /* reset the FPGA */
> writel(0xdeadbeef, base + 1*4);
> return 0;
> @@ -240,7 +240,7 @@ static int hci_extcap_vendor_specific(struct i3c_hci *hci, void __iomem *base,
> }
>
> if (!vendor_cap_entry) {
> - dev_notice(&hci->master.dev,
> + dev_dbg(&hci->master.dev,
> "unknown ext_cap 0x%02x for vendor 0x%02x\n",
> cap_id, hci->vendor_mipi_id);
> return 0;
> @@ -295,8 +295,7 @@ int i3c_hci_parse_ext_caps(struct i3c_hci *hci)
> }
> }
> if (!cap_entry) {
> - dev_notice(&hci->master.dev,
> - "unknown ext_cap 0x%02x\n", cap_id);
> + dev_dbg(&hci->master.dev, "unknown ext_cap 0x%02x\n", cap_id);
> } else if (cap_length < cap_entry->min_length) {
> dev_err(&hci->master.dev,
> "ext_cap 0x%02x has size %d (expecting >= %d)\n",
> diff --git a/drivers/i3c/master/mipi-i3c-hci/pio.c b/drivers/i3c/master/mipi-i3c-hci/pio.c
> index 142f3f79415b..5020c84c323f 100644
> --- a/drivers/i3c/master/mipi-i3c-hci/pio.c
> +++ b/drivers/i3c/master/mipi-i3c-hci/pio.c
> @@ -148,13 +148,13 @@ static int hci_pio_init(struct i3c_hci *hci)
> spin_lock_init(&pio->lock);
>
> size_val = pio_reg_read(QUEUE_SIZE);
> - dev_info(&hci->master.dev, "CMD/RESP FIFO = %ld entries\n",
> + dev_dbg(&hci->master.dev, "CMD/RESP FIFO = %ld entries\n",
> FIELD_GET(CR_QUEUE_SIZE, size_val));
> - dev_info(&hci->master.dev, "IBI FIFO = %ld bytes\n",
> + dev_dbg(&hci->master.dev, "IBI FIFO = %ld bytes\n",
> 4 * FIELD_GET(IBI_STATUS_SIZE, size_val));
> - dev_info(&hci->master.dev, "RX data FIFO = %d bytes\n",
> + dev_dbg(&hci->master.dev, "RX data FIFO = %d bytes\n",
> 4 * (2 << FIELD_GET(RX_DATA_BUFFER_SIZE, size_val)));
> - dev_info(&hci->master.dev, "TX data FIFO = %d bytes\n",
> + dev_dbg(&hci->master.dev, "TX data FIFO = %d bytes\n",
> 4 * (2 << FIELD_GET(TX_DATA_BUFFER_SIZE, size_val)));
>
> /*
> --
> 2.51.0
>
>
> --
> linux-i3c mailing list
> linux-i3c at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-i3c
More information about the linux-i3c
mailing list