OpenBMC porting to portwell pcomc660-ast2600 device.

[Lior Weintraub]

Hi Andrew,

The AST2600 Development kit from Portwell comes with a slightly old version of OpenBMC.
The sources came as a tar.xz file and could be downloaded from: https://www.dropbox.com/s/t2nb11mkuoisj0s/PCOM-C660-AST2600_BSP_20211210.tar.xz?dl=0

What I try to do is to get the latest OpenBMC codebase and integrate meta-portwell layer from the old sources (dated from Dec. 2021).
If you take a look on the 3 last commits from https://github.com/wlbe4/openbmc/tree/portwell, you can see my migration attempt.

I have upgraded the linux-aspeed version they used (5.10.67) to 6.1.15 but left their patch as it includes the device tree for their platform.
I have removed the patch they had for u-boot-aspeed-sdk as an attempt to solve the errors I got even though I think this patch might be needed.

The current build error with u-boot-aspeed-sdk is:
build/pcomc660-ast2600/tmp/work/pcomc660_ast2600-openbmc-linux-gnueabi/u-boot-aspeed-sdk/1_v2019.04+gitAUTOINC+fd915728e1-r0/git/lib/smbios.c:84:58: error: 'U_BOOT_DMI_DATE' undeclared (first use in this function)
   84 |         t->bios_release_date = smbios_add_string(t->eos, U_BOOT_DMI_DATE);

Regarding the errors I had with obmc-console:
ERROR: /media/data/workspace/openbmc/openbmc-fork/meta-phosphor/recipes-phosphor/console/obmc-console_git.bb: Unable to get checksum for obmc-console SRC_URI entry obmc-console.conffile: file could not be found

Their original code included a bbappend file for the obmc-console:
FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}/:"
OBMC_CONSOLE_HOST_TTY = "ttyS0"

SRC_URI:remove = "file://${BPN}.conf"
SRC_URI:append = "file://server.ttyS0.conf"

do_install:append() {
        # Remove upstream-provided configuration
        rm -rf ${D}${sysconfdir}/${BPN}

        # Install the server configuration
        install -m 0755 -d ${D}${sysconfdir}/${BPN}
        install -m 0644 ${WORKDIR}/*.conf ${D}${sysconfdir}/${BPN}/
        # Remove upstream-provided server configuration
        rm -f ${D}${sysconfdir}/${BPN}/server.ttyVUART0.conf
}

I suspected that their removal of "file://${BPN}.conf" wasn't welcomed by the obmc-console_git.bb recipe 
So I just commented it all out (except for the first line), copied the obmc-console.conf and the error was gone. 

Their server.ttyS0.conf file included:
local-tty = ttyS0
baud = 115200
So I have and added the local-tty line to my obmc-console.conf copy.

I am not sure that the console will work after my changes as I currently don't have the board for testing (will arrive in 2 weeks time).
So I am currently focusing my efforts on migration and fixing the compilation errors.

Thanks for your help,
Lior.
 
-----Original Message-----
From: Andrew Jeffery <andrew at aj.id.au> 
Sent: Monday, April 17, 2023 5:58 AM
To: Lior Weintraub <liorw at pliops.com>; Patrick Williams <patrick at stwcx.xyz>; Ryan Chen <ryan_chen at aspeedtech.com>
Cc: openbmc at lists.ozlabs.org
Subject: Re: OpenBMC porting to portwell pcomc660-ast2600 device.

CAUTION: External Sender

Hi Lior,

On Sat, 15 Apr 2023, at 20:08, Lior Weintraub wrote:
> Hi Guys,
>
> I have forked the openbmc project and opened a "portwell" branch from 
> it's "master".
> The repo is under: https://github.com/wlbe4/openbmc I am trying to 
> port OpenBMC into this board:
> https://portwell.com/solutions/openBMC.php
>
> My first commit to this branch was the original code of meta-portwell 
> layer given to me by Portwell support (PCOM-C660-AST2600_BSP_20211210).
> The second commit tries to migrate meta-portwell layer to the latest 
> Yocto syntax.
>
> Changes include (among others):
> 1. replace _append with :append
> 2. replace _prepend with :prepend
> 3. move bblayers.conf.sample, conf-notes.txt and local.conf.sample 
> into templates/default folder.
> 4. Try to fix build errors on u-boot and aspeed-linux - Still there 
> are errors

This doesn't give us any insight as to what the errors were. Can you provide more detail?

> 5. Try to fix obmc-console - The build errors are gone but not sure it 
> will work (no platform for testing yet)

What build errors did you see?

>
> Some of the changes were just a syntax change but some requires deeper 
> understanding.
> Few examples are the changes in obmc-concole, u-boot and aspeed-linux.
> Since the original version is based on older u-boot and linux kernel, 
> the patches might not be relevant anymore.
>
> Can you please review the following changes on my repo:
> meta-portwell/meta-pcomc660-ast2600/recipes-phosphor/console/obmc-cons
> ole_%.bbappend

It's hard to say whether that's correct for your platform, but I don't think there's anything obviously wrong there.

> meta-portwell/meta-pcomc660-ast2600/classes/socsec-sign.bbclass

Can you unpack what were you trying to achieve by copying socsec-sign.bbclass?

> meta-portwell/meta-pcomc660-ast2600/conf/machine/pcomc660-ast2600.conf
> meta-portwell/meta-pcomc660-ast2600/recipes-bsp/u-boot/u-boot-aspeed-s
> dk_2019.04.bbappend 
> meta-portwell/meta-pcomc660-ast2600/recipes-kernel/linux/linux-aspeed_
> git.bb

Have you tried building without applying patches to u-boot and linux-aspeed? It's not clear whether its your own patches that are breaking the build or if something else is afoot.

Andrew