* [Buildroot] [PATCH] imx_usb_loader: Allow target build
@ 2018-05-16 9:16 Vincent Prince
2018-05-19 12:29 ` Thomas Petazzoni
2018-05-19 20:43 ` Thomas Petazzoni
0 siblings, 2 replies; 4+ messages in thread
From: Vincent Prince @ 2018-05-16 9:16 UTC (permalink / raw)
To: buildroot
Signed-off-by: Vincent Prince <vincent.prince.fr@gmail.com>
---
package/Config.in | 1 +
package/imx-usb-loader/Config.in | 10 ++++++++++
package/imx-usb-loader/imx-usb-loader.mk | 8 ++++++++
3 files changed, 19 insertions(+)
create mode 100644 package/imx-usb-loader/Config.in
diff --git a/package/Config.in b/package/Config.in
index 2be6c64..3cf53ed 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -186,6 +186,7 @@ menu "Filesystem and flash utilities"
source "package/genext2fs/Config.in"
source "package/genpart/Config.in"
source "package/genromfs/Config.in"
+ source "package/imx-usb-loader/Config.in"
source "package/mfgtools/Config.in"
source "package/mmc-utils/Config.in"
source "package/mtd/Config.in"
diff --git a/package/imx-usb-loader/Config.in b/package/imx-usb-loader/Config.in
new file mode 100644
index 0000000..41585e2
--- /dev/null
+++ b/package/imx-usb-loader/Config.in
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_IMX_USB_LOADER
+ bool "imx-usb-loader"
+ depends on BR2_arm
+ select BR2_PACKAGE_LIBUSB
+ help
+ This package contains tools to download and execute code
+ on Freescale i.MX5/i.MX6 and Vybrid SoCs through the
+ Serial Download Protocol.
+
+ https://github.com/boundarydevices/imx_usb_loader
diff --git a/package/imx-usb-loader/imx-usb-loader.mk b/package/imx-usb-loader/imx-usb-loader.mk
index 946a1f3..22698dd 100644
--- a/package/imx-usb-loader/imx-usb-loader.mk
+++ b/package/imx-usb-loader/imx-usb-loader.mk
@@ -8,14 +8,22 @@ IMX_USB_LOADER_VERSION = 7e77787c83288a421080a9bf2f67063931f561be
IMX_USB_LOADER_SITE = $(call github,boundarydevices,imx_usb_loader,$(IMX_USB_LOADER_VERSION))
IMX_USB_LOADER_LICENSE = LGPL-2.1+
IMX_USB_LOADER_LICENSE_FILES = COPYING
+IMX_USB_LOADER_DEPENDENCIES = libusb
HOST_IMX_USB_LOADER_DEPENDENCIES = host-libusb host-pkgconf
+define IMX_USB_LOADER_BUILD_CMDS
+ $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D)
+endef
define HOST_IMX_USB_LOADER_BUILD_CMDS
$(HOST_CONFIGURE_OPTS) $(MAKE) -C $(@D)
endef
+define IMX_USB_LOADER_INSTALL_TARGET_CMDS
+ $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) prefix=$(TARGET_DIR) install
+endef
define HOST_IMX_USB_LOADER_INSTALL_CMDS
$(HOST_CONFIGURE_OPTS) $(MAKE) -C $(@D) prefix=$(HOST_DIR) install
endef
+$(eval $(generic-package))
$(eval $(host-generic-package))
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] imx_usb_loader: Allow target build
2018-05-16 9:16 [Buildroot] [PATCH] imx_usb_loader: Allow target build Vincent Prince
@ 2018-05-19 12:29 ` Thomas Petazzoni
2018-05-19 17:34 ` Vincent Prince
2018-05-19 20:43 ` Thomas Petazzoni
1 sibling, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2018-05-19 12:29 UTC (permalink / raw)
To: buildroot
Hello,
On Wed, 16 May 2018 11:16:05 +0200, Vincent Prince wrote:
> Signed-off-by: Vincent Prince <vincent.prince.fr@gmail.com>
Could you give some details on the use case for having a target package
for the imx_usb_loader ? Are you building a system with Buildroot that
itself is responsible for loading a bootloader over USB to a separate
i.MX6 target ? If not, what is your use case ?
> diff --git a/package/imx-usb-loader/Config.in b/package/imx-usb-loader/Config.in
> new file mode 100644
> index 0000000..41585e2
> --- /dev/null
> +++ b/package/imx-usb-loader/Config.in
> @@ -0,0 +1,10 @@
> +config BR2_PACKAGE_IMX_USB_LOADER
> + bool "imx-usb-loader"
> + depends on BR2_arm
Why do you have this dependency ? If your use case is as explained
above to build a Buildroot system that itself will be responsible for
loading over USB the bootloader of a separate i.MX6 device, then this
system may be running on something else than an ARM platform.
> + select BR2_PACKAGE_LIBUSB
Indentation is wrong for this line, it should be indented with one tab.
I can fix this up when applying once the use case is clearer.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] imx_usb_loader: Allow target build
2018-05-19 12:29 ` Thomas Petazzoni
@ 2018-05-19 17:34 ` Vincent Prince
0 siblings, 0 replies; 4+ messages in thread
From: Vincent Prince @ 2018-05-19 17:34 UTC (permalink / raw)
To: buildroot
Hi Thomas,
For automatic testing purpose, I use an intermediate target (arm) between
host and DUT.
It uses imx_usb_loader and otp_com (https://github.com/ixonos/utp_com) as
described here:
https://community.nxp.com/thread/441563 to flash imx6 DUT with labgrid.
Indeed, extra arm depends is a mistake.
Sorry for indentation,
Best regards,
Vincent
2018-05-19 14:29 GMT+02:00 Thomas Petazzoni <thomas.petazzoni@bootlin.com>:
> Hello,
>
> On Wed, 16 May 2018 11:16:05 +0200, Vincent Prince wrote:
> > Signed-off-by: Vincent Prince <vincent.prince.fr@gmail.com>
>
> Could you give some details on the use case for having a target package
> for the imx_usb_loader ? Are you building a system with Buildroot that
> itself is responsible for loading a bootloader over USB to a separate
> i.MX6 target ? If not, what is your use case ?
>
> > diff --git a/package/imx-usb-loader/Config.in b/package/imx-usb-loader/
> Config.in
> > new file mode 100644
> > index 0000000..41585e2
> > --- /dev/null
> > +++ b/package/imx-usb-loader/Config.in
> > @@ -0,0 +1,10 @@
> > +config BR2_PACKAGE_IMX_USB_LOADER
> > + bool "imx-usb-loader"
> > + depends on BR2_arm
>
> Why do you have this dependency ? If your use case is as explained
> above to build a Buildroot system that itself will be responsible for
> loading over USB the bootloader of a separate i.MX6 device, then this
> system may be running on something else than an ARM platform.
>
> > + select BR2_PACKAGE_LIBUSB
>
> Indentation is wrong for this line, it should be indented with one tab.
> I can fix this up when applying once the use case is clearer.
>
> Best regards,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180519/07e33e5a/attachment.html>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] imx_usb_loader: Allow target build
2018-05-16 9:16 [Buildroot] [PATCH] imx_usb_loader: Allow target build Vincent Prince
2018-05-19 12:29 ` Thomas Petazzoni
@ 2018-05-19 20:43 ` Thomas Petazzoni
1 sibling, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2018-05-19 20:43 UTC (permalink / raw)
To: buildroot
Hello,
On Wed, 16 May 2018 11:16:05 +0200, Vincent Prince wrote:
> Signed-off-by: Vincent Prince <vincent.prince.fr@gmail.com>
> ---
> package/Config.in | 1 +
> package/imx-usb-loader/Config.in | 10 ++++++++++
> package/imx-usb-loader/imx-usb-loader.mk | 8 ++++++++
> 3 files changed, 19 insertions(+)
> create mode 100644 package/imx-usb-loader/Config.in
I've applied to next, after doing the following changes:
[Thomas:
- improve commit log
- fix indentation
- drop BR2_arm dependency
- add missing host-pkgconf dependency
- fix prefix variable to install in $(TARGET_DIR)/usr]
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-05-19 20:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-16 9:16 [Buildroot] [PATCH] imx_usb_loader: Allow target build Vincent Prince
2018-05-19 12:29 ` Thomas Petazzoni
2018-05-19 17:34 ` Vincent Prince
2018-05-19 20:43 ` Thomas Petazzoni
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.