[PATCH net-next] docs: phonet: Fix the struct sockaddr_pn name in the example

Subsystems: documentation, networking [general], phonet protocol, the rest

COLD30d IN LINUX-NEXT

Queued in linux-next as 413999d2e588 on 2026-09-08.

3 messages, 3 authors, 2026-09-08 · open the first message on its own page

[PATCH net-next] docs: phonet: Fix the struct sockaddr_pn name in the example

From: Karl Mehltretter <hidden>
Date: 2026-09-05 10:41:31

struct sockaddr_spn does not exist; the name is struct sockaddr_pn.

Use struct sockaddr_pn.

Fixes: 297edb600326 ("Phonet: support active connection without pipe controller on modem")
Assisted-by: LLM
Signed-off-by: Karl Mehltretter <redacted>
---
 Documentation/networking/phonet.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/networking/phonet.rst b/Documentation/networking/phonet.rst
index d705cc5b09fc..e5551fc45c78 100644
--- a/Documentation/networking/phonet.rst
+++ b/Documentation/networking/phonet.rst
@@ -172,7 +172,7 @@ two endpoints directly, using connect() on the active side. This is
 intended to support the newer Nokia Wireless Modem API, as found in
 e.g. the Nokia Slim Modem in the ST-Ericsson U8500 platform::
 
-  struct sockaddr_spn spn;
+  struct sockaddr_pn spn;
   int fd;
 
   fd = socket(PF_PHONET, SOCK_SEQPACKET, PN_PROTO_PIPE);
-- 
2.53.0

Re: [PATCH net-next] docs: phonet: Fix the struct sockaddr_pn name in the example

From: Randy Dunlap <rdunlap@infradead.org>
Date: 2026-09-05 19:30:57


On 9/5/26 3:41 AM, Karl Mehltretter wrote:
struct sockaddr_spn does not exist; the name is struct sockaddr_pn.

Use struct sockaddr_pn.

Fixes: 297edb600326 ("Phonet: support active connection without pipe controller on modem")
Assisted-by: LLM
Signed-off-by: Karl Mehltretter <redacted>
Acked-by: Randy Dunlap <rdunlap@infradead.org>
thanks.
quoted hunk
---
 Documentation/networking/phonet.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/networking/phonet.rst b/Documentation/networking/phonet.rst
index d705cc5b09fc..e5551fc45c78 100644
--- a/Documentation/networking/phonet.rst
+++ b/Documentation/networking/phonet.rst
@@ -172,7 +172,7 @@ two endpoints directly, using connect() on the active side. This is
 intended to support the newer Nokia Wireless Modem API, as found in
 e.g. the Nokia Slim Modem in the ST-Ericsson U8500 platform::
 
-  struct sockaddr_spn spn;
+  struct sockaddr_pn spn;
   int fd;
 
   fd = socket(PF_PHONET, SOCK_SEQPACKET, PN_PROTO_PIPE);
-- 
~Randy

Re: [PATCH net-next] docs: phonet: Fix the struct sockaddr_pn name in the example

From: patchwork-bot+netdevbpf@kernel.org
Date: 2026-09-08 23:51:15

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski [off-list ref]:

On Sat,  5 Sep 2026 12:41:23 +0200 you wrote:
struct sockaddr_spn does not exist; the name is struct sockaddr_pn.

Use struct sockaddr_pn.

Fixes: 297edb600326 ("Phonet: support active connection without pipe controller on modem")
Assisted-by: LLM
Signed-off-by: Karl Mehltretter <redacted>

[...]
Here is the summary with links:
  - [net-next] docs: phonet: Fix the struct sockaddr_pn name in the example
    https://git.kernel.org/netdev/net-next/c/413999d2e588

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help