On Sun, Dec 17, 2017 at 11:15:37AM +0100, Geert Uytterhoeven wrote: > On Sat, Dec 16, 2017 at 3:00 AM, kbuild test robot > <[email protected]> wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git > > tty-next > > head: 38d5583ff8900c543322a74471b436d1b8b4018a > > commit: f6731485a51978ca0931c787fcb8a0bc4dcc9303 [23/34] tty: serial: > > sh-sci: Hide number of ports config question > > config: ia64-allyesconfig (attached as .config) > > compiler: ia64-linux-gcc (GCC) 7.2.0 > > reproduce: > > wget > > https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O > > ~/bin/make.cross > > chmod +x ~/bin/make.cross > > git checkout f6731485a51978ca0931c787fcb8a0bc4dcc9303 > > # save the attached .config to linux build tree > > make.cross ARCH=ia64 > > > > All errors (new ones prefixed by >>): > > > > .config:3814:warning: symbol value '' invalid for SERIAL_SH_SCI_NR_UARTS > > config SERIAL_SH_SCI_NR_UARTS > int "Maximum number of SCI(F) serial ports" if EXPERT > depends on SERIAL_SH_SCI > default "3" if H8300 > default "10" if SUPERH > default "18" if ARCH_RENESAS > > Doh, if test compiling for non-H8300/SUPERH/ARCH_RENESAS, "int" doesn't > assign a sane numerical default value to the symbol, but an empty string :-( > > So we do need e.g. > > default "2" > > at the bottom. > > Greg: Do you want a v2, or an incremental patch?
Incremental is good, I don't want to have to revert and then add a v2. thanks, greg k-h
