[marvell-arm64]: Enable SDK module for bookworm #16909
Conversation
|
@Pavan-Nokia please review |
| @@ -747,8 +747,7 @@ if [[ $TARGET_BOOTLOADER == uboot ]]; then | |||
| sudo LANG=C chroot $FILESYSTEM_ROOT mv /boot/u${INITRD_FILE} /boot/$INITRD_FILE | |||
| elif [[ $CONFIGURED_ARCH == arm64 ]]; then | |||
| sudo cp -v $PLATFORM_DIR/${sonic_asic_platform}-${CONFIGURED_ARCH}/sonic_fit.its $FILESYSTEM_ROOT/boot/ | |||
| # TODO: fix for bookworm, probably needs arm64 patches in sonic-linux-kernel to be updated | |||
| #sudo LANG=C chroot $FILESYSTEM_ROOT mkimage -f /boot/sonic_fit.its /boot/sonic_${CONFIGURED_ARCH}.fit | |||
| sudo LANG=C chroot $FILESYSTEM_ROOT mkimage -f /boot/sonic_fit.its /boot/sonic_${CONFIGURED_ARCH}.fit | |||
There was a problem hiding this comment.
This command is failing with FATAL ERROR: Couldn't open "/usr/lib/linux-image-5.10.0-23-2-arm64/marvell/ac5-98dx35xx-rd.dtb": No such file or directory. I think some patches in sonic-linux-kernel need to be updated/enabled first?
There was a problem hiding this comment.
looks like below line is missed in your last commit of changing the kernel version.
There was a problem hiding this comment.
Ah, would you be able to make that change here?
Also, I see that you have also opened sonic-net/sonic-linux-kernel#352. That would probably fix the dtb file error.
There was a problem hiding this comment.
@saiarcot895 I was on leave from last 6days hence the delay in response.
I have taken care this "https://github.com/sonic-net/sonic-buildimage/blob/bookworm/platform/marvell-arm64/sonic_fit.its#L23" change and updated the PR.
yes, sonic-net/sonic-linux-kernel#352 would take care of dtb file errors.
There was a problem hiding this comment.
Can you include the sonic-linux-kernel submodule update in this PR?
There was a problem hiding this comment.
Thanks Saikrishna for looking into this PR. I have updated sonic-linux-kernel submodule to this PR.
Keshavg-marvell
force-pushed
the
bookworm
branch
from
a96b11a
to
c3a8262
Compare
This patch enables compiling of Marvell platform module and fixes sonic-platform-nokia compilation issues for bookworm.
Keshavg-marvell
force-pushed
the
bookworm
branch
from
c3a8262
to
363c6cc
Compare
|
I believe it is good to merge now. Let us know if anything is needed from our side to merge these changes. |
|
@Keshavg-marvell Can armhf be enabled as well? |
|
@saiarcot895 armhf we are planning to address with separate PR. |
|
Got it, is there an ETA for that PR? |
ETA which we are internally tracking is 7th Nov. |
|
/azpw run Azure.sonic-buildimage |
1 similar comment
|
/azpw run Azure.sonic-buildimage |
|
/azpw run "Azure.sonic-buildimage" |
|
@saiarcot895 , please try again. I tried on my PR, it works now. |
|
/azpw run Azure.sonic-buildimage |
|
New pipeline manually run here, where all checks pass. |
|
@saiarcot895 , Comment only works for Keshavg-marvell, the PR owner. |
This patch enables compiling of Marvell platform module and fixes sonic-platform-nokia compilation issues for bookworm.
This patch enables compiling of Marvell platform module and fixes sonic-platform-nokia compilation issues for bookworm.
This patch enables compiling of Marvell platform module and fixes sonic-platform-nokia compilation issues for bookworm.
This patch enables compiling of Marvell platform
module and fixes sonic-platform-nokia compilation
issues for bookworm.
Why I did it
For enabling bookworm build on Marvell-arm64 and fixing #16734
Work item tracking
How I did it
Port Marvell SDK driver for bookworm 6.1 kernel.
Port Nokia CPLD driver for bookworm 6.1 kernel
How to verify it
Built image from bookworm branch and installed on Marvell platform to
verify "show interface status".
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)