Re: [PATCH 2/2] spi: dw: use DW_SPI_INT_MASK instead of hardcoded 0xff
From: Mark Brown
Date: Tue Aug 18 2026 - 13:07:35 EST
On Mon, Aug 17, 2026 at 11:14:04PM +0800, Jisheng Zhang wrote:
> The Interrupt Mask Register valid bits is bit[5:0] which is well
> defined with DW_SPI_INT_MASK, use it instead of the incorrect(but no
> harm) and hardcoded 0xff.
> dw_reader(dws);
> if (!dws->rx_len) {
> - dw_spi_mask_intr(dws, 0xff);
> + dw_spi_mask_intr(dws, DW_SPI_INT_MASK);
As I'm reasonably sure I said before this looks fairly obviously like
defensive programming intended to handle future versions of the IP that
define more mask bits sensibly.
Attachment:
signature.asc
Description: PGP signature