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

[Macpaul Lin]

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.

Thanks
Macpaul Lin