From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heinrich Schuchardt Date: Mon, 8 Feb 2021 18:56:35 +0100 Subject: [PATCH v5 2/4] button: add a simple Analog to Digital Converter device based button driver In-Reply-To: References: <20210126095036.6429-1-m.szyprowski@samsung.com> <20210126095036.6429-3-m.szyprowski@samsung.com> <1d5b1718-9b28-61dd-8137-3a50d9ec2e6e@samsung.com> <5e353225-608f-5790-cbe6-2f12cb2b7ca9@samsung.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 2/8/21 6:08 PM, Simon Glass wrote: > HI Marek, > > On Mon, 8 Feb 2021 at 09:10, Marek Szyprowski wrote: >> >> Hi Simon, >> >> On 06.02.2021 17:21, Simon Glass wrote: >>> On Thu, 4 Feb 2021 at 03:36, Marek Szyprowski wrote: >>>> ... >>>> Could you give me a bit more hints or point where to start? I've tried >>>> to build sandbox, but it fails for v2021.01 release (I've did make >>>> sandbox_defconfig && make all). I assume I would need to add adc and >>>> adc-keys devices to some sandbox dts and some code triggering and >>>> checking the key values, but that's all I know now. >>> Well you do need to be able to build sandbox or you will get >>> nowhere...what error did you get? Once we understand what went wrong >>> we can update the docs. Maybe it is missing a dependency. >> >> $ gcc --version >> gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0 >> Copyright (C) 2017 Free Software Foundation, Inc. >> This is free software; see the source for copying conditions. There is NO >> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. >> >> $ git checkout v2021.01 >> >> $ make sandbox_defconfig >> # >> # configuration written to .config >> # >> >> $ make >> scripts/kconfig/conf --syncconfig Kconfig >> CFG u-boot.cfg >> GEN include/autoconf.mk >> GEN include/autoconf.mk.dep >> CFGCHK u-boot.cfg >> UPD include/generated/timestamp_autogenerated.h >> HOSTCC tools/mkenvimage.o >> HOSTLD tools/mkenvimage >> HOSTCC tools/fit_image.o >> HOSTCC tools/image-host.o >> HOSTCC tools/dumpimage.o >> HOSTLD tools/dumpimage >> HOSTCC tools/mkimage.o >> HOSTLD tools/mkimage >> HOSTLD tools/fit_info >> HOSTLD tools/fit_check_sign >> >> ... >> >> CC arch/sandbox/cpu/cpu.o >> In file included from include/common.h:26:0, >> from arch/sandbox/cpu/cpu.c:6: >> include/asm/global_data.h:112:58: warning: call-clobbered register used >> for global register variable >> #define DECLARE_GLOBAL_DATA_PTR register volatile gd_t *gd asm ("r9") >> ^ >> include/dm/of.h:86:1: note: in expansion of macro ?DECLARE_GLOBAL_DATA_PTR? >> DECLARE_GLOBAL_DATA_PTR; > > This is pretty mysterious. Are you sure you are using an x86_64 machine? r9 relates to ARMv7. You have to unset CROSS_COMPILE when you build the sandbox. The sandbox runs fine on aarch64. There are a bunch of errors currently when building on 32bit architectures. Simon has a lot of patches pending. Best regards Heinrich