From: Stefan Agner <stefan@agner.ch>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 0/3] spl: USB Gadget cleanup
Date: Tue, 15 Nov 2016 13:02:43 -0800 [thread overview]
Message-ID: <20161115210246.2245-1-stefan@agner.ch> (raw)
From: Stefan Agner <stefan.agner@toradex.com>
This patchset cleans up the SPL USB Gadget Kconfig and Makfile.
The first two patches are actual fixes and we might consider to
add these two for the v2016.11 release. The third one adds a new
config option for cleaner structure. I plan to add another USB
function with SPL support which is the reason I looked into
this...
Build tested using buildman for all boards on powerpc mips and
arm. No new errors or warnings:
Building 4 commits for 1072 boards (8 threads, 1 job per thread)
4044 0 244 /4288 P1022DS_NAND
This patchset is probably most interesting for
am335x_evm_usbspl_defconfig (CONFIG_SPL_USBETH_SUPPORT)
And the board with which SPL DFU support has been introduced
dra7xx_evm_defconfig (CONFIG_SPL_DFU_SUPPORT, not set by default)
Due to lack of any of this hardware I did not run the binaries!
Since it is mostly Makefile shuffling I don't expect any runtime
behavior changes... If anyone (maybe Ravi or Lukasz, afaik you
worked on the SPL DFU support for dra7xx_evm) could test the
patchset on actual hardware I would be glad!
--
Stefan
Changes in v3:
- Fix depends on SPL_USB_GADGET_SUPPORT and use if statement
Changes in v2:
- Use CONFIG_SPL_RAM_SUPPORT to descide whether to compile the
function in first place.
- Leave DWC3 config in top level SPL Makefile (as U-Boot does)
Stefan Agner (3):
spl: add RAM boot device only if it is actually defined
spl: dfu: move DFU Kconfig to SPL Kconfig
spl: add USB Gadget config option
Kconfig | 27 ------------------
common/spl/Kconfig | 57 ++++++++++++++++++++++++++++++-------
common/spl/spl.c | 4 ++-
configs/am335x_evm_usbspl_defconfig | 1 +
drivers/Makefile | 4 ++-
drivers/usb/gadget/Makefile | 9 ++++--
include/configs/dra7xx_evm.h | 1 -
scripts/Makefile.spl | 5 +---
8 files changed, 61 insertions(+), 47 deletions(-)
--
2.10.2
next reply other threads:[~2016-11-15 21:02 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-15 21:02 Stefan Agner [this message]
2016-11-15 21:02 ` [U-Boot] [PATCH v3 1/3] spl: add RAM boot device only if it is actually defined Stefan Agner
2016-11-16 13:44 ` Lukasz Majewski
2016-11-18 1:14 ` Simon Glass
2016-11-18 1:27 ` Stefan Agner
2016-11-19 13:48 ` Simon Glass
2016-11-15 21:02 ` [U-Boot] [PATCH v3 2/3] spl: dfu: move DFU Kconfig to SPL Kconfig Stefan Agner
2016-11-15 21:34 ` Brüns, Stefan
2016-11-16 14:06 ` Lukasz Majewski
2016-11-15 21:02 ` [U-Boot] [PATCH v3 3/3] spl: add USB Gadget config option Stefan Agner
2016-11-16 14:31 ` Lukasz Majewski
2016-11-16 7:04 ` [U-Boot] [PATCH v3 0/3] spl: USB Gadget cleanup B, Ravi
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=20161115210246.2245-1-stefan@agner.ch \
--to=stefan@agner.ch \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox