On Mon, Dec 19, 2022 at 11:36:44AM +0800, Macpaul Lin wrote: > On 12/13/22 03:02, Tom Rini wrote: > > On Mon, Dec 12, 2022 at 01:53:05PM -0500, Tom Rini wrote: > > > On Mon, Dec 12, 2022 at 11:53:04AM -0500, Tom Rini wrote: > > > > On Thu, Nov 10, 2022 at 03:34:53PM +0800, Macpaul Lin wrote: > > > > > > > > > From: Fabien Parent <fpar...@baylibre.com> > > > > > > > > > > Add mt8195-demo board support. > > > > > This demo purpose board uses MediaTek's MT8195 SoC. > > > > > > > > > > Signed-off-by: Fabien Parent <fpar...@baylibre.com> > > > > > Signed-off-by: Amjad Ouled-Ameur <aouledam...@baylibre.com> > > > > > Signed-off-by: Macpaul Lin <macpaul....@mediatek.com> > > > > [snip] > > > > > +#include <linux/sizes.h> > > > > > + > > > > > +#define CONFIG_SYS_NS16550_SERIAL > > > > > +#define CONFIG_SYS_NS16550_REG_SIZE -4 > > > > > +#define CONFIG_SYS_NS16550_MEM32 > > > > > +#define CONFIG_SYS_NS16550_COM1 0x11002000 > > > > > +#define CONFIG_SYS_NS16550_CLK 26000000 > > > > > > > > This is unused, I believe. But when trying to build with current next I > > > > get: > > > > Error: Load Address must be set. > > > > Error: Bad parameters for image type > > > > > > > > Please rebase on top of current next and repost, thanks! > > Thanks for the information, will send patch v4 later. > > > > > > > Sorry for the noise, this failed due to another patch that needs to be > > > reworked instead. > > > > Welp, I should have finished my re-testing. The patch that broke > > phycore-rk3288 and I assumed broke this platform too, did not break this > > platform, there's still some other problem here. > > > > I've tested it on uboot/next and got the following result. > > $ make ARCH=arm phycore-rk3288_defconfig > $ make ARCH=arm > scripts/kconfig/conf --syncconfig Kconfig > UPD include/config.h > CFG u-boot.cfg > GEN include/autoconf.mk > GEN include/autoconf.mk.dep > CFG spl/u-boot.cfg > GEN spl/include/autoconf.mk > UPD include/config/uboot.release > UPD include/generated/version_autogenerated.h > UPD include/generated/timestamp_autogenerated.h > UPD include/generated/dt.h > ENVC include/generated/env.txt > ENVP include/generated/env.in > ENVT include/generated/environment.h > CC lib/asm-offsets.s > cc1: error: bad value (‘generic-armv7-a’) for ‘-mtune=’ switch > cc1: note: valid arguments to ‘-mtune=’ switch are: nocona core2 nehalem > corei7 westmere sandybridge corei7-avx ivybridge core-avx-i haswell > core-avx2 broadwell skylake skylake-avx512 bonnell atom silvermont slm knl > intel x86-64 eden-x2 nano nano-1000 nano-2000 nano-3000 nano-x2 eden-x4 > nano-x4 k8 k8-sse3 opteron opteron-sse3 athlon64 athlon64-sse3 athlon-fx > amdfam10 barcelona bdver1 bdver2 bdver3 bdver4 znver1 btver1 btver2 generic > scripts/Makefile.build:147: recipe for target 'lib/asm-offsets.s' failed > make[1]: *** [lib/asm-offsets.s] Error 1 > Makefile:1932: recipe for target 'prepare0' failed > make: *** [prepare0] Error 2 > > > I'm not sure should I test it on custodian tree: arm/next, too. This > failure seems not related to the 2nd patch of mt8195-demo board.
You don't pass ARCH= when building U-Boot, and you didn't set CROSS_COMPILE to your arm32 compiler, so that's why it failed there. v4 shows the same problem as v3 for me, so please make sure to build and boot v5, thanks. -- Tom
signature.asc
Description: PGP signature