* [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file
@ 2021-01-27 15:34 Daniele Palmas
2021-01-27 15:34 ` [PATCH net-next v2 1/2] net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces Daniele Palmas
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Daniele Palmas @ 2021-01-27 15:34 UTC (permalink / raw)
To: Bjørn Mork, David S . Miller, Jakub Kicinski
Cc: netdev, linux-usb, Aleksander Morgado, Daniele Palmas
Hello,
this patch series add a sysfs file to let userspace know which mux
id has been used to create a qmimux network interface.
I'm aware that adding new sysfs files is not usually the right path,
but my understanding is that this piece of information can't be
retrieved in any other way and its absence restricts how
userspace application (e.g. like libqmi) can take advantage of the
qmimux implementation in qmi_wwan.
Thanks,
Daniele
v2: used sysfs_emit in mux_id_show
Daniele Palmas (2):
net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces
net: qmi_wwan: document qmap/mux_id sysfs file
Documentation/ABI/testing/sysfs-class-net-qmi | 10 +++++++
drivers/net/usb/qmi_wwan.c | 27 +++++++++++++++++++
2 files changed, 37 insertions(+)
--
2.17.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH net-next v2 1/2] net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces
2021-01-27 15:34 [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file Daniele Palmas
@ 2021-01-27 15:34 ` Daniele Palmas
2021-01-27 15:34 ` [PATCH net-next v2 2/2] net: qmi_wwan: document qmap/mux_id sysfs file Daniele Palmas
2021-01-29 2:10 ` [PATCH 0/2] net: usb: qmi_wwan: new mux_id " patchwork-bot+netdevbpf
2 siblings, 0 replies; 5+ messages in thread
From: Daniele Palmas @ 2021-01-27 15:34 UTC (permalink / raw)
To: Bjørn Mork, David S . Miller, Jakub Kicinski
Cc: netdev, linux-usb, Aleksander Morgado, Daniele Palmas
Add qmimux interface sysfs file qmap/mux_id to show qmap id set
during the interface creation, in order to provide a method for
userspace to associate QMI control channels to network interfaces.
Signed-off-by: Daniele Palmas <dnlplm@gmail.com>
---
drivers/net/usb/qmi_wwan.c | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c
index 7ea113f51074..bcfb1e2788fd 100644
--- a/drivers/net/usb/qmi_wwan.c
+++ b/drivers/net/usb/qmi_wwan.c
@@ -218,6 +218,28 @@ static int qmimux_rx_fixup(struct usbnet *dev, struct sk_buff *skb)
return 1;
}
+static ssize_t mux_id_show(struct device *d, struct device_attribute *attr, char *buf)
+{
+ struct net_device *dev = to_net_dev(d);
+ struct qmimux_priv *priv;
+
+ priv = netdev_priv(dev);
+
+ return sysfs_emit(buf, "0x%02x\n", priv->mux_id);
+}
+
+static DEVICE_ATTR_RO(mux_id);
+
+static struct attribute *qmi_wwan_sysfs_qmimux_attrs[] = {
+ &dev_attr_mux_id.attr,
+ NULL,
+};
+
+static struct attribute_group qmi_wwan_sysfs_qmimux_attr_group = {
+ .name = "qmap",
+ .attrs = qmi_wwan_sysfs_qmimux_attrs,
+};
+
static int qmimux_register_device(struct net_device *real_dev, u8 mux_id)
{
struct net_device *new_dev;
@@ -240,6 +262,8 @@ static int qmimux_register_device(struct net_device *real_dev, u8 mux_id)
goto out_free_newdev;
}
+ new_dev->sysfs_groups[0] = &qmi_wwan_sysfs_qmimux_attr_group;
+
err = register_netdevice(new_dev);
if (err < 0)
goto out_free_newdev;
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH net-next v2 2/2] net: qmi_wwan: document qmap/mux_id sysfs file
2021-01-27 15:34 [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file Daniele Palmas
2021-01-27 15:34 ` [PATCH net-next v2 1/2] net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces Daniele Palmas
@ 2021-01-27 15:34 ` Daniele Palmas
2021-01-29 2:10 ` [PATCH 0/2] net: usb: qmi_wwan: new mux_id " patchwork-bot+netdevbpf
2 siblings, 0 replies; 5+ messages in thread
From: Daniele Palmas @ 2021-01-27 15:34 UTC (permalink / raw)
To: Bjørn Mork, David S . Miller, Jakub Kicinski
Cc: netdev, linux-usb, Aleksander Morgado, Daniele Palmas
Document qmap/mux_id sysfs file showing qmimux interface id
Signed-off-by: Daniele Palmas <dnlplm@gmail.com>
---
Documentation/ABI/testing/sysfs-class-net-qmi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/Documentation/ABI/testing/sysfs-class-net-qmi b/Documentation/ABI/testing/sysfs-class-net-qmi
index c310db4ccbc2..ed79f5893421 100644
--- a/Documentation/ABI/testing/sysfs-class-net-qmi
+++ b/Documentation/ABI/testing/sysfs-class-net-qmi
@@ -48,3 +48,13 @@ Description:
Write a number ranging from 1 to 254 to delete a previously
created qmap mux based network device.
+
+What: /sys/class/net/<qmimux iface>/qmap/mux_id
+Date: January 2021
+KernelVersion: 5.12
+Contact: Daniele Palmas <dnlplm@gmail.com>
+Description:
+ Unsigned integer
+
+ Indicates the mux id associated to the qmimux network interface
+ during its creation.
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file
2021-01-27 15:34 [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file Daniele Palmas
2021-01-27 15:34 ` [PATCH net-next v2 1/2] net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces Daniele Palmas
2021-01-27 15:34 ` [PATCH net-next v2 2/2] net: qmi_wwan: document qmap/mux_id sysfs file Daniele Palmas
@ 2021-01-29 2:10 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 5+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-01-29 2:10 UTC (permalink / raw)
To: Daniele Palmas; +Cc: bjorn, davem, kuba, netdev, linux-usb, aleksander
Hello:
This series was applied to netdev/net-next.git (refs/heads/master):
On Wed, 27 Jan 2021 16:34:31 +0100 you wrote:
> Hello,
>
> this patch series add a sysfs file to let userspace know which mux
> id has been used to create a qmimux network interface.
>
> I'm aware that adding new sysfs files is not usually the right path,
> but my understanding is that this piece of information can't be
> retrieved in any other way and its absence restricts how
> userspace application (e.g. like libqmi) can take advantage of the
> qmimux implementation in qmi_wwan.
>
> [...]
Here is the summary with links:
- [net-next,v2,1/2] net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces
https://git.kernel.org/netdev/net-next/c/e594ad980ec2
- [net-next,v2,2/2] net: qmi_wwan: document qmap/mux_id sysfs file
https://git.kernel.org/netdev/net-next/c/b4b91e24094a
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file
@ 2021-01-25 15:22 Daniele Palmas
0 siblings, 0 replies; 5+ messages in thread
From: Daniele Palmas @ 2021-01-25 15:22 UTC (permalink / raw)
To: Bjørn Mork, David S . Miller, Jakub Kicinski
Cc: netdev, linux-usb, Aleksander Morgado, Daniele Palmas
Hello,
this patch series add a sysfs file to let userspace know which mux
id has been used to create a qmimux network interface.
I'm aware that adding new sysfs files is not usually the right path,
but my understanding is that this piece of information can't be
retrieved in any other way and its absence restricts how
userspace application (e.g. like libqmi) can take advantage of the
qmimux implementation in qmi_wwan.
Thanks,
Daniele
Daniele Palmas (2):
net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces
net: qmi_wwan: document qmap/mux_id sysfs file
Documentation/ABI/testing/sysfs-class-net-qmi | 10 +++++++
drivers/net/usb/qmi_wwan.c | 27 +++++++++++++++++++
2 files changed, 37 insertions(+)
--
2.17.1
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-01-29 2:10 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-27 15:34 [PATCH 0/2] net: usb: qmi_wwan: new mux_id sysfs file Daniele Palmas
2021-01-27 15:34 ` [PATCH net-next v2 1/2] net: usb: qmi_wwan: add qmap id sysfs file for qmimux interfaces Daniele Palmas
2021-01-27 15:34 ` [PATCH net-next v2 2/2] net: qmi_wwan: document qmap/mux_id sysfs file Daniele Palmas
2021-01-29 2:10 ` [PATCH 0/2] net: usb: qmi_wwan: new mux_id " patchwork-bot+netdevbpf
-- strict thread matches above, loose matches on Subject: below --
2021-01-25 15:22 Daniele Palmas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).