-
Notifications
You must be signed in to change notification settings - Fork 346
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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.
7da429a
to
9d7bfb9
Compare
There FPGA and leds devices for AMX3200. So, we need to add driver to control these devices. |
468e048
to
7cbef43
Compare
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]>
2d63171
to
2fc6e21
Compare
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