Linux Serial subsystem development
 help / color / mirror / Atom feed
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>,
	kbuild test robot <fengguang.wu@intel.com>,
	kbuild-all@01.org, linux-serial@vger.kernel.org,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>
Subject: Re: [tty:tty-next 23/34] include/linux/kconfig.h:5:10: fatal error: generated/autoconf.h: No such file or directory
Date: Sun, 17 Dec 2017 12:57:31 +0100	[thread overview]
Message-ID: <20171217115731.GB19641@kroah.com> (raw)
In-Reply-To: <CAMuHMdXNNB7+-J7+M+0k+s=TDvTs_MVM4=AjjSi0x63p2HBaNQ@mail.gmail.com>

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
> <fengguang.wu@intel.com> 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

      reply	other threads:[~2017-12-17 11:57 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <201712161029.StXUUOBC%fengguang.wu@intel.com>
2017-12-17 10:15 ` [tty:tty-next 23/34] include/linux/kconfig.h:5:10: fatal error: generated/autoconf.h: No such file or directory Geert Uytterhoeven
2017-12-17 11:57   ` Greg Kroah-Hartman [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171217115731.GB19641@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=fengguang.wu@intel.com \
    --cc=geert+renesas@glider.be \
    --cc=geert@linux-m68k.org \
    --cc=kbuild-all@01.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox