Skip to content

Add ARM architecture support to manage-config#146

Merged
lguohan merged 2 commits intosonic-net:masterfrom
taocy001:centec_arm64
Jun 12, 2020
Merged

Add ARM architecture support to manage-config#146
lguohan merged 2 commits intosonic-net:masterfrom
taocy001:centec_arm64

Conversation

@taocy001
Copy link
Copy Markdown
Contributor

@taocy001 taocy001 commented Jun 8, 2020

There are some kernel compilation options that are only supported on certain architectures, such as CONFIG_SERIAL_8250_LPSS depends on X86.

  1. Split options into multiple sections for different architectures, section [common] for all architectures.
  2. centec PR Add centec arm64 architecture support for E530 sonic-buildimage#4641 depends on this PR, we should remove option CONFIG_STRICT_DEVMEM.
  3. move tsingma SoC driver to initramfs as ko, close PR add CENTEC tsingma SoC platform support #139

…that are only supported by amd64 for arm64.
@taocy001
Copy link
Copy Markdown
Contributor Author

@lguohan
I moved all the options in the current inclusion file to [amd64], do you think that's ok? CENTEC arm64 ARCH depends none of them.
Or do you need to keep the x86 and ARM kernel options as consistent as possible?

@taocy001 taocy001 requested a review from lguohan June 12, 2020 02:39
@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Jun 12, 2020

you probably need the optoe driver, but I think we can move those later. i am signing off this pr. great!

@lguohan lguohan merged commit a37a8bf into sonic-net:master Jun 12, 2020
paulmenzel added a commit to paulmenzel/sonic-linux-kernel that referenced this pull request Nov 6, 2020
Fixes: a37a8bf ("Add ARM architecture support to manage-config (sonic-net#146)")
paulmenzel added a commit to paulmenzel/sonic-linux-kernel that referenced this pull request Nov 6, 2020
Fixes: a37a8bf ("Add ARM architecture support to manage-config (sonic-net#146)")
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