DistroKit Mailinglist
 help / color / mirror / Atom feed
* [DistroKit] [PATCH] v7a: barebox: imx6: don't enable fastboot net by default
@ 2024-03-06 18:46 Ahmad Fatoum
  2024-03-11 12:06 ` Robert Schwebel
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2024-03-06 18:46 UTC (permalink / raw)
  To: distrokit; +Cc: Ahmad Fatoum

The idea behind enabling the USB gadget by default was to allow
quick experimentation with only a USB connection by exposing both
fastboot and a ACM serial gadget.

However, Fastboot over UDP, isn't as useful:

  - the user first has to find out the IP address assigned to the board
  - getting a DHCP lease on every boot takes a bit of time
  - The shell is unusable until the DHCP request succeeds or times out.
    Ctrl+C can't be used as it's not called interactively

Therefore, disable Fastboot over UDP and leave only USB enabled by
default. User wishing to use Fastboot over UDP can just set

  nv.fastboot.net.autostart=1

manually to restore the old behavior.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 .../barebox-mx6-defaultenv/nv/fastboot.net.autostart             | 1 -
 1 file changed, 1 deletion(-)
 delete mode 100644 configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart

diff --git a/configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart b/configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart
deleted file mode 100644
index d00491fd7e5b..000000000000
--- a/configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart
+++ /dev/null
@@ -1 +0,0 @@
-1
-- 
2.39.2




^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [DistroKit] [PATCH] v7a: barebox: imx6: don't enable fastboot net by default
  2024-03-06 18:46 [DistroKit] [PATCH] v7a: barebox: imx6: don't enable fastboot net by default Ahmad Fatoum
@ 2024-03-11 12:06 ` Robert Schwebel
  0 siblings, 0 replies; 2+ messages in thread
From: Robert Schwebel @ 2024-03-11 12:06 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: distrokit

Applied to next.

On Wed, Mar 06, 2024 at 07:46:40PM +0100, Ahmad Fatoum wrote:
> The idea behind enabling the USB gadget by default was to allow
> quick experimentation with only a USB connection by exposing both
> fastboot and a ACM serial gadget.
> 
> However, Fastboot over UDP, isn't as useful:
> 
>   - the user first has to find out the IP address assigned to the board
>   - getting a DHCP lease on every boot takes a bit of time
>   - The shell is unusable until the DHCP request succeeds or times out.
>     Ctrl+C can't be used as it's not called interactively
> 
> Therefore, disable Fastboot over UDP and leave only USB enabled by
> default. User wishing to use Fastboot over UDP can just set
> 
>   nv.fastboot.net.autostart=1
> 
> manually to restore the old behavior.
> 
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
>  .../barebox-mx6-defaultenv/nv/fastboot.net.autostart             | 1 -
>  1 file changed, 1 deletion(-)
>  delete mode 100644 configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart
> 
> diff --git a/configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart b/configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart
> deleted file mode 100644
> index d00491fd7e5b..000000000000
> --- a/configs/platform-v7a/barebox-mx6-defaultenv/nv/fastboot.net.autostart
> +++ /dev/null
> @@ -1 +0,0 @@
> -1
> -- 
> 2.39.2
> 
> 
> 

-- 
Pengutronix e.K.                           | Dipl.-Ing. Robert Schwebel  |
Steuerwalder Str. 21                       | https://www.pengutronix.de/ |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-9    |



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-03-11 12:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-06 18:46 [DistroKit] [PATCH] v7a: barebox: imx6: don't enable fastboot net by default Ahmad Fatoum
2024-03-11 12:06 ` Robert Schwebel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox