[PATCH v3 2/2] board: mediatek: add mt8195 demo board

[Tom Rini]

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 <fparent at baylibre.com>
> > > > > 
> > > > > Add mt8195-demo board support.
> > > > > This demo purpose board uses MediaTek's MT8195 SoC.
> > > > > 
> > > > > Signed-off-by: Fabien Parent <fparent at baylibre.com>
> > > > > Signed-off-by: Amjad Ouled-Ameur <aouledameur at baylibre.com>
> > > > > Signed-off-by: Macpaul Lin <macpaul.lin at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230102/4ef83b7f/attachment.sig>