Re: [PATCH spi-next 04/11] spi: spi-fsl-lpspi: fsl_lpspi_set_watermark(): use FIELD_PREP() to encode FIFO Status register

From: Mark Brown

Date: Mon Mar 16 2026 - 10:41:54 EST


On Mon, Mar 16, 2026 at 09:39:05AM +0100, Marc Kleine-Budde wrote:

> +#define FCR_RXWATER GENMASK(18, 16)
> +#define FCR_TXWATER GENMASK(2, 0)

The source of the watermark values appears to be

temp = readl(fsl_lpspi->base + IMX7ULP_PARAM);
fsl_lpspi->txfifosize = 1 << (temp & 0x0f);
fsl_lpspi->rxfifosize = 1 << ((temp >> 8) & 0x0f);

which suggests 4 bits of watermark not 3? Or at least that something
isn't joined up somewhere.

Attachment: signature.asc
Description: PGP signature