On Wed, Jan 21, 2026 at 03:16:33PM +0100, Loic Poulain wrote:

On Wed, Jan 21, 2026 at 3:09 AM Slark Xiao [off-list ref] wrote:

quoted

At 2026-01-20 20:30:37, "Simon Horman" [off-list ref] wrote:

quoted

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

...

quoted

diff --git a/drivers/net/wwan/wwan_core.c b/drivers/net/wwan/wwan_core.c

...

quoted

+#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.

I think this shall be a mistake.
For a GNSS device, these 2 API shall not be called frequently.
So there is no necessary to add "inline" to improve performance.

Indeed, it's fine to remove that explicit inlines here.

Thanks!