Re: [LINUX PATCHv3 6/9] serial-uartlite: Change logic how console_port is setup

[kbuild test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3466 bytes --]

Hi Shubhrajyoti,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on tty/tty-testing]
[also build test ERROR on next-20181016]
[cannot apply to v4.19-rc8]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/shubhrajyoti-datta-gmail-com/serial-uartlite-Add-support-for-dynamic-allocation/20181016-210709
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git tty-testing
config: i386-randconfig-x019-201841 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

Note: the linux-review/shubhrajyoti-datta-gmail-com/serial-uartlite-Add-support-for-dynamic-allocation/20181016-210709 HEAD 7aba44c5f67836025decfe4dc4ab364c1f83f44d builds fine.
      It only hurts bisectibility.

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/ioport.h:13:0,
                    from include/linux/device.h:15,
                    from include/linux/platform_device.h:14,
                    from drivers/tty//serial/uartlite.c:9:
   drivers/tty//serial/uartlite.c: In function 'ulite_assign':
>> drivers/tty//serial/uartlite.c:684:25: error: invalid type argument of '->' (have 'struct uart_driver')
         !(ulite_uart_driver->cons->flags & CON_ENABLED))
                            ^
   include/linux/compiler.h:58:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> drivers/tty//serial/uartlite.c:683:2: note: in expansion of macro 'if'
     if (console_port == port &&
     ^~
>> drivers/tty//serial/uartlite.c:684:25: error: invalid type argument of '->' (have 'struct uart_driver')
         !(ulite_uart_driver->cons->flags & CON_ENABLED))
                            ^
   include/linux/compiler.h:58:42: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                             ^~~~
>> drivers/tty//serial/uartlite.c:683:2: note: in expansion of macro 'if'
     if (console_port == port &&
     ^~
>> drivers/tty//serial/uartlite.c:684:25: error: invalid type argument of '->' (have 'struct uart_driver')
         !(ulite_uart_driver->cons->flags & CON_ENABLED))
                            ^
   include/linux/compiler.h:69:16: note: in definition of macro '__trace_if'
      ______r = !!(cond);     \
                   ^~~~
>> drivers/tty//serial/uartlite.c:683:2: note: in expansion of macro 'if'
     if (console_port == port &&
     ^~

vim +684 drivers/tty//serial/uartlite.c

   671	
   672		/* Register the port */
   673		rc = uart_add_one_port(&ulite_uart_driver, port);
   674		if (rc) {
   675			dev_err(dev, "uart_add_one_port() failed; err=%i\n", rc);
   676			port->mapbase = 0;
   677			dev_set_drvdata(dev, NULL);
   678			return rc;
   679		}
   680	
   681	#ifdef CONFIG_SERIAL_UARTLITE_CONSOLE
   682		/* This is not port which is used for console that's why clean it up */
 > 683		if (console_port == port &&
 > 684		    !(ulite_uart_driver->cons->flags & CON_ENABLED))
   685			console_port = NULL;
   686	#endif
   687	
   688		return 0;
   689	}
   690	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 28007 bytes --]