On Wed, Oct 31, 2012 at 8:00 AM, Yuanhan Liu <yuanhan....@linux.intel.com> wrote:
> Those functions are availabe only when CONFIG_GPIOLIB is set. So, we > should not call them under !CONFIG_GPIOLIB block. > > This would fix following build errros: > include/asm-generic/gpio.h: In function 'gpio_get_value_cansleep': > include/asm-generic/gpio.h:220:2: error: implicit declaration of function > '__gpio_get_value' > include/asm-generic/gpio.h: In function 'gpio_set_value_cansleep': > nclude/asm-generic/gpio.h:226:2: error: implicit declaration of function > '__gpio_set_value' OK... > static inline int gpio_get_value_cansleep(unsigned gpio) > { > - might_sleep(); So why are you deleting this very useful might_sleep() runtime semantic check? > - return __gpio_get_value(gpio); > + WARN_ON(1); > + return 0; > } > > static inline void gpio_set_value_cansleep(unsigned gpio, int value) > { > - might_sleep(); > - __gpio_set_value(gpio, value); > + WARN_ON(1); > } Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/