Hello Alexander, On Wed, Feb 3, 2016 at 1:58 PM, Alexander Kuleshov wrote: > Signed-off-by: Alexander Kuleshov <kuleshovm...@gmail.com>
Please describe the changes in each patch. Why do you need this warning? If you haven't read it yet, please read (esp. Section 2): https://www.kernel.org/doc/Documentation/SubmittingPatches > --- > init/do_mounts.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/init/do_mounts.c b/init/do_mounts.c > index dea5de9..ca5da05 100644 > --- a/init/do_mounts.c > +++ b/init/do_mounts.c > @@ -505,7 +505,9 @@ void __init change_floppy(char *fmt, ...) > termios.c_lflag |= ICANON; > sys_ioctl(fd, TCSETSF, (long)&termios); > sys_close(fd); > - } > + } else > + pr_err("Warning: unable to open /dev/console.\n"); Your message says 'Warning'. But you use pr_err(). I think pr_warn() might be a better option. > + > } > #endif > > -- > 2.7.0.25.gfc10eb5 > Thanks, -- Pranith
