Hi Chris
Please add this to mmc-next.
Regards
Adrian
On 28/05/13 09:27, Adrian Hunter wrote:
> On 28/05/13 08:35, Jingoo Han wrote:
>> On Tuesday, May 28, 2013 2:26 PM, Wei Yongjun wrote:
>>>
>>> From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
>>>
>>> Fix to return a negative error code in the gpio_to_irq() error
>>> handling case instead of 0, as done elsewhere in this function.
>>>
>>> Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
>>
>> It looks good.
>>
>> Reviewed-by: Jingoo Han <jg1.han@xxxxxxxxxxx>
>
> Yes, thank you!
>
> Acked-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>
>
>>
>> Best regards,
>> Jingoo Han
>>
>>> ---
>>> drivers/mmc/host/sdhci-acpi.c | 4 +++-
>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
>>> index a51e603..08a85ec 100644
>>> --- a/drivers/mmc/host/sdhci-acpi.c
>>> +++ b/drivers/mmc/host/sdhci-acpi.c
>>> @@ -189,8 +189,10 @@ static int sdhci_acpi_add_own_cd(struct device *dev, int gpio,
>>> goto out;
>>>
>>> irq = gpio_to_irq(gpio);
>>> - if (irq < 0)
>>> + if (irq < 0) {
>>> + err = irq;
>>> goto out_free;
>>> + }
>>>
>>> flags = IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING;
>>> err = devm_request_irq(dev, irq, sdhci_acpi_sd_cd, flags, "sd_cd", mmc);
>>
>>
>>
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
