Skip to content

arm64: dts: marvell: Add DTS for 7215-IXS-A1 board#378

Merged
saiarcot895 merged 3 commits intosonic-net:masterfrom
Pavan-Nokia:dev_add_nokia_dts_2
Mar 11, 2024
Merged

arm64: dts: marvell: Add DTS for 7215-IXS-A1 board#378
saiarcot895 merged 3 commits intosonic-net:masterfrom
Pavan-Nokia:dev_add_nokia_dts_2

Conversation

@Pavan-Nokia
Copy link
Copy Markdown
Contributor

@Pavan-Nokia Pavan-Nokia commented Mar 4, 2024

Create platform specific DTS for Marvell-arm64 based Nokia 7215-IXS-A1 platform.

7215-IXS-A1 is an aggregation switch based on Marvell AlleyCat5X. This DTS is extracted from Marvell cn9130-crb (reference board) and removed unused nodes along with platform specific changes.

Upstream Kernel PR
https://lore.kernel.org/linux-arm-kernel/CAJOx763HwgK-EEt-AkVqtoObhx_VGokj-Mn0yhS7Sno0uDOZfw@mail.gmail.com/T/#m3d1fc5bce0dcb5ec001f8496a207ccb485903507

7215-IXS-A1 is an aggregation switch based on Marvell AlleyCat5X.
This dts is extracted from Marvell cn9130-crb and removed unused
nodes along with platform specific changes.
@Pavan-Nokia
Copy link
Copy Markdown
Contributor Author

@saiarcot895 Please help review and merge

Comment thread patch/0011-arm64-dts-marvell-Add-Nokia-7215-IXS-A1-board.patch
@Pavan-Nokia Pavan-Nokia requested a review from saiarcot895 March 8, 2024 15:20
Copy link
Copy Markdown
Contributor

@saiarcot895 saiarcot895 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding the comments in the kernel mail thread, I concur with what Andrew has said there. The patch that is submitted to mainline should be using mainline's features/APIs. For the backport to 5.10, that patch can be modified to fit whatever is available in 5.10.x.

Because SONiC's kernel is based on Debian's kernel, we will always be behind the mainline, so there will always be some level of backporting that would need to be done.

@Pavan-Nokia
Copy link
Copy Markdown
Contributor Author

@saiarcot895 Understood and agree. The comment you are referring to was from our previous version of the DTS that we tried to upstream.
We have modified the DTS to address most of Andrew's comments in this version of the DTS as much as our platform supports.
Thank you for help with review and merging

@saiarcot895 saiarcot895 merged commit b6fd1ba into sonic-net:master Mar 11, 2024
@Pavan-Nokia Pavan-Nokia deleted the dev_add_nokia_dts_2 branch March 12, 2024 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants