Re: [PATCH v2 4/5] irqchip/alpine-msi: Fix format and wording of the registration debug print

From: Radu Rendec

Date: Wed Nov 04 2026 - 07:01:27 EST


On Sat, 2026-10-24 at 19:36 +0000, Eliav Farber wrote:
> The "Registering" debug message in alpine_msix_init() prints
> priv->num_spis and priv->spi_first, both u32, with %d. Use %u to match
> their type, and fix the "msixs" wording while at it. No functional
> change.
>
> Signed-off-by: Eliav Farber <farbere@xxxxxxxxxx>
> ---
> v2: New patch (Radu).
>
>  drivers/irqchip/irq-alpine-msi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/irqchip/irq-alpine-msi.c b/drivers/irqchip/irq-alpine-msi.c
> index 8155f63cfd29..fb84b6046193 100644
> --- a/drivers/irqchip/irq-alpine-msi.c
> +++ b/drivers/irqchip/irq-alpine-msi.c
> @@ -229,7 +229,7 @@ static int alpine_msix_init(struct device_node *node, struct device_node *parent
>   return -ENOMEM;
>   priv->msi_map = msi_map;
>  
> - pr_debug("Registering %d msixs, starting at %d\n", priv->num_spis, priv->spi_first);
> + pr_debug("Registering %u MSIs, starting at SPI %u\n", priv->num_spis, priv->spi_first);
>  
>   ret = alpine_msix_init_domains(priv, node);
>   if (ret)

Reviewed-by: Radu Rendec <radu@xxxxxxxxxx>