[PATCH] Documentation: boards: stm32mp: document eMMC boot configuration

2020-08-28 Thread Bastian Krause
From: Ahmad Fatoum 

In order to boot from eMMC on the STM32MP15x, the boot_ack bit in the
ext_csd register must be set. Document this and while at it, fix a
misformatted code block in the same section.

Signed-off-by: Ahmad Fatoum 
Signed-off-by: Bastian Krause 
---
 Documentation/boards/stm32mp.rst | 13 -
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/Documentation/boards/stm32mp.rst b/Documentation/boards/stm32mp.rst
index 7c657eb990..b235c39927 100644
--- a/Documentation/boards/stm32mp.rst
+++ b/Documentation/boards/stm32mp.rst
@@ -64,7 +64,7 @@ An appropriate image for a SD-Card can be generated with 
following
   }
 
 For eMMC, the boot partitions are used as the FSBL partitions and so the user
-partitions may look like this:
+partitions may look like this::
 
   image @STM32MP_BOARD@.img {
   partition ssbl {
@@ -80,6 +80,17 @@ partitions may look like this:
 The fsbl1 and fsbl2 can be flashed by writing to barebox ``/dev/mmcX.boot0`` 
and
 ``/dev/mmcX.boot1`` respectively or from a booted operating system.
 
+Additionally, the eMMC's ``ext_csd`` register must be modified to activate the
+boot acknowledge signal (``BOOT_ACK``) and to select a boot partition.
+
+Assuming ``CONFIG_CMD_MMC_EXTCSD`` is enabled and the board shall boot from
+``/dev/mmc1.boot1``::
+
+  mmc_extcsd /dev/mmc1 -i 179 -v 0x50
+
+The STM32MP1 BootROM does *not* support booting from eMMC without fast boot
+acknowledge.
+
 Boot source selection
 -
 
-- 
2.28.0


___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


Re: [PATCH] commands: import memtester 4.3.0 from Debian GNU/Linux

2020-08-28 Thread Peter Mamonov
On Thu, Aug 27, 2020 at 10:45:03AM +0200, Roland Hieber wrote:
> On Wed, Aug 26, 2020 at 05:26:32PM +0300, Peter Mamonov wrote:
> > Memtester is an utility for testing the memory subsystem for faults.  For
> > hardware developers, memtester can be told to test memory starting at a
> > particular physical address.
> > 
> > This port is based on the sources from Debian GNU/Linux. Debian package meta
> > data is as follows:
> > 
> > Package: memtester
> > Version: 4.3.0-5
> > Homepage: http://pyropus.ca/software/memtester/
> > APT-Sources: http://ftp.ru.debian.org/debian testing/main amd64 Packages
> > 
> > Dissected version of this patch can be found at
> > https://github.com/pmamonov/barebox/commits/memtester and consists of two
> > patches:
> > 
> > commands: import memtester 4.3.0 sources from Debian GNU/Linux
> > commands: memtester: integrate it into barebox
> > 
> > Signed-off-by: Peter Mamonov 
> > ---
> >  commands/Kconfig   |   8 +
> >  commands/Makefile  |   1 +
> >  commands/memtester/Makefile|   1 +
> >  commands/memtester/memtester.c | 316 +++
> >  commands/memtester/memtester.h |  22 ++
> >  commands/memtester/sizes.h |  38 +++
> >  commands/memtester/tests.c | 538 +
> >  commands/memtester/tests.h |  37 +++
> >  commands/memtester/types.h |  36 +++
> >  9 files changed, 997 insertions(+)
> >  create mode 100644 commands/memtester/Makefile
> >  create mode 100644 commands/memtester/memtester.c
> >  create mode 100644 commands/memtester/memtester.h
> >  create mode 100644 commands/memtester/sizes.h
> >  create mode 100644 commands/memtester/tests.c
> >  create mode 100644 commands/memtester/tests.h
> >  create mode 100644 commands/memtester/types.h
> > 
> > diff --git a/commands/Kconfig b/commands/Kconfig
> > index 3789f33c3b..34c24f7d25 100644
> > --- a/commands/Kconfig
> > +++ b/commands/Kconfig
> > @@ -1590,6 +1590,14 @@ config CMD_MEMTEST
> >   -i ITERATIONS perform number of iterations (default 1, 0 is 
> > endless)
> >   -bperform only a test on bus lines
> >  
> > +config CMD_MEMTESTER
> > +   tristate
> > +   prompt "memtester"
> > +   help
> > + Utility for testing the memory subsystem.
> > +
> > + Homepage: http://pyropus.ca/software/memtester/
> > +
> >  config CMD_MM
> > tristate
> > select DEV_MEM
> > diff --git a/commands/Makefile b/commands/Makefile
> > index 01082de44c..191527c84e 100644
> > --- a/commands/Makefile
> > +++ b/commands/Makefile
> > @@ -49,6 +49,7 @@ obj-$(CONFIG_CMD_LOADENV) += loadenv.o
> >  obj-$(CONFIG_CMD_NAND) += nand.o
> >  obj-$(CONFIG_CMD_NANDTEST) += nandtest.o
> >  obj-$(CONFIG_CMD_MEMTEST)  += memtest.o
> > +obj-$(CONFIG_CMD_MEMTESTER)+= memtester/
> >  obj-$(CONFIG_CMD_TRUE) += true.o
> >  obj-$(CONFIG_CMD_FALSE)+= false.o
> >  obj-$(CONFIG_CMD_VERSION)  += version.o
> > diff --git a/commands/memtester/Makefile b/commands/memtester/Makefile
> > new file mode 100644
> > index 00..17a2429276
> > --- /dev/null
> > +++ b/commands/memtester/Makefile
> > @@ -0,0 +1 @@
> > +obj-y += tests.o memtester.o
> > diff --git a/commands/memtester/memtester.c b/commands/memtester/memtester.c
> > new file mode 100644
> > index 00..7be6a9c693
> > --- /dev/null
> > +++ b/commands/memtester/memtester.c
> > @@ -0,0 +1,316 @@
> > +/*
> > + * memtester version 4
> > + *
> > + * Very simple but very effective user-space memory tester.
> > + * Originally by Simon Kirby  
> > + * Version 2 by Charles Cazabon 
> > + * Version 3 not publicly released.
> > + * Version 4 rewrite:
> > + * Copyright (C) 2004-2012 Charles Cazabon 
> > + * Licensed under the terms of the GNU General Public License version 2 
> > (only).
> > + * See the file COPYING for details.
> > + *
> > + */
> 
> Please add SPDX identifiers to new files, see [1] for more info. In this
> case it should be
> 
> /* SPDX-License-Identifier: GPL-2.0-only */
> 
> You can then drop the "Licensed under the terms..." line.
> 
> [1]: 
> 

Ok, will do so in the next version.

Thanks,
Peter

> > +
> > +#define __version__ "4.3.0"
> > +
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +#include 
> > +
> > +#include "types.h"
> > +#include "sizes.h"
> > +#include "tests.h"
> > +
> > +#define EXIT_FAIL_NONSTARTER0x01
> > +#define EXIT_FAIL_ADDRESSLINES  0x02
> > +#define EXIT_FAIL_OTHERTEST 0x04
> > +
> > +struct test tests[] = {
> > +{ "Random Value", test_random_value },
> > +{ "Compare XOR", test_xor_comparison },
> > +{ "Compare SUB", test_sub_comparison },
> > +{ "Compare MUL", test_mul_comparison },
> > +{ "Compare DIV",test_div_comparison },
> > +{ "Compare OR", 

Re: [PATCH v2] sandbox: add a minimal defconfig to build only the host tools

2020-08-28 Thread Roland Hieber
On Fri, Aug 28, 2020 at 11:40:58AM +0300, Antony Pavlov wrote:
> On Mon, 24 Aug 2020 00:39:27 +0200
> Roland Hieber  wrote:
> 
> Hi!
> 
> > For distro packaging, make it possible to build all host tools in one go
> 
> Is there any stuff for Debian packaging?
> 
> There is u-boot-tools package for Debian distro.
> Alas there is no barebox-tools package still.

Current state is at .
ITP bug is at .

 - Roland

-- 
Roland Hieber, Pengutronix e.K.  | r.hie...@pengutronix.de |
Steuerwalder Str. 21 | https://www.pengutronix.de/ |
31137 Hildesheim, Germany| Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917- |

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


Re: [PATCH v2] sandbox: add a minimal defconfig to build only the host tools

2020-08-28 Thread Antony Pavlov
On Mon, 24 Aug 2020 00:39:27 +0200
Roland Hieber  wrote:

Hi!

> For distro packaging, make it possible to build all host tools in one go

Is there any stuff for Debian packaging?

There is u-boot-tools package for Debian distro.
Alas there is no barebox-tools package still.


> (and a very minimal barebox image as an unimportant side artifact).
> 
> Signed-off-by: Roland Hieber 
> ---
> v2:
>  * also select IMD to get scripts/bareboximd
> 
>  arch/sandbox/configs/hosttools_defconfig | 7 +++
>  1 file changed, 7 insertions(+)
>  create mode 100644 arch/sandbox/configs/hosttools_defconfig
> 
> diff --git a/arch/sandbox/configs/hosttools_defconfig 
> b/arch/sandbox/configs/hosttools_defconfig
> new file mode 100644
> index ..72ec0fc462b2
> --- /dev/null
> +++ b/arch/sandbox/configs/hosttools_defconfig
> @@ -0,0 +1,7 @@
> +CONFIG_IMD=y
> +CONFIG_COMPILE_HOST_TOOLS=y
> +CONFIG_ARCH_IMX_USBLOADER=y
> +CONFIG_MVEBU_HOSTTOOLS=y
> +CONFIG_MXS_HOSTTOOLS=y
> +CONFIG_OMAP3_USB_LOADER=y
> +CONFIG_OMAP4_HOSTTOOL_USBBOOT=y
> -- 
> 2.28.0
> 
> 
> ___
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox


-- 
Best regards,
  Antony Pavlov

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


[RFC] ARM: run barebox on STM32F429 MCU

2020-08-28 Thread Antony Pavlov
This commit demonstrates how to run barebox on STM32F429 MCU
with Cortex-M4 core. I suppose code can bee easely improved
to support other STM32 MCUs (STM32F7xx and STM32H7xx).

These MCU can run Linux, see
https://github.com/torvalds/linux/blob/master/arch/arm/mach-stm32/Kconfig#L22

Also U-Boot runs on these MCUs, see
https://github.com/u-boot/u-boot/tree/master/arch/arm/mach-stm32

32F429IDISCOVERY Discovery kit (stm32f429i-disc1) is used
for barebox demonstration, see
https://www.st.com/en/evaluation-tools/32f429idiscovery.html

The patch makes several ugly hacks in barebox arm Makefiles.
These hacks have to be eliminated.

Barebox runs from internal MCU SRAM, no internal ROM or
external RAM is used. The only supported hardware is UART
(thanks to drivers/serial/serial_stm32.c).
pinctrl setup is made by ad-hoc functions imported from libopencm3.
No hardware clocksource is used.

BUILD and RUN HOWTO
===

   git clone -b 20200827.stm32 https://github.com/frantony/barebox
   cd barebox
   export ARCH=arm
   export CROSS_COMPILE=arm-none-eabi-
   make stm32_defconfig
   make

Resulting file is 'barebox'.

Connect to stm32f429i-disc1' STLink via USB.
Run openocd in a separate shell:

   sudo openocd -f interface/stlink-v2.cfg -c "transport select hla_swd"
-f target/stm32f4x.cfg -c init

Next load barebox into MCU's SRAM and run it:

   gdb-multiarch -x gdb.conf

TODO


  [ ] separate debug_ll for stm32f4 uart
  [ ] stack setup (at the moment gdb script sets up stack registers)
  [ ] support clocksource (see st,stm32-timer/ARMV7M_SYSTICK)
  [ ] separate stm32mp and stm32f, rename arch/arm/mach-stm32mp to 
arch/arm/mach-stm32
  [ ] add options for cache/nommu/-mcpu=cortex-m4 ...,
  e.g. see linux kernel options
  * CPU_THUMBONLY
  * CPU_V7M
  * CPU_CACHE_V7M
  * ARMV7M_SYSTICK
  [ ] use st,stm32-rcc
  [ ] support external SDRAM on the board
  [ ] use gpio driver
  [ ] internal MCU flash support

Signed-off-by: Antony Pavlov 
---
 arch/arm/Kconfig  |   2 +-
 arch/arm/Makefile |   2 +-
 arch/arm/configs/stm32_defconfig  |  55 +++
 arch/arm/cpu/Makefile |   8 +-
 arch/arm/cpu/no-mmu.c |   2 +-
 arch/arm/cpu/start.c  | 325 +-
 arch/arm/dts/Makefile |   2 +
 arch/arm/dts/stm32f429-disco.dts  |  72 
 arch/arm/dts/stm32f429.dtsi   |  68 
 arch/arm/mach-stm32mp/Kconfig |   3 +
 arch/arm/mach-stm32mp/Makefile|   4 +-
 arch/arm/mach-stm32mp/include/mach/debug_ll.h |  21 ++
 drivers/clocksource/Makefile  |   2 +-
 gdb.conf  |   7 +
 14 files changed, 556 insertions(+), 17 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 95fd8ecfe7..cebbc05d17 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -205,7 +205,7 @@ config ARCH_STM32MP
select ARCH_HAS_RESET_CONTROLLER
select ARM_AMBA
select ARM_SMCCC
-   select ARM_USE_COMPRESSED_DTB
+   select HAVE_CONFIGURABLE_MEMORY_LAYOUT
 
 config ARCH_VERSATILE
bool "ARM Versatile boards (ARM926EJ-S)"
diff --git a/arch/arm/Makefile b/arch/arm/Makefile
index 96613cc5ba..7facda819d 100644
--- a/arch/arm/Makefile
+++ b/arch/arm/Makefile
@@ -40,7 +40,7 @@ endif
 # macro, but instead defines a whole series of macros which makes
 # testing for a specific architecture or later rather impossible.
 arch-$(CONFIG_CPU_64v8):= -D__LINUX_ARM_ARCH__=8 $(call 
cc-option,-march=armv8-a)
-arch-$(CONFIG_CPU_32v7):=-D__LINUX_ARM_ARCH__=7 $(call 
cc-option,-march=armv7-a,-march=armv5t -Wa$(comma)-march=armv7-a)
+arch-$(CONFIG_CPU_32v7):=-D__LINUX_ARM_ARCH__=7 -mcpu=cortex-m4
 arch-$(CONFIG_CPU_32v6):=-D__LINUX_ARM_ARCH__=6 $(call 
cc-option,-march=armv6,-march=armv5t -Wa$(comma)-march=armv6)
 arch-$(CONFIG_CPU_32v5):=-D__LINUX_ARM_ARCH__=5 $(call 
cc-option,-march=armv5te,-march=armv4t)
 arch-$(CONFIG_CPU_32v4T)   :=-D__LINUX_ARM_ARCH__=4 -march=armv4t
diff --git a/arch/arm/configs/stm32_defconfig b/arch/arm/configs/stm32_defconfig
new file mode 100644
index 00..07fec1b9a2
--- /dev/null
+++ b/arch/arm/configs/stm32_defconfig
@@ -0,0 +1,55 @@
+CONFIG_TEXT_BASE=0x2000
+CONFIG_ARCH_STM32MP=y
+CONFIG_BOARD_STM32F429I_DISC1=y
+CONFIG_THUMB2_BAREBOX=y
+# CONFIG_ARM_EXCEPTIONS is not set
+CONFIG_ARM_UNWIND=y
+CONFIG_IMAGE_COMPRESSION_NONE=y
+CONFIG_MEMORY_LAYOUT_FIXED=y
+CONFIG_STACK_BASE=0x2002c000
+CONFIG_STACK_SIZE=0x4000
+CONFIG_MALLOC_BASE=0x1000
+CONFIG_MALLOC_SIZE=0x1
+CONFIG_MALLOC_TLSF=y
+CONFIG_KALLSYMS=y
+CONFIG_CMDLINE_EDITING=y
+CONFIG_AUTO_COMPLETE=y
+# CONFIG_TIMESTAMP is not set
+CONFIG_BLSPEC=y
+CONFIG_FLEXIBLE_BOOTARGS=y
+CONFIG_POLLER=y
+CONFIG_RESET_SOURCE=y
+CONFIG_DEBUG_LL=y

Re: [PATCH] commands: import memtester 4.3.0 from Debian GNU/Linux

2020-08-28 Thread Sascha Hauer
Hi Peter,

On Wed, Aug 26, 2020 at 05:26:32PM +0300, Peter Mamonov wrote:
> diff --git a/commands/memtester/memtester.c b/commands/memtester/memtester.c
> new file mode 100644
> index 00..7be6a9c693
> --- /dev/null
> +++ b/commands/memtester/memtester.c

Is this file original memtester code or have you written it for barebox?
It looks like originally memtester but not much is left from it. Could
you put the barebox part into a separate (new) file for easier future
updates?

> @@ -0,0 +1,316 @@
> +/*
> + * memtester version 4
> + *
> + * Very simple but very effective user-space memory tester.
> + * Originally by Simon Kirby  
> + * Version 2 by Charles Cazabon 
> + * Version 3 not publicly released.
> + * Version 4 rewrite:
> + * Copyright (C) 2004-2012 Charles Cazabon 
> + * Licensed under the terms of the GNU General Public License version 2 
> (only).
> + * See the file COPYING for details.
> + *
> + */
> +
> +#define __version__ "4.3.0"
> +
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +
> +#include "types.h"
> +#include "sizes.h"
> +#include "tests.h"
> +
> +#define EXIT_FAIL_NONSTARTER0x01
> +#define EXIT_FAIL_ADDRESSLINES  0x02
> +#define EXIT_FAIL_OTHERTEST 0x04
> +
> +struct test tests[] = {
> +{ "Random Value", test_random_value },
> +{ "Compare XOR", test_xor_comparison },
> +{ "Compare SUB", test_sub_comparison },
> +{ "Compare MUL", test_mul_comparison },
> +{ "Compare DIV",test_div_comparison },
> +{ "Compare OR", test_or_comparison },
> +{ "Compare AND", test_and_comparison },
> +{ "Sequential Increment", test_seqinc_comparison },
> +{ "Solid Bits", test_solidbits_comparison },
> +{ "Block Sequential", test_blockseq_comparison },
> +{ "Checkerboard", test_checkerboard_comparison },
> +{ "Bit Spread", test_bitspread_comparison },
> +{ "Bit Flip", test_bitflip_comparison },
> +{ "Walking Ones", test_walkbits1_comparison },
> +{ "Walking Zeroes", test_walkbits0_comparison },
> +{ "8-bit Writes", test_8bit_wide_random },
> +{ "16-bit Writes", test_16bit_wide_random },
> +{ NULL, NULL }
> +};

Should be static

> +
> +/* Function declarations */
> +
> +/* Global vars - so tests have access to this information */
> +int use_phys = 0;
> +off_t physaddrbase = 0;
> +
> +static int do_memtester(int argc, char **argv) {
> +ul loops, loop, i;
> +size_t wantraw, wantmb, wantbytes, wantbytes_orig, bufsize,
> + halflen, count;
> +char *memsuffix, *addrsuffix, *loopsuffix;
> +void volatile *buf, *aligned;
> +ulv *bufa, *bufb;
> +int exit_code = 0, ret;
> +int memfd = 0, opt, memshift;
> +size_t maxbytes = -1; /* addressable memory, in bytes */
> +size_t maxmb = (maxbytes >> 20) + 1; /* addressable memory, in MB */
> +/* Device to mmap memory from with -p, default is normal core */
> +char *device_name = "/dev/mem";
> +struct stat statbuf;
> +int device_specified = 0;
> +const char *env_testmask = 0;
> +ul testmask = 0;
> +
> +printf("memtester version " __version__ " (%d-bit)\n", UL_LEN);
> +printf("Copyright (C) 2001-2012 Charles Cazabon.\n");
> +printf("Licensed under the GNU General Public License version 2 
> (only).\n");
> +printf("\n");
> +
> +/* If MEMTESTER_TEST_MASK is set, we use its value as a mask of which
> +   tests we run.
> + */
> +if ((env_testmask = getenv("MEMTESTER_TEST_MASK"))) {i

Why is this an envrionment variable? I would expect this to be a
commandline option.

> +errno = 0;
> +testmask = simple_strtoul(env_testmask, 0, 0);
> +if (errno) {
> +printf("error parsing MEMTESTER_TEST_MASK %s: %s\n",
> +env_testmask, strerror(errno));
> +return COMMAND_ERROR_USAGE;
> +}
> +printf("using testmask 0x%lx\n", testmask);
> +}
> +
> +while ((opt = getopt(argc, argv, "p:d:")) != -1) {
> +switch (opt) {
> +case 'p':
> +errno = 0;
> +physaddrbase = (off_t) simple_strtoull(optarg, , 
> 16);
> +if (errno != 0) {
> +printf("failed to parse physaddrbase arg; should be hex "
> +   "address (0x123...)\n");
> +return COMMAND_ERROR_USAGE;
> +}
> +if (*addrsuffix != '\0') {
> +/* got an invalid character in the address */
> +printf("failed to parse physaddrbase arg; should be hex "
> +   "address (0x123...)\n");
> +return COMMAND_ERROR_USAGE;
> +}
> +/* okay, got address */
> +use_phys = 1;
> +break;
> +case 'd':
> +if (stat(optarg,)) {
> +printf("can 

Re: [PATCH v2] sandbox: add a minimal defconfig to build only the host tools

2020-08-28 Thread Sascha Hauer
On Mon, Aug 24, 2020 at 12:39:27AM +0200, Roland Hieber wrote:
> For distro packaging, make it possible to build all host tools in one go
> (and a very minimal barebox image as an unimportant side artifact).
> 
> Signed-off-by: Roland Hieber 
> ---
> v2:
>  * also select IMD to get scripts/bareboximd
> 
>  arch/sandbox/configs/hosttools_defconfig | 7 +++
>  1 file changed, 7 insertions(+)
>  create mode 100644 arch/sandbox/configs/hosttools_defconfig

Applied, thanks

Sascha

> 
> diff --git a/arch/sandbox/configs/hosttools_defconfig 
> b/arch/sandbox/configs/hosttools_defconfig
> new file mode 100644
> index ..72ec0fc462b2
> --- /dev/null
> +++ b/arch/sandbox/configs/hosttools_defconfig
> @@ -0,0 +1,7 @@
> +CONFIG_IMD=y
> +CONFIG_COMPILE_HOST_TOOLS=y
> +CONFIG_ARCH_IMX_USBLOADER=y
> +CONFIG_MVEBU_HOSTTOOLS=y
> +CONFIG_MXS_HOSTTOOLS=y
> +CONFIG_OMAP3_USB_LOADER=y
> +CONFIG_OMAP4_HOSTTOOL_USBBOOT=y
> -- 
> 2.28.0
> 
> 
> ___
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

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

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


Re: [PATCH] common: remove negative dependency on SANDBOX for IMD_TARGET

2020-08-28 Thread Sascha Hauer
On Mon, Aug 24, 2020 at 12:30:53AM +0200, Roland Hieber wrote:
> This dependency was added in commit b3d5c43cb8a2d492355a (2016-04-07,
> Antony Pavlov: "common: add dependency !SANDBOX on imd target tool"),
> but it seems to compile fine four years down the line.
> 
> Signed-off-by: Roland Hieber 
> ---
>  common/Kconfig | 1 -
>  1 file changed, 1 deletion(-)

Applied, thanks

Sascha

> 
> diff --git a/common/Kconfig b/common/Kconfig
> index 658437f01c5e..9e2417569bd5 100644
> --- a/common/Kconfig
> +++ b/common/Kconfig
> @@ -699,7 +699,6 @@ config IMD
>  config IMD_TARGET
>   bool "build bareboximd target tool"
>   depends on IMD
> - depends on !SANDBOX
>  
>  config KERNEL_INSTALL_TARGET
>   bool
> -- 
> 2.28.0
> 
> 
> ___
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

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

___
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox