From mboxrd@z Thu Jan  1 00:00:00 1970
From: sshtylyov@ru.mvista.com (Sergei Shtylyov)
Date: Mon, 17 Oct 2011 14:54:40 +0400
Subject: [PATCH 04/11] ARM: at91/pata: use gpio_is_valid to check the gpio
In-Reply-To: <1318792160-23279-4-git-send-email-plagnioj@jcrosoft.com>
References: <20111016190528.GI1459@game.jcrosoft.org>
	<1318792160-23279-4-git-send-email-plagnioj@jcrosoft.com>
Message-ID: <4E9C0970.5000707@ru.mvista.com>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

Hello.

On 16.10.2011 23:09, Jean-Christophe PLAGNIOL-VILLARD wrote:

> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
>   drivers/ata/pata_at91.c |    6 +++---
>   1 files changed, 3 insertions(+), 3 deletions(-)

    You should post this patch to linux-ide at vger.kernel.org...

> diff --git a/drivers/ata/pata_at91.c b/drivers/ata/pata_at91.c
> index 960c725..3915353 100644
> --- a/drivers/ata/pata_at91.c
> +++ b/drivers/ata/pata_at91.c
> @@ -360,7 +360,7 @@ static int __devinit pata_at91_probe(struct platform_device *pdev)
>   	ap->flags |= ATA_FLAG_SLAVE_POSS;
>   	ap->pio_mask = ATA_PIO4;
>
> -	if (!irq) {
> +	if (!gpio_is_valid(irq)) {
>   		ap->flags |= ATA_FLAG_PIO_POLLING;
>   		ata_port_desc(ap, "no IRQ, using PIO polling");
>   	}
> @@ -414,8 +414,8 @@ static int __devinit pata_at91_probe(struct platform_device *pdev)
>
>   	host->private_data = info;
>
> -	return ata_host_activate(host, irq ? gpio_to_irq(irq) : 0,
> -			irq ? ata_sff_interrupt : NULL,
> +	return ata_host_activate(host, gpio_is_valid(irq) ? gpio_to_irq(irq) : 0,
> +			gpio_is_valid(irq) ? ata_sff_interrupt : NULL,
>   			irq_flags,&pata_at91_sht);
>   err_put:

    BTW, the driver forgets to call clk_put() iff ata_host_activate() fails; 
I've already posted a patch to fix this -- it conflicts with this one.

WBR, Sergei