Linux Serial subsystem development
 help / color / mirror / Atom feed
* [PATCH] serial: 8250_hub6: add missing include for hub6_match_port()
@ 2026-07-14  1:26 Hugo Villeneuve
  2026-07-14 23:42 ` kernel test robot
  0 siblings, 1 reply; 2+ messages in thread
From: Hugo Villeneuve @ 2026-07-14  1:26 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Jiri Slaby, Hugo Villeneuve
  Cc: hugo, kernel test robot, linux-kernel, linux-serial

From: Hugo Villeneuve <hvilleneuve@dimonoff.com>

Add missing include to fix compile warning:

  drivers/tty/serial/8250/8250_hub6.c:44:6: warning: no previous prototype for
  'hub6_match_port' [-Wmissing-prototypes]

Fixes: 3d406299d882 ("serial: 8250_hub6: add hub6_match_port()")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202607110715.VGT2dVVz-lkp@intel.com/
Closes: https://lore.kernel.org/oe-kbuild-all/202607111219.QG9uOW8H-lkp@intel.com/
Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com>
---
Not CCing stable as commit is still in Greg's tty-next tree
---
 drivers/tty/serial/8250/8250_hub6.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/tty/serial/8250/8250_hub6.c b/drivers/tty/serial/8250/8250_hub6.c
index eae32c924e29b..b6767633c966c 100644
--- a/drivers/tty/serial/8250/8250_hub6.c
+++ b/drivers/tty/serial/8250/8250_hub6.c
@@ -7,6 +7,8 @@
 #include <linux/init.h>
 #include <linux/serial_8250.h>
 
+#include "8250.h"
+
 #define HUB6(card, port)						\
 	{								\
 		.iobase		= 0x302,				\

base-commit: c1d4ce2d9eaeccd7e8aef7ec1109df49eaf4f503
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] serial: 8250_hub6: add missing include for hub6_match_port()
  2026-07-14  1:26 [PATCH] serial: 8250_hub6: add missing include for hub6_match_port() Hugo Villeneuve
@ 2026-07-14 23:42 ` kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2026-07-14 23:42 UTC (permalink / raw)
  To: Hugo Villeneuve, Greg Kroah-Hartman, Jiri Slaby, Hugo Villeneuve
  Cc: oe-kbuild-all, hugo, kernel test robot, linux-kernel,
	linux-serial

Hi Hugo,

kernel test robot noticed the following build errors:

[auto build test ERROR on c1d4ce2d9eaeccd7e8aef7ec1109df49eaf4f503]

url:    https://github.com/intel-lab-lkp/linux/commits/Hugo-Villeneuve/serial-8250_hub6-add-missing-include-for-hub6_match_port/20260714-092814
base:   c1d4ce2d9eaeccd7e8aef7ec1109df49eaf4f503
patch link:    https://lore.kernel.org/r/20260714012610.576746-1-hugo%40hugovil.com
patch subject: [PATCH] serial: 8250_hub6: add missing include for hub6_match_port()
config: parisc-allmodconfig (https://download.01.org/0day-ci/archive/20260715/202607150717.2YxVdWpX-lkp@intel.com/config)
compiler: hppa-linux-gcc (GCC) 16.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260715/202607150717.2YxVdWpX-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202607150717.2YxVdWpX-lkp@intel.com/

All errors (new ones prefixed by >>):

>> drivers/tty/serial/8250/8250_hub6.c:46:6: error: redefinition of 'hub6_match_port'
      46 | bool hub6_match_port(const struct uart_port *port1, const struct uart_port *port2)
         |      ^~~~~~~~~~~~~~~
   In file included from drivers/tty/serial/8250/8250_hub6.c:10:
   drivers/tty/serial/8250/8250.h:340:20: note: previous definition of 'hub6_match_port' with type 'bool(const struct uart_port *, const struct uart_port *)' {aka '_Bool(const struct uart_port *, const struct uart_port *)'}
     340 | static inline bool hub6_match_port(const struct uart_port *port1, const struct uart_port *port2)
         |                    ^~~~~~~~~~~~~~~


vim +/hub6_match_port +46 drivers/tty/serial/8250/8250_hub6.c

ec9f47cd6a14ca drivers/serial/8250_hub6.c          Russell King    2005-06-27  45  
3d406299d88297 drivers/tty/serial/8250/8250_hub6.c Hugo Villeneuve 2026-05-21 @46  bool hub6_match_port(const struct uart_port *port1, const struct uart_port *port2)
3d406299d88297 drivers/tty/serial/8250/8250_hub6.c Hugo Villeneuve 2026-05-21  47  {
3d406299d88297 drivers/tty/serial/8250/8250_hub6.c Hugo Villeneuve 2026-05-21  48  	return port1->iobase == port2->iobase && port1->hub6 == port2->hub6;
3d406299d88297 drivers/tty/serial/8250/8250_hub6.c Hugo Villeneuve 2026-05-21  49  }
3d406299d88297 drivers/tty/serial/8250/8250_hub6.c Hugo Villeneuve 2026-05-21  50  EXPORT_SYMBOL_GPL(hub6_match_port);
3d406299d88297 drivers/tty/serial/8250/8250_hub6.c Hugo Villeneuve 2026-05-21  51  

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-07-14 23:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-14  1:26 [PATCH] serial: 8250_hub6: add missing include for hub6_match_port() Hugo Villeneuve
2026-07-14 23:42 ` kernel test robot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox