Hi all,

On Thu, 2 Nov 2017 16:20:38 +1100 Stephen Rothwell <s...@canb.auug.org.au> 
wrote:
>
> After merging the gpio tree, today's linux-next build (arm
> multi_v7_defconfig) failed like this:
> 
> drivers/gpio/gpio-uniphier.c:324:14: error: initialization from incompatible 
> pointer type [-Werror=incompatible-pointer-types]
>   .activate = uniphier_gpio_irq_domain_activate,
>               ^
> 
> Caused by commit
> 
>   dbe776c2ca54 ("gpio: uniphier: add UniPhier GPIO controller driver")
> 
> interacting with commit
> 
>   72491643469a ("genirq/irqdomain: Update irq_domain_ops.activate() 
> signature")
> 
> from the tip tree
> 
> I added the following merge fix patch:
> 
> From: Stephen Rothwell <s...@canb.auug.org.au>
> Date: Thu, 2 Nov 2017 16:10:46 +1100
> Subject: [PATCH] gpio: uniphier: merge fix for "Update
>  irq_domain_ops.activate() signature"
> 
> Signed-off-by: Stephen Rothwell <s...@canb.auug.org.au>
> ---
>  drivers/gpio/gpio-uniphier.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpio/gpio-uniphier.c b/drivers/gpio/gpio-uniphier.c
> index d62cea4ed6b7..d0069202ade3 100644
> --- a/drivers/gpio/gpio-uniphier.c
> +++ b/drivers/gpio/gpio-uniphier.c
> @@ -300,13 +300,15 @@ static int uniphier_gpio_irq_domain_alloc(struct 
> irq_domain *domain,
>       return irq_domain_alloc_irqs_parent(domain, virq, 1, &parent_fwspec);
>  }
>  
> -static void uniphier_gpio_irq_domain_activate(struct irq_domain *domain,
> -                                           struct irq_data *data)
> +static int uniphier_gpio_irq_domain_activate(struct irq_domain *domain,
> +                                           struct irq_data *data,
> +                                           bool early)
>  {
>       struct uniphier_gpio_priv *priv = domain->host_data;
>       struct gpio_chip *chip = &priv->chip;
>  
>       gpiochip_lock_as_irq(chip, data->hwirq + UNIPHIER_GPIO_IRQ_OFFSET);
> +     return 0;
>  }
>  
>  static void uniphier_gpio_irq_domain_deactivate(struct irq_domain *domain,

Just a reminder that this conflict still exists.

-- 
Cheers,
Stephen Rothwell

Reply via email to