Re: [PATCH] irqchip/irq-realtek-rtl: change to __raw reads and writes

[Thomas Gleixner <tglx@kernel.org>]

On Sat, Jul 11 2026 at 13:23, Rustam Adilov wrote:

> To make the interrupt driver operable with SWAP_IO_SPACE
> config enabled, replace all instances of readl/writel with
> their __raw variants. Otherwise readl/writel will do a byte
> swap which this driver does not intend to do.

Sorry, but this word salad does not qualify as a change log. See

  https://docs.kernel.org/process/maintainer-tip.html#changelog

First you want to explain what the context:

  When CONFIG_SWAP_IO_SPACE is enabled readl() and writel() imply a byte
  swap.

Then you want to explain why this is not correct for this
driver/hardware. Just saying 'does not intend to do' contains zero
information:

  This is incorrect for the RTL driver, because <Insert proper technical explanation>

Then you tell how you cure it:

  Fix this by converting the MMIO accesses to __raw_readl() and
  __raw_writel(), which do not byte swap the data.

Thanks,

        tglx