diff mbox series

net: fastboot: make UDP port net: configurable

Message ID 20220113074016.15163-1-christian.gmeiner@gmail.com
State Accepted
Commit 046bf8d4c512ad6501fe9e77508bbe1292a29fef
Delegated to: Ramon Fried
Headers show
Series net: fastboot: make UDP port net: configurable | expand

Commit Message

Christian Gmeiner Jan. 13, 2022, 7:40 a.m. UTC
The fastboot protocol uses per default the UDP port 5554. In some cases
it might be needed to change the used port. The fastboot utility provides
a way to specifiy an other port number to use already.

  fastboot -s udp:192.168.1.76:1234 boot fastboot.img

Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
---
 drivers/fastboot/Kconfig | 7 +++++++
 net/fastboot.c           | 5 +----
 2 files changed, 8 insertions(+), 4 deletions(-)

Comments

Heiko Schocher Jan. 13, 2022, 12:02 p.m. UTC | #1
Hello Christian,

On 13.01.22 08:40, Christian Gmeiner wrote:
> The fastboot protocol uses per default the UDP port 5554. In some cases
> it might be needed to change the used port. The fastboot utility provides
> a way to specifiy an other port number to use already.
> 
>   fastboot -s udp:192.168.1.76:1234 boot fastboot.img
> 
> Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> ---
>  drivers/fastboot/Kconfig | 7 +++++++
>  net/fastboot.c           | 5 +----
>  2 files changed, 8 insertions(+), 4 deletions(-)

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko
Ramon Fried Jan. 15, 2022, 11:58 a.m. UTC | #2
On Thu, Jan 13, 2022 at 2:02 PM Heiko Schocher <hs@denx.de> wrote:
>
> Hello Christian,
>
> On 13.01.22 08:40, Christian Gmeiner wrote:
> > The fastboot protocol uses per default the UDP port 5554. In some cases
> > it might be needed to change the used port. The fastboot utility provides
> > a way to specifiy an other port number to use already.
> >
> >   fastboot -s udp:192.168.1.76:1234 boot fastboot.img
> >
> > Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> > ---
> >  drivers/fastboot/Kconfig | 7 +++++++
> >  net/fastboot.c           | 5 +----
> >  2 files changed, 8 insertions(+), 4 deletions(-)
>
> Reviewed-by: Heiko Schocher <hs@denx.de>
>
> bye,
> Heiko
> --
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
Ramon Fried Jan. 15, 2022, 4:54 p.m. UTC | #3
On Sat, Jan 15, 2022 at 1:58 PM Ramon Fried <rfried.dev@gmail.com> wrote:
>
> On Thu, Jan 13, 2022 at 2:02 PM Heiko Schocher <hs@denx.de> wrote:
> >
> > Hello Christian,
> >
> > On 13.01.22 08:40, Christian Gmeiner wrote:
> > > The fastboot protocol uses per default the UDP port 5554. In some cases
> > > it might be needed to change the used port. The fastboot utility provides
> > > a way to specifiy an other port number to use already.
> > >
> > >   fastboot -s udp:192.168.1.76:1234 boot fastboot.img
> > >
> > > Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> > > ---
> > >  drivers/fastboot/Kconfig | 7 +++++++
> > >  net/fastboot.c           | 5 +----
> > >  2 files changed, 8 insertions(+), 4 deletions(-)
> >
> > Reviewed-by: Heiko Schocher <hs@denx.de>
> >
> > bye,
> > Heiko
> > --
> > DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> > Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de
> Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
Applied to u-boot-net/next
Thanks !
diff mbox series

Patch

diff --git a/drivers/fastboot/Kconfig b/drivers/fastboot/Kconfig
index d5e4a02098..b97c67bf60 100644
--- a/drivers/fastboot/Kconfig
+++ b/drivers/fastboot/Kconfig
@@ -21,6 +21,13 @@  config UDP_FUNCTION_FASTBOOT
 	help
 	  This enables the fastboot protocol over UDP.
 
+config UDP_FUNCTION_FASTBOOT_PORT
+	depends on UDP_FUNCTION_FASTBOOT
+	int "Define FASTBOOT UDP port"
+	default 5554
+	help
+	  The fastboot protocol requires a UDP port number.
+
 if FASTBOOT
 
 config FASTBOOT_BUF_ADDR
diff --git a/net/fastboot.c b/net/fastboot.c
index 7e7a601b9f..139233b86c 100644
--- a/net/fastboot.c
+++ b/net/fastboot.c
@@ -9,9 +9,6 @@ 
 #include <net.h>
 #include <net/fastboot.h>
 
-/* Fastboot port # defined in spec */
-#define WELL_KNOWN_PORT 5554
-
 enum {
 	FASTBOOT_ERROR = 0,
 	FASTBOOT_QUERY = 1,
@@ -310,7 +307,7 @@  void fastboot_start_server(void)
 	printf("Using %s device\n", eth_get_name());
 	printf("Listening for fastboot command on %pI4\n", &net_ip);
 
-	fastboot_our_port = WELL_KNOWN_PORT;
+	fastboot_our_port = CONFIG_UDP_FUNCTION_FASTBOOT_PORT;
 
 #if CONFIG_IS_ENABLED(FASTBOOT_FLASH)
 	fastboot_set_progress_callback(fastboot_timed_send_info);