On Tue, Jan 23, 2018 at 12:37:34AM +0530, Kohli, Gaurav wrote: > Hi Greg, > > Sorry for the mistake from my side, as i didn't test without TTY. > I think we have to remove full else part as no one is user of these function with CONFIG_TTY disabled. > > In file included from init/do_mounts.c:15:0: > include/linux/tty.h:436:19: warning: 'tty_ldisc_lock' declared 'static' but never defined [-Wunused-function] > static inline int tty_ldisc_lock(struct tty_struct *tty, unsigned long timeout); > ^~~~~~~~~~~~~~ > include/linux/tty.h:438:20: warning: 'tty_ldisc_unlock' declared 'static' but never defined [-Wunused-function] > static inline void tty_ldisc_unlock(struct tty_struct *tty); ^~~~~~~~~~~~~~~~ > > Please let me know if you want me to re upload. Yes please send me a new version of the patch, I will drop this older version now. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html