diff mbox series

[v2,01/14] net: wwan: Add default MTU size

Message ID 20211101035635.26999-2-ricardo.martinez@linux.intel.com (mailing list archive)
State Not Applicable
Delegated to: Johannes Berg
Headers show
Series net: wwan: t7xx: PCIe driver for MediaTek M.2 modem | expand

Commit Message

Ricardo Martinez Nov. 1, 2021, 3:56 a.m. UTC
Add a default MTU size definition that new WWAN drivers can refer to.

Signed-off-by: Ricardo Martinez <ricardo.martinez@linux.intel.com>
---
 include/linux/wwan.h | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Sergey Ryazanov Nov. 6, 2021, 6:01 p.m. UTC | #1
On Mon, Nov 1, 2021 at 6:57 AM Ricardo Martinez
<ricardo.martinez@linux.intel.com> wrote:
> Add a default MTU size definition that new WWAN drivers can refer to.

[skipped]

> +/*
> + * Default WWAN interface MTU value
> + */
> +#define WWAN_DEFAULT_MTU       1500

Why do you need another one macro for the 1500 bytes default MTU?
Consider using ETH_DATA_LEN macro.

--
Sergey
diff mbox series

Patch

diff --git a/include/linux/wwan.h b/include/linux/wwan.h
index 9fac819f92e3..28934b7dd0ae 100644
--- a/include/linux/wwan.h
+++ b/include/linux/wwan.h
@@ -171,4 +171,9 @@  int wwan_register_ops(struct device *parent, const struct wwan_ops *ops,
 
 void wwan_unregister_ops(struct device *parent);
 
+/*
+ * Default WWAN interface MTU value
+ */
+#define WWAN_DEFAULT_MTU       1500
+
 #endif /* __WWAN_H */