Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ACCTON][AMX3200] Add new platform #995

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

WillyLiu-EC
Copy link
Contributor

CPU sub-system
CPU : CPU: Intel Atom Processor C3508,1.6GHZ
DDR4 SDRAM: 8GB 2400MHz with ECC (SO-DIMM) DDR4 *1 SPI Flash (Boot): 16Mb *2
M.2 SSD: SATA 128GB MLC
EC: IT8528E/VG-I
TPM: SLB 9670XQ2.0 FW7.63 INFINEON

Management
UART RS232 console port (RJ45),
WGI210AT, INTEL, Out-band Management Ethernet port (RJ45)

PHY
META DX2+ PM6214*2
Microchip 1.6T Ethernet MAC/PHYs

Ethernet Ports
2CFP2 400G + 8QSFP-28 100G

Power Supply
AC 850W PSU, DELTA,
1+1 redundant load-sharing,
hot-swappable

Cooling
F2B fixed fan with 5 pcs of 40mmx40mmx28mm 12V fans

BMC Module
AST2600

Copy link

@paulmenzel paulmenzel left a comment

Choose a reason for hiding this comment

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

CPU : CPU: Intel Atom Processor C3508,1.6GHZ

Please remove the space before the colon (:).

Linux 5.4 is quite old but still maintained until Dec, 2025. It’d be great if EdgeCore invested time into Open Network Linux to keep the Linux kernel up to date, currently 5.4.284 from September 12th, 2024.

Lastly two drivers are added: FPGA and LED. Could you please elaborate. Did you copy them? What documentation was used?

@@ -0,0 +1 @@
include $(ONL)/make/pkg.mk

Choose a reason for hiding this comment

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

Please use UNIX style line endings.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We follow this file of other platform, like as9926
What is the "UNIX style line endings" you say?

@@ -2753,7 +2753,7 @@ CONFIG_SENSORS_CORETEMP=y
# CONFIG_SENSORS_TMP102 is not set
# CONFIG_SENSORS_TMP103 is not set
# CONFIG_SENSORS_TMP108 is not set
# CONFIG_SENSORS_TMP401 is not set
CONFIG_SENSORS_TMP401=y

Choose a reason for hiding this comment

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

Please mention, that you enable it in the commit message.

@WillyLiu-EC WillyLiu-EC force-pushed the add_amx3200 branch 4 times, most recently from 7da429a to 9d7bfb9 Compare October 14, 2024 02:36
@WillyLiu-EC
Copy link
Contributor Author

Linux 5.4 is quite old but still maintained until Dec, 2025. It’d be great if EdgeCore invested time into Open Network Linux to keep the Linux kernel up to date, currently 5.4.284 from September 12th, 2024.

Lastly two drivers are added: FPGA and LED. Could you please elaborate. Did you copy them? What documentation was used?

There FPGA and leds devices for AMX3200. So, we need to add driver to control these devices.

CPU sub-system
CPU: Intel Atom Processor C3508,1.6GHZ
DDR4 SDRAM: 8GB 2400MHz with ECC (SO-DIMM) DDR4 *1
SPI Flash (Boot): 16Mb *2
M.2 SSD: SATA 128GB MLC
EC: IT8528E/VG-I
TPM: SLB 9670XQ2.0 FW7.63 INFINEON

Management
UART RS232 console port (RJ45),
WGI210AT, INTEL, Out-band Management Ethernet port (RJ45)

PHY
META DX2+ PM6214*2
Microchip 1.6T Ethernet MAC/PHYs

Ethernet Ports
2*CFP2 400G + 8*QSFP-28 100G

Power Supply
AC 850W PSU, DELTA,
1+1 redundant load-sharing,
hot-swappable

Cooling
F2B fixed fan with 5 pcs of 40mmx40mmx28mm 12V fans

BMC Module
AST2600

Enable sensors tmp401 config for TMP435A

Signed-off-by: willy_liu <[email protected]>

[AMX3200] modify UNIX style line endings for some files

Signed-off-by: willy_liu <[email protected]>
@WillyLiu-EC WillyLiu-EC changed the title [AMX3200] Add new platform [ACCTON][AMX3200] Add new platform Nov 4, 2024
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