Re: [PATCH 3/4] irqchip/alpine-msi: Add missing space after comma in GENMASK_ULL

[Radu Rendec]

On Wed, 2026-09-23 at 08:37 +0000, Eliav Farber wrote:
> Fix the coding-style issue reported by checkpatch:
> 
>   ERROR: space required after that ',' (ctx:VxV)
> 
> No functional change.
> 
> Signed-off-by: Eliav Farber <farbere@xxxxxxxxxx>
> ---
>  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 20f1f7f7612b..8155f63cfd29 100644
> --- a/drivers/irqchip/irq-alpine-msi.c
> +++ b/drivers/irqchip/irq-alpine-msi.c
> @@ -210,7 +210,7 @@ static int alpine_msix_init(struct device_node *node, struct device_node *parent
>  	 * To select the primary GIC as the target GIC, bits [18:17] must be set
>  	 * to 0x0. In this case, bit 16 (SPI_TARGET_CLUSTER0) must be set.
>  	 */
> -	priv->addr = res.start & GENMASK_ULL(63,20);
> +	priv->addr = res.start & GENMASK_ULL(63, 20);
>  	priv->addr |= ALPINE_MSIX_SPI_TARGET_CLUSTER0;
>  
>  	if (of_property_read_u32(node, "al,msi-base-spi", &priv->spi_first)) {

Reviewed-by: Radu Rendec <radu@xxxxxxxxxx>