[PATCH net-next v2] ppp: update Kconfig help message

Qingfang Deng posted 1 patch 2 months, 1 week ago
drivers/net/ppp/Kconfig | 16 +++++++---------
1 file changed, 7 insertions(+), 9 deletions(-)
[PATCH net-next v2] ppp: update Kconfig help message
Posted by Qingfang Deng 2 months, 1 week ago
Both links of the PPPoE section are no longer valid, and the CVS version
is no longer relevant.

- Replace the TLDP URL with the pppd project homepage.
- Update pppd version requirement for PPPoE.
- Update RP-PPPoE project homepage, and clarify that it's only needed
  for server mode.

Signed-off-by: Qingfang Deng <qingfang.deng@linux.dev>
---
v2:
 - replace TLDP URL with pppd homepage
 - update pppd version requirement
 - document that rp-pppoe is only required for server
 https://lore.kernel.org/netdev/20260331033303.5664-1-dqfext@gmail.com/

 drivers/net/ppp/Kconfig | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/drivers/net/ppp/Kconfig b/drivers/net/ppp/Kconfig
index f57fba84fe55..753354b4e36c 100644
--- a/drivers/net/ppp/Kconfig
+++ b/drivers/net/ppp/Kconfig
@@ -13,10 +13,9 @@ config PPP
 	  otherwise you can't use it; most Internet access providers these
 	  days support PPP rather than SLIP.
 
-	  To use PPP, you need an additional program called pppd as described
-	  in the PPP-HOWTO, available at
-	  <http://www.tldp.org/docs.html#howto>.  Make sure that you have
-	  the version of pppd recommended in <file:Documentation/Changes>.
+	  To use PPP, you need an additional program called pppd, available at
+	  <https://ppp.samba.org>.  Make sure that you have the version of pppd
+	  recommended in <file:Documentation/Changes>.
 	  The PPP option enlarges your kernel by about 16 KB.
 
 	  There are actually two versions of PPP: the traditional PPP for
@@ -116,11 +115,10 @@ config PPPOE
 	help
 	  Support for PPP over Ethernet.
 
-	  This driver requires the latest version of pppd from the CVS
-	  repository at cvs.samba.org.  Alternatively, see the
-	  RoaringPenguin package (<http://www.roaringpenguin.com/pppoe>)
-	  which contains instruction on how to use this driver (under
-	  the heading "Kernel mode PPPoE").
+	  To work in client mode, pppd version 2.4.2 or later is required.
+	  To work in server mode, the pppoe-server command from the RP-PPPoE
+	  package is also required, available at
+	  <https://dianne.skoll.ca/projects/rp-pppoe/>.
 
 choice
 	prompt "Number of PPPoE hash bits"
-- 
2.43.0
Re: [PATCH net-next v2] ppp: update Kconfig help message
Posted by Julian Braha 2 months, 1 week ago
I didn't realize there were dead links in the Kconfig help texts.
I've added a check for this to kconfirm and found many more instances!
In most cases, I'm not the right person to be replacing these links;
I'll send out a message soon to the mailing list with what's been
detected. Thanks for the inspiration, Qingfang!

Also, LGTM, so:
Reviewed-by: Julian Braha <julianbraha@gmail.com>
Re: [PATCH net-next v2] ppp: update Kconfig help message
Posted by Jaco Kroon 2 months, 1 week ago
Hi,

On 2026/04/02 07:00, Qingfang Deng wrote:
> Both links of the PPPoE section are no longer valid, and the CVS version
> is no longer relevant.
>
> - Replace the TLDP URL with the pppd project homepage.
> - Update pppd version requirement for PPPoE.
> - Update RP-PPPoE project homepage, and clarify that it's only needed
>    for server mode.
>
> Signed-off-by: Qingfang Deng <qingfang.deng@linux.dev>
FWIW:  Acked-by: Jaco Kroon <jaco@uls.co.za>

Looks good to me.

> ---
> v2:
>   - replace TLDP URL with pppd homepage
>   - update pppd version requirement
>   - document that rp-pppoe is only required for server
>   https://lore.kernel.org/netdev/20260331033303.5664-1-dqfext@gmail.com/
>
>   drivers/net/ppp/Kconfig | 16 +++++++---------
>   1 file changed, 7 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/net/ppp/Kconfig b/drivers/net/ppp/Kconfig
> index f57fba84fe55..753354b4e36c 100644
> --- a/drivers/net/ppp/Kconfig
> +++ b/drivers/net/ppp/Kconfig
> @@ -13,10 +13,9 @@ config PPP
>   	  otherwise you can't use it; most Internet access providers these
>   	  days support PPP rather than SLIP.
>   
> -	  To use PPP, you need an additional program called pppd as described
> -	  in the PPP-HOWTO, available at
> -	  <http://www.tldp.org/docs.html#howto>.  Make sure that you have
> -	  the version of pppd recommended in <file:Documentation/Changes>.
> +	  To use PPP, you need an additional program called pppd, available at
> +	  <https://ppp.samba.org>.  Make sure that you have the version of pppd
> +	  recommended in <file:Documentation/Changes>.
>   	  The PPP option enlarges your kernel by about 16 KB.
>   
>   	  There are actually two versions of PPP: the traditional PPP for
> @@ -116,11 +115,10 @@ config PPPOE
>   	help
>   	  Support for PPP over Ethernet.
>   
> -	  This driver requires the latest version of pppd from the CVS
> -	  repository at cvs.samba.org.  Alternatively, see the
> -	  RoaringPenguin package (<http://www.roaringpenguin.com/pppoe>)
> -	  which contains instruction on how to use this driver (under
> -	  the heading "Kernel mode PPPoE").
> +	  To work in client mode, pppd version 2.4.2 or later is required.
> +	  To work in server mode, the pppoe-server command from the RP-PPPoE
> +	  package is also required, available at
> +	  <https://dianne.skoll.ca/projects/rp-pppoe/>.
>   
>   choice
>   	prompt "Number of PPPoE hash bits"