Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <8dbed370-c6d6-496c-eb41-efce2ee78286@microchip.com>
Date:   Mon, 11 Jul 2022 14:03:07 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: [PATCH] gpio: GPIO_SAMA5D2_PIOBU should depend on ARCH_AT91
Content-Language: en-US
To:     Geert Uytterhoeven <geert+renesas@glider.be>,
        Ludovic Desroches <ludovic.desroches@microchip.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        Bartosz Golaszewski <brgl@bgdev.pl>,
        Conor Dooley <Conor.Dooley@microchip.com>,
        Horatiu Vultur <Horatiu.Vultur@microchip.com>,
        Kavyasree Kotagiri <Kavyasree.Kotagiri@microchip.com>
CC:     <linux-gpio@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>
References: <9996cb86818d2e935494d6b414d549432f91797a.1657528504.git.geert+renesas@glider.be>
From:   Nicolas Ferre <nicolas.ferre@microchip.com>
Organization: microchip
In-Reply-To: <9996cb86818d2e935494d6b414d549432f91797a.1657528504.git.geert+renesas@glider.be>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 11/07/2022 at 10:36, Geert Uytterhoeven wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> The SAMA5D2 PIOBU is only present on some AT91/Microchip SoCs.  Hence
> add a dependency on ARCH_AT91, to prevent asking the user about this
> driver when configuring a kernel without AT91/Microchip SoC support.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Looks good to me:
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>

I'm adding to the thread my colleagues from lan966 and mpfs if it 
happens that they use this same IP on their SoC. Anyway, we can add them 
afterwards.

Thanks for your heads-up Geert, Best regards,
   Nicolas

> ---
>   drivers/gpio/Kconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
> index 63a89ff7865e374f..1852656d5eb248f6 100644
> --- a/drivers/gpio/Kconfig
> +++ b/drivers/gpio/Kconfig
> @@ -553,6 +553,7 @@ config GPIO_SAMA5D2_PIOBU
>          tristate "SAMA5D2 PIOBU GPIO support"
>          depends on MFD_SYSCON
>          depends on OF_GPIO
> +       depends on ARCH_AT91 || COMPILE_TEST
>          select GPIO_SYSCON
>          help
>            Say yes here to use the PIOBU pins as GPIOs.
> --
> 2.25.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


-- 
Nicolas Ferre