All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Kiernan <alex.kiernan@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH v1 0/5] Add fastboot UDP support
Date: Tue, 24 Apr 2018 09:37:02 +0000	[thread overview]
Message-ID: <1524562627-5794-1-git-send-email-alex.kiernan@gmail.com> (raw)


This series merges the fastboot UDP support from AOSP into mainline
U-Boot.

Open questions:

- what's the correct way of attributing the original authors? I've added
  Co-authored-by, is that right? checkpatch doesn't seem to know any
  of the co- tags
- currently there's no NAND support and I've no way of testing that, so
  my inclination is towards leaving it like that until someone with that
  particular itch to scratch can look at it
- you can select both USB and UDP fastboot, but the comments in the
  AOSP code suggest that needs fixing - again, I've no board I can test
  USB fastboot on
- the USB and UDP code want consolidating, with this series there would
  then be two separate implementations of the same protocol
- the syntax for the USB fastboot command changes from `fastboot <controller>`
  to `fastboot usb <controller>`, that feels unacceptable and we probably
  want something like `fastboot <controller>` or `fastboot udp`?
- unrelated to this series, but a show-stopper for me, there's no FIT image
  support, but that doesn't feel unsolveable - something like add an option
  to pass in the load address and/or use loadaddr, then something else to
  let you override the bootm invocation on the server side

I've not tested all the code paths yet, but the obvious stuff works
(basic interaction, download, boot) - every interaction elicits a
`WARNING: unknown variable: slot-count` on the console; I'm guessing that
my local end is sending a getvar for that, but I've not investigated.

Without any way of testing any of the USB/NAND code I'm nervous of wading
into that kind of refactoring, would that be a pre-requisite for merging?


Alex Kiernan (5):
  dfu: Refactor fastboot_okay/fail to take response
  dfu: Extract fastboot_okay/fail to fb_common.c
  net: dfu: Merge AOSP UDP fastboot
  dfu: Resolve Kconfig dependency loops
  net: dfu: Support building without MMC

 cmd/fastboot.c                  |  32 ++-
 cmd/fastboot/Kconfig            |  21 +-
 cmd/net.c                       |   6 +
 common/Makefile                 |   4 +
 common/fb_common.c              |  44 ++++
 common/fb_mmc.c                 | 114 ++++++---
 common/fb_nand.c                |  31 +--
 common/image-sparse.c           |  41 ++-
 drivers/usb/gadget/f_fastboot.c |  36 +--
 include/fastboot.h              |  17 +-
 include/fb_mmc.h                |   4 +-
 include/fb_nand.h               |   4 +-
 include/image-sparse.h          |   2 +-
 include/net.h                   |   6 +-
 include/net/fastboot.h          |  27 ++
 net/Makefile                    |   1 +
 net/fastboot.c                  | 548 ++++++++++++++++++++++++++++++++++++++++
 net/net.c                       |   9 +
 18 files changed, 824 insertions(+), 123 deletions(-)
 create mode 100644 common/fb_common.c
 create mode 100644 include/net/fastboot.h
 create mode 100644 net/fastboot.c

-- 
2.7.4

             reply	other threads:[~2018-04-24  9:37 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-24  9:37 Alex Kiernan [this message]
2018-04-24  9:37 ` [U-Boot] [RFC PATCH v1 1/5] dfu: Refactor fastboot_okay/fail to take response Alex Kiernan
2018-04-25  7:27   ` Lukasz Majewski
2018-04-25  8:10     ` Alex Kiernan
2018-04-24  9:37 ` [U-Boot] [RFC PATCH v1 2/5] dfu: Extract fastboot_okay/fail to fb_common.c Alex Kiernan
2018-04-25  7:32   ` Lukasz Majewski
2018-04-25  8:15     ` Alex Kiernan
2018-04-24  9:37 ` [U-Boot] [RFC PATCH v1 3/5] net: dfu: Merge AOSP UDP fastboot Alex Kiernan
2018-04-24  9:37 ` [U-Boot] [RFC PATCH v1 4/5] dfu: Resolve Kconfig dependency loops Alex Kiernan
2018-04-25  7:53   ` Lukasz Majewski
2018-04-25  8:55     ` Alex Deymo
2018-04-24  9:37 ` [U-Boot] [RFC PATCH v1 5/5] net: dfu: Support building without MMC Alex Kiernan
2018-04-24 10:24 ` [U-Boot] [RFC PATCH v1 0/5] Add fastboot UDP support Alex Deymo
2018-04-24 11:58   ` Alex Kiernan
2018-04-24 17:10     ` Jocelyn Bohr
2018-04-25  7:53       ` Alex Kiernan
2018-04-25 12:50         ` Alex Kiernan
2018-04-25 15:19           ` Lukasz Majewski
2018-04-26  3:27       ` Kever Yang
2018-04-27  8:04         ` Lukasz Majewski
2018-04-27 12:20           ` Alex Kiernan
2018-04-30  8:37             ` Alex Kiernan
2018-05-01  6:16               ` Jocelyn Bohr
2018-04-25  7:40     ` Lukasz Majewski
2018-04-25  7:52 ` Lukasz Majewski
2018-04-25  7:57   ` Lukasz Majewski
2018-04-25  8:43   ` Alex Kiernan
2018-04-25  8:46     ` Alex Deymo
2018-04-27 19:10 ` Sam Protsenko
2018-04-27 19:23   ` Alex Kiernan
2018-04-30  8:56   ` Alex Deymo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1524562627-5794-1-git-send-email-alex.kiernan@gmail.com \
    --to=alex.kiernan@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.