It is wrong to pass 'int' to IS_ERR_VALUE(), Linus's commit
aa00edc1 point out this abuse.
This patch use operator instead.
Signed-off-by: Haibo Chen <haibo.chen@nxp.com>
int gpio_cd = mmc_gpio_get_cd(host->mmc);
if ((host->quirks & SDHCI_QUIRK_BROKEN_CARD_DETECTION) ||
- !mmc_card_is_removable(host->mmc) || !IS_ERR_VALUE(gpio_cd))
+ !mmc_card_is_removable(host->mmc) || (gpio_cd >= 0))
return;
if (enable) {
val |= mask ;
/* Avoid fake wake up */
if (host->quirks & SDHCI_QUIRK_BROKEN_CARD_DETECTION ||
- !IS_ERR_VALUE(gpio_cd)) {
+ (gpio_cd >= 0)) {
val &= ~(SDHCI_WAKE_ON_INSERT | SDHCI_WAKE_ON_REMOVE);
irq_val &= ~(SDHCI_INT_CARD_INSERT | SDHCI_INT_CARD_REMOVE);
}