[PATCH] spi: spi-mem: Introduce a default ->exec_op() debug log
Tudor Ambarus
tudor.ambarus at linaro.org
Thu Mar 6 01:05:39 PST 2025
On 3/5/25 8:11 PM, Miquel Raynal wrote:
> --- a/drivers/spi/spi-mem.c
> +++ b/drivers/spi/spi-mem.c
> @@ -377,6 +377,17 @@ int spi_mem_exec_op(struct spi_mem *mem, const struct spi_mem_op *op)
> /* Make sure the operation frequency is correct before going futher */
> spi_mem_adjust_op_freq(mem, (struct spi_mem_op *)op);
>
> + dev_dbg(&mem->spi->dev, "[cmd: 0x%02x][%dB addr: %#8llx][%dB dummy][%4dB data %s] %d%c-%d%c-%d%c-%d%c @ %uHz\n",
Isn't this too "chatty", especially on page program ops? I wouldn't be
surprised if the prints introduce timings that change controller's
behavior. How about using dev_vdbg?
More information about the linux-mtd
mailing list