Thread (22 messages) 22 messages, 5 authors, 2026-01-22

Re: [net-next v7 5/8] net: wwan: add NMEA port support

From: Simon Horman <horms@kernel.org>
Date: 2026-01-20 12:30:43
Also in: lkml

On Thu, Jan 15, 2026 at 07:46:22PM +0800, Slark Xiao wrote:

...
quoted hunk ↗ jump to hunk
diff --git a/drivers/net/wwan/wwan_core.c b/drivers/net/wwan/wwan_core.c
...
+#else
+static inline int wwan_port_register_gnss(struct wwan_port *port)
+{
+	return -EOPNOTSUPP;
+}
+
+static inline void wwan_port_unregister_gnss(struct wwan_port *port)
+{
+	WARN_ON(1);	/* This handler cannot be called */
+}
Please don't use the inline keyword in .c files unless there is a
demonstrable - usually performance - reason to do so. Rather, let the
compiler inline code as it sees fit.

Using the inline keyword in .h is of course fine.
+#endif
+
...
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help