* Create new platform for arm system
@ 2017-11-09 5:53 Chen.KenYY 陳永營 TAO
2017-11-10 3:00 ` Lei YU
2017-11-30 13:32 ` Brad Bishop
0 siblings, 2 replies; 4+ messages in thread
From: Chen.KenYY 陳永營 TAO @ 2017-11-09 5:53 UTC (permalink / raw)
To: openbmc
[-- Attachment #1: Type: text/plain, Size: 10827 bytes --]
Hi all,
I have a platform which is arm system.
My customer want to update the platform code to openBMC.
For now, the code structure as below.
meta-arm/
├── common
│ └── recipes-phosphor
│ ├── dbus
│ │ ├── openpower-dbus-interfaces.bb
│ │ ├── openpower-dbus-interfaces-mapper-config-native.bb
│ │ └── phosphor-mapper-config-native%.bbappend
│ ├── debug
│ │ ├── openpower-debug-collector
│ │ │ ├── openpower-debug-collector-checkstop@.service
│ │ │ └── openpower-debug-collector-watchdog@.service
│ │ ├── openpower-debug-collector.bb
│ │ ├── openpower-debug-collector-error-native.bb
│ │ ├── openpower-debug-collector.inc
│ │ └── phosphor-logging-error-logs-native.bbappend
│ ├── ipmi
│ │ ├── hostboot-inventory-config
│ │ │ └── config.yaml
│ │ ├── hostboot-inventory-config-native.bb
│ │ ├── openpower-ipmi-oem.bb
│ │ ├── openpower-ipmi-oem-whitelist
│ │ │ └── openpower-ipmi-oem-whitelist-native.conf
│ │ └── openpower-ipmi-oem-whitelist-native.bb
│ └── packagegroups
│ ├── packagegroup-obmc-apps.bbappend
│ ├── packagegroup-obmc-ipmid-providers.bbappend
│ ├── packagegroup-obmc-phosphor-debugtools.bbappend
│ └── packagegroup-op-apps.bb
├── conf
│ ├── layer.conf
│ └── machine
│ └── include
│ └── arm.inc
├── COPYING.apache-2.0
├── COPYING.MIT
├── LICENSE
└── meta-qualcomm
├── conf
│ ├── layer.conf
│ └── machine
│ └── include
│ └── qualcomm.inc
├── meta-2400-rep
│ ├── conf
│ │ ├── bblayers.conf.sample
│ │ ├── conf-notes.txt
│ │ ├── layer.conf
│ │ ├── local.conf.sample
│ │ └── machine
│ │ └── 2400-rep.conf
│ ├── recipes-bsp
│ │ └── u-boot
│ │ ├── u-boot
│ │ │ └── 0001-Centriq2400-REP-platform-board-initial-Ethernet-Phy-reset.patch
│ │ └── u-boot_%.bbappend
│ ├── recipes-kernel
│ │ └── linux
│ │ ├── linux-obmc
│ │ │ ├── 0001-2400REP-Add-GPIOC7-config-for-host-spi-rom-init.patch
│ │ │ ├── 0001-Add-driver-for-PCA9641-Arbitration.patch
│ │ │ ├── 0001-Driver-Add-DS1100-driver-for-PMBUS.patch
│ │ │ ├── 0001-driver-add-IR38613-driver.patch
│ │ │ ├── 0001-Driver-Modify-fan-driver.patch
│ │ │ ├── 0001-Enabling-temp-monitoring-by-default-on-adm1278.patch
│ │ │ ├── 0001-Migrate-the-JTAG-driver-from-the-BSP-from-Aspeed.patch
│ │ │ ├── 0001-PMBUS-modify-pmbus.patch
│ │ │ ├── 0001-spi-add-aspeed-spi-generic-driver.patch
│ │ │ ├── 0001-spi-Add-centriq-2400-spi-hwmon-driver.patch
│ │ │ ├── 0002-driver-add-PXM1310-driver.patch
│ │ │ ├── 0002-PMBUS-modify-pmbus.patch
│ │ │ ├── 0003-thermal-sensor-NVT210.patch
│ │ │ ├── 2400rep.cfg
│ │ │ ├── aspeed-bmc-arm-2400-rep.dts
│ │ │ └── v3-0001-Add-SSIF-driver-for-I2C-slave-device.patch
│ │ └── linux-obmc_%.bbappend
│ ├── recipes-phosphor
│ │ ├── bootmodeutil
│ │ │ ├── bootmodeutil.bb
│ │ │ └── files
│ │ │ ├── BOOTMODEFILE
│ │ │ └── bootmodeutil.c
│ │ ├── chassis
│ │ │ ├── ir38163-workaround
│ │ │ │ ├── ir38163_workaround.sh
│ │ │ │ ├── ir-add@.service
│ │ │ │ └── ir-remove@.service
│ │ │ └── ir38163-workaround.bb
│ │ ├── console
│ │ │ ├── obmc-console
│ │ │ │ └── obmc-console.conf
│ │ │ └── obmc-console.bbappend
│ │ ├── cpld
│ │ │ ├── cpld-fwupdate
│ │ │ │ ├── ast-jtag.c
│ │ │ │ ├── ast-jtag.h
│ │ │ │ ├── lattice.c
│ │ │ │ ├── lattice.h
│ │ │ │ └── main.c
│ │ │ ├── cpld-fwupdate.bb
│ │ │ ├── cpld-version
│ │ │ │ ├── cpld-version.service
│ │ │ │ └── cpld_version.sh
│ │ │ └── cpld-version.bb
│ │ ├── gpioinit
│ │ │ ├── files
│ │ │ │ ├── gpioinit.c
│ │ │ │ └── gpioinit.service
│ │ │ └── gpioinit.bb
│ │ ├── ipmi
│ │ │ ├── 2400-rep-ipmi-inventory-map
│ │ │ │ └── config.yaml
│ │ │ ├── 2400-rep-ipmi-inventory-map-native.bb
│ │ │ ├── openpower-ipmi-oem
│ │ │ │ ├── 0001-Add-detection-of-broad-and-node-command.patch
│ │ │ │ ├── 0001-Add-Get-BMC-Vendor-ID-command.patch
│ │ │ │ ├── 0001-Add-Get-pcie-slot-status-command.patch
│ │ │ │ └── 0001-Add-Send-Soc-MAC-from-host.patch
│ │ │ ├── openpower-ipmi-oem.bbappend
│ │ │ ├── phosphor-ipmi-bt
│ │ │ │ ├── 0001-btbrigde-to-ssif-slave.patch
│ │ │ │ └── 0001-Modify-btbridged-device-link.patch
│ │ │ ├── phosphor-ipmi-bt.bbappend
│ │ │ ├── phosphor-ipmi-fru-properties%
│ │ │ │ └── extra-properties.yaml
│ │ │ ├── phosphor-ipmi-fru-properties%.bbappend
│ │ │ ├── phosphor-ipmi-sensor-inventory%
│ │ │ │ └── config.yaml
│ │ │ └── phosphor-ipmi-sensor-inventory%.bbappend
│ │ ├── leds
│ │ │ ├── 2400-rep-led-manager-config
│ │ │ │ └── led.yaml
│ │ │ └── 2400-rep-led-manager-config-native.bb
│ │ ├── macsetup
│ │ │ ├── files
│ │ │ │ ├── macsetup.c
│ │ │ │ └── macsetup.service
│ │ │ └── macsetup.bb
│ │ ├── qspitest
│ │ │ ├── files
│ │ │ │ ├── qspi_test.c
│ │ │ │ └── qspi_test.h
│ │ │ └── qspitest.bb
│ │ ├── sensors
│ │ │ ├── phosphor-hwmon%
│ │ │ │ ├── 0001-phosphor-hwmon-fixed-open-system-file-timeout.patch
│ │ │ │ └── obmc
│ │ │ │ └── hwmon
│ │ │ │ └── ahb
│ │ │ │ ├── 1e631000.flash-controller
│ │ │ │ │ └── spi2@0
│ │ │ │ │ └── centriq2400-spi-hwmon@0.conf
│ │ │ │ └── apb
│ │ │ │ └── i2c@1e78a000
│ │ │ │ ├── i2c-bus@180
│ │ │ │ │ ├── ir38163@12.conf
│ │ │ │ │ ├── ir38163@42.conf
│ │ │ │ │ ├── ir38163@44.conf
│ │ │ │ │ ├── ir38163@46.conf
│ │ │ │ │ ├── ir38163@48.conf
│ │ │ │ │ ├── pxm1310@02.conf
│ │ │ │ │ └── pxm1310@04.conf
│ │ │ │ ├── i2c-bus@1c0
│ │ │ │ │ ├── nvt210@4c.conf
│ │ │ │ │ ├── tmp421@1d.conf
│ │ │ │ │ ├── tmp421@1f.conf
│ │ │ │ │ ├── tmp421@4d.conf
│ │ │ │ │ └── tmp421@4f.conf
│ │ │ │ ├── i2c-bus@300
│ │ │ │ │ ├── adm1278@10.conf
│ │ │ │ │ └── adm1278@11.conf
│ │ │ │ ├── i2c-bus@340
│ │ │ │ │ └── pca9641@70
│ │ │ │ │ └── i2c-arb
│ │ │ │ │ ├── adm1278@12.conf
│ │ │ │ │ ├── ds1100@58.conf
│ │ │ │ │ ├── max31790@20.conf
│ │ │ │ │ ├── max31790@23.conf
│ │ │ │ │ └── tmp421@1d.conf
│ │ │ │ └── i2c-bus@80
│ │ │ │ ├── tmp421@1c.conf
│ │ │ │ ├── tmp421@1e.conf
│ │ │ │ ├── tmp421@2a.conf
│ │ │ │ └── tmp421@4e.conf
│ │ │ └── phosphor-hwmon%.bbappend
│ │ ├── settings
│ │ │ ├── phosphor-settings
│ │ │ │ ├── powerpolicy-default-ALWAYS_POWER_ON.override.yml
│ │ │ │ └── timeowner-default-BOTH.override.yml
│ │ │ └── phosphor-settings.bbappend
│ │ ├── spitest
│ │ │ ├── files
│ │ │ │ └── spitest.c
│ │ │ └── spitest.bb
│ │ ├── thermalutil
│ │ │ ├── files
│ │ │ │ ├── thermal_control.sh
│ │ │ │ ├── thermal-start@.service
│ │ │ │ ├── thermal-stop@.service
│ │ │ │ └── thermalutil.c
│ │ │ └── thermalutil.bb
│ │ └── workbook
│ │ ├── 2400-rep-config
│ │ │ └── 2400-rep.py
│ │ └── 2400-rep-config.bb
│ └── recipes.txt
└── skeleton
└── skeleton.tar.gz
Could anyone help to review this structure and tell me how to separate this code will be good to reviewer?
As I know, the kernel patch should not in machine file.
If any other question, please feel free to me. Thanks.
Best Regards,
Ken
[-- Attachment #2: Type: text/html, Size: 118568 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Create new platform for arm system
2017-11-09 5:53 Create new platform for arm system Chen.KenYY 陳永營 TAO
@ 2017-11-10 3:00 ` Lei YU
2017-11-13 0:04 ` Joel Stanley
2017-11-30 13:32 ` Brad Bishop
1 sibling, 1 reply; 4+ messages in thread
From: Lei YU @ 2017-11-10 3:00 UTC (permalink / raw)
To: Chen.KenYY 陳永營 TAO; +Cc: openbmc
Hi Ken,
It's good to know OpenBMC will be working on an ARM system.
To submit patches, I would suggest follow how other machines enters OpenBMC.
E.g. meta-openbmc-machines/meta-x86/meta-quanta.
You can use `git log` command to get the commit log, and see how the machine
is added into OpenBMC project.
E.g.
```
git log --oneline meta-openbmc-machines/meta-x86/meta-quanta
...
a1bc1d2 Add basic quanta-q7l1 obmc image recipe.
bfe9b60 Add sensor and some FRU definitions for ipmi.
b36c004 quanta-q71l package for controlling the SPI master.
5327795 quanta-q71l: add phosphor-hwmon configs
b7975ed quanta-q71l: initial machine support
```
You can try to split your commit into separated smaller commits as quanta-q71l,
and then submit your changes.
Thanks!
--
BRs,
Lei YU
On Thu, Nov 9, 2017 at 1:53 PM, Chen.KenYY 陳永營 TAO
<Chen.KenYY@inventec.com> wrote:
> Hi all,
>
>
>
> I have a platform which is arm system.
>
> My customer want to update the platform code to openBMC.
>
> For now, the code structure as below.
>
> meta-arm/
>
> ├── common
>
> │ └── recipes-phosphor
>
> │ ├── dbus
>
> │ │ ├── openpower-dbus-interfaces.bb
>
> │ │ ├── openpower-dbus-interfaces-mapper-config-native.bb
>
> │ │ └── phosphor-mapper-config-native%.bbappend
>
> │ ├── debug
>
> │ │ ├── openpower-debug-collector
>
> │ │ │ ├── openpower-debug-collector-checkstop@.service
>
> │ │ │ └── openpower-debug-collector-watchdog@.service
>
> │ │ ├── openpower-debug-collector.bb
>
> │ │ ├── openpower-debug-collector-error-native.bb
>
> │ │ ├── openpower-debug-collector.inc
>
> │ │ └── phosphor-logging-error-logs-native.bbappend
>
> │ ├── ipmi
>
> │ │ ├── hostboot-inventory-config
>
> │ │ │ └── config.yaml
>
> │ │ ├── hostboot-inventory-config-native.bb
>
> │ │ ├── openpower-ipmi-oem.bb
>
> │ │ ├── openpower-ipmi-oem-whitelist
>
> │ │ │ └── openpower-ipmi-oem-whitelist-native.conf
>
> │ │ └── openpower-ipmi-oem-whitelist-native.bb
>
> │ └── packagegroups
>
> │ ├── packagegroup-obmc-apps.bbappend
>
> │ ├── packagegroup-obmc-ipmid-providers.bbappend
>
> │ ├── packagegroup-obmc-phosphor-debugtools.bbappend
>
> │ └── packagegroup-op-apps.bb
>
> ├── conf
>
> │ ├── layer.conf
>
> │ └── machine
>
> │ └── include
>
> │ └── arm.inc
>
> ├── COPYING.apache-2.0
>
> ├── COPYING.MIT
>
> ├── LICENSE
>
> └── meta-qualcomm
>
> ├── conf
>
> │ ├── layer.conf
>
> │ └── machine
>
> │ └── include
>
> │ └── qualcomm.inc
>
> ├── meta-2400-rep
>
> │ ├── conf
>
> │ │ ├── bblayers.conf.sample
>
> │ │ ├── conf-notes.txt
>
> │ │ ├── layer.conf
>
> │ │ ├── local.conf.sample
>
> │ │ └── machine
>
> │ │ └── 2400-rep.conf
>
> │ ├── recipes-bsp
>
> │ │ └── u-boot
>
> │ │ ├── u-boot
>
> │ │ │ └──
> 0001-Centriq2400-REP-platform-board-initial-Ethernet-Phy-reset.patch
>
> │ │ └── u-boot_%.bbappend
>
> │ ├── recipes-kernel
>
> │ │ └── linux
>
> │ │ ├── linux-obmc
>
> │ │ │ ├──
> 0001-2400REP-Add-GPIOC7-config-for-host-spi-rom-init.patch
>
> │ │ │ ├── 0001-Add-driver-for-PCA9641-Arbitration.patch
>
> │ │ │ ├── 0001-Driver-Add-DS1100-driver-for-PMBUS.patch
>
> │ │ │ ├── 0001-driver-add-IR38613-driver.patch
>
> │ │ │ ├── 0001-Driver-Modify-fan-driver.patch
>
> │ │ │ ├──
> 0001-Enabling-temp-monitoring-by-default-on-adm1278.patch
>
> │ │ │ ├──
> 0001-Migrate-the-JTAG-driver-from-the-BSP-from-Aspeed.patch
>
> │ │ │ ├── 0001-PMBUS-modify-pmbus.patch
>
> │ │ │ ├── 0001-spi-add-aspeed-spi-generic-driver.patch
>
> │ │ │ ├── 0001-spi-Add-centriq-2400-spi-hwmon-driver.patch
>
> │ │ │ ├── 0002-driver-add-PXM1310-driver.patch
>
> │ │ │ ├── 0002-PMBUS-modify-pmbus.patch
>
> │ │ │ ├── 0003-thermal-sensor-NVT210.patch
>
> │ │ │ ├── 2400rep.cfg
>
> │ │ │ ├── aspeed-bmc-arm-2400-rep.dts
>
> │ │ │ └── v3-0001-Add-SSIF-driver-for-I2C-slave-device.patch
>
> │ │ └── linux-obmc_%.bbappend
>
> │ ├── recipes-phosphor
>
> │ │ ├── bootmodeutil
>
> │ │ │ ├── bootmodeutil.bb
>
> │ │ │ └── files
>
> │ │ │ ├── BOOTMODEFILE
>
> │ │ │ └── bootmodeutil.c
>
> │ │ ├── chassis
>
> │ │ │ ├── ir38163-workaround
>
> │ │ │ │ ├── ir38163_workaround.sh
>
> │ │ │ │ ├── ir-add@.service
>
> │ │ │ │ └── ir-remove@.service
>
> │ │ │ └── ir38163-workaround.bb
>
> │ │ ├── console
>
> │ │ │ ├── obmc-console
>
> │ │ │ │ └── obmc-console.conf
>
> │ │ │ └── obmc-console.bbappend
>
> │ │ ├── cpld
>
> │ │ │ ├── cpld-fwupdate
>
> │ │ │ │ ├── ast-jtag.c
>
> │ │ │ │ ├── ast-jtag.h
>
> │ │ │ │ ├── lattice.c
>
> │ │ │ │ ├── lattice.h
>
> │ │ │ │ └── main.c
>
> │ │ │ ├── cpld-fwupdate.bb
>
> │ │ │ ├── cpld-version
>
> │ │ │ │ ├── cpld-version.service
>
> │ │ │ │ └── cpld_version.sh
>
> │ │ │ └── cpld-version.bb
>
> │ │ ├── gpioinit
>
> │ │ │ ├── files
>
> │ │ │ │ ├── gpioinit.c
>
> │ │ │ │ └── gpioinit.service
>
> │ │ │ └── gpioinit.bb
>
> │ │ ├── ipmi
>
> │ │ │ ├── 2400-rep-ipmi-inventory-map
>
> │ │ │ │ └── config.yaml
>
> │ │ │ ├── 2400-rep-ipmi-inventory-map-native.bb
>
> │ │ │ ├── openpower-ipmi-oem
>
> │ │ │ │ ├── 0001-Add-detection-of-broad-and-node-command.patch
>
> │ │ │ │ ├── 0001-Add-Get-BMC-Vendor-ID-command.patch
>
> │ │ │ │ ├── 0001-Add-Get-pcie-slot-status-command.patch
>
> │ │ │ │ └── 0001-Add-Send-Soc-MAC-from-host.patch
>
> │ │ │ ├── openpower-ipmi-oem.bbappend
>
> │ │ │ ├── phosphor-ipmi-bt
>
> │ │ │ │ ├── 0001-btbrigde-to-ssif-slave.patch
>
> │ │ │ │ └── 0001-Modify-btbridged-device-link.patch
>
> │ │ │ ├── phosphor-ipmi-bt.bbappend
>
> │ │ │ ├── phosphor-ipmi-fru-properties%
>
> │ │ │ │ └── extra-properties.yaml
>
> │ │ │ ├── phosphor-ipmi-fru-properties%.bbappend
>
> │ │ │ ├── phosphor-ipmi-sensor-inventory%
>
> │ │ │ │ └── config.yaml
>
> │ │ │ └── phosphor-ipmi-sensor-inventory%.bbappend
>
> │ │ ├── leds
>
> │ │ │ ├── 2400-rep-led-manager-config
>
> │ │ │ │ └── led.yaml
>
> │ │ │ └── 2400-rep-led-manager-config-native.bb
>
> │ │ ├── macsetup
>
> │ │ │ ├── files
>
> │ │ │ │ ├── macsetup.c
>
> │ │ │ │ └── macsetup.service
>
> │ │ │ └── macsetup.bb
>
> │ │ ├── qspitest
>
> │ │ │ ├── files
>
> │ │ │ │ ├── qspi_test.c
>
> │ │ │ │ └── qspi_test.h
>
> │ │ │ └── qspitest.bb
>
> │ │ ├── sensors
>
> │ │ │ ├── phosphor-hwmon%
>
> │ │ │ │ ├──
> 0001-phosphor-hwmon-fixed-open-system-file-timeout.patch
>
> │ │ │ │ └── obmc
>
> │ │ │ │ └── hwmon
>
> │ │ │ │ └── ahb
>
> │ │ │ │ ├── 1e631000.flash-controller
>
> │ │ │ │ │ └── spi2@0
>
> │ │ │ │ │ └── centriq2400-spi-hwmon@0.conf
>
> │ │ │ │ └── apb
>
> │ │ │ │ └── i2c@1e78a000
>
> │ │ │ │ ├── i2c-bus@180
>
> │ │ │ │ │ ├── ir38163@12.conf
>
> │ │ │ │ │ ├── ir38163@42.conf
>
> │ │ │ │ │ ├── ir38163@44.conf
>
> │ │ │ │ │ ├── ir38163@46.conf
>
> │ │ │ │ │ ├── ir38163@48.conf
>
> │ │ │ │ │ ├── pxm1310@02.conf
>
> │ │ │ │ │ └── pxm1310@04.conf
>
> │ │ │ │ ├── i2c-bus@1c0
>
> │ │ │ │ │ ├── nvt210@4c.conf
>
> │ │ │ │ │ ├── tmp421@1d.conf
>
> │ │ │ │ │ ├── tmp421@1f.conf
>
> │ │ │ │ │ ├── tmp421@4d.conf
>
> │ │ │ │ │ └── tmp421@4f.conf
>
> │ │ │ │ ├── i2c-bus@300
>
> │ │ │ │ │ ├── adm1278@10.conf
>
> │ │ │ │ │ └── adm1278@11.conf
>
> │ │ │ │ ├── i2c-bus@340
>
> │ │ │ │ │ └── pca9641@70
>
> │ │ │ │ │ └── i2c-arb
>
> │ │ │ │ │ ├── adm1278@12.conf
>
> │ │ │ │ │ ├── ds1100@58.conf
>
> │ │ │ │ │ ├── max31790@20.conf
>
> │ │ │ │ │ ├── max31790@23.conf
>
> │ │ │ │ │ └── tmp421@1d.conf
>
> │ │ │ │ └── i2c-bus@80
>
> │ │ │ │ ├── tmp421@1c.conf
>
> │ │ │ │ ├── tmp421@1e.conf
>
> │ │ │ │ ├── tmp421@2a.conf
>
> │ │ │ │ └── tmp421@4e.conf
>
> │ │ │ └── phosphor-hwmon%.bbappend
>
> │ │ ├── settings
>
> │ │ │ ├── phosphor-settings
>
> │ │ │ │ ├── powerpolicy-default-ALWAYS_POWER_ON.override.yml
>
> │ │ │ │ └── timeowner-default-BOTH.override.yml
>
> │ │ │ └── phosphor-settings.bbappend
>
> │ │ ├── spitest
>
> │ │ │ ├── files
>
> │ │ │ │ └── spitest.c
>
> │ │ │ └── spitest.bb
>
> │ │ ├── thermalutil
>
> │ │ │ ├── files
>
> │ │ │ │ ├── thermal_control.sh
>
> │ │ │ │ ├── thermal-start@.service
>
> │ │ │ │ ├── thermal-stop@.service
>
> │ │ │ │ └── thermalutil.c
>
> │ │ │ └── thermalutil.bb
>
> │ │ └── workbook
>
> │ │ ├── 2400-rep-config
>
> │ │ │ └── 2400-rep.py
>
> │ │ └── 2400-rep-config.bb
>
> │ └── recipes.txt
>
> └── skeleton
>
> └── skeleton.tar.gz
>
>
>
> Could anyone help to review this structure and tell me how to separate this
> code will be good to reviewer?
>
> As I know, the kernel patch should not in machine file.
>
>
>
> If any other question, please feel free to me. Thanks.
>
>
>
> Best Regards,
>
> Ken
>
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Create new platform for arm system
2017-11-10 3:00 ` Lei YU
@ 2017-11-13 0:04 ` Joel Stanley
0 siblings, 0 replies; 4+ messages in thread
From: Joel Stanley @ 2017-11-13 0:04 UTC (permalink / raw)
To: Lei YU; +Cc: Chen.KenYY 陳永營 TAO, openbmc
Hello Ken,
On Fri, Nov 10, 2017 at 1:30 PM, Lei YU <mine260309@gmail.com> wrote:
>> Could anyone help to review this structure and tell me how to separate this
>> code will be good to reviewer?
>>
>> As I know, the kernel patch should not in machine file.
>>
I am the maintainer for the OpenBMC kernel. We would be happy to help
you get your changes into our kernel tree, and from there upstream.
Where possible we will ask you to submit them upstream first.
Do you have a git tree with your changes published? If so, I can take
a look and make recommendations.
The kernel follows a similar process to the upstream kernel
development process. We ask you to submit your patches to the mailing
list, where they will be reviewed and then merged.
Cheers,
Joel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Create new platform for arm system
2017-11-09 5:53 Create new platform for arm system Chen.KenYY 陳永營 TAO
2017-11-10 3:00 ` Lei YU
@ 2017-11-30 13:32 ` Brad Bishop
1 sibling, 0 replies; 4+ messages in thread
From: Brad Bishop @ 2017-11-30 13:32 UTC (permalink / raw)
To: Chen.KenYY 陳永營 TAO, openbmc
On Thu, 2017-11-09 at 05:53 +0000, Chen.KenYY 陳永營 TAO wrote:
> Hi all,
>
> I have a platform which is arm system.
> My customer want to update the platform code to openBMC.
> For now, the code structure as below.
> meta-arm/
> ├── common
> │ └── recipes-phosphor
> │ ├── dbus
> │ │ ├── openpower-dbus-interfaces.bb
> │ │ ├── openpower-dbus-interfaces-mapper-config-native.bb
> │ │ └── phosphor-mapper-config-native%.bbappend
> │ ├── debug
> │ │ ├── openpower-debug-collector
> │ │ │ ├── openpower-debug-collector-checkstop@.service
> │ │ │ └── openpower-debug-collector-watchdog@.service
> │ │ ├── openpower-debug-collector.bb
> │ │ ├── openpower-debug-collector-error-native.bb
> │ │ ├── openpower-debug-collector.inc
> │ │ └── phosphor-logging-error-logs-native.bbappend
> │ ├── ipmi
> │ │ ├── hostboot-inventory-config
> │ │ │ └── config.yaml
> │ │ ├── hostboot-inventory-config-native.bb
> │ │ ├── openpower-ipmi-oem.bb
> │ │ ├── openpower-ipmi-oem-whitelist
> │ │ │ └── openpower-ipmi-oem-whitelist-native.conf
> │ │ └── openpower-ipmi-oem-whitelist-native.bb
> │ └── packagegroups
> │ ├── packagegroup-obmc-apps.bbappend
> │ ├── packagegroup-obmc-ipmid-providers.bbappend
> │ ├── packagegroup-obmc-phosphor-debugtools.bbappend
> │ └── packagegroup-op-apps.bb
> ├── conf
> │ ├── layer.conf
> │ └── machine
> │ └── include
> │ └── arm.inc
> ├── COPYING.apache-2.0
> ├── COPYING.MIT
> ├── LICENSE
> └── meta-qualcomm
> ├── conf
> │ ├── layer.conf
> │ └── machine
> │ └── include
> │ └── qualcomm.inc
> ├── meta-2400-rep
> │ ├── conf
> │ │ ├── bblayers.conf.sample
> │ │ ├── conf-notes.txt
> │ │ ├── layer.conf
> │ │ ├── local.conf.sample
> │ │ └── machine
> │ │ └── 2400-rep.conf
> │ ├── recipes-bsp
> │ │ └── u-boot
> │ │ ├── u-boot
> │ │ │ └── 0001-Centriq2400-REP-platform-board-initial-
> Ethernet-Phy-reset.patch
> │ │ └── u-boot_%.bbappend
> │ ├── recipes-kernel
> │ │ └── linux
> │ │ ├── linux-obmc
> │ │ │ ├── 0001-2400REP-Add-GPIOC7-config-for-host-spi-
> rom-init.patch
> │ │ │ ├── 0001-Add-driver-for-PCA9641-Arbitration.patch
> │ │ │ ├── 0001-Driver-Add-DS1100-driver-for-PMBUS.patch
> │ │ │ ├── 0001-driver-add-IR38613-driver.patch
> │ │ │ ├── 0001-Driver-Modify-fan-driver.patch
> │ │ │ ├── 0001-Enabling-temp-monitoring-by-default-on-
> adm1278.patch
> │ │ │ ├── 0001-Migrate-the-JTAG-driver-from-the-BSP-
> from-Aspeed.patch
> │ │ │ ├── 0001-PMBUS-modify-pmbus.patch
> │ │ │ ├── 0001-spi-add-aspeed-spi-generic-driver.patch
> │ │ │ ├── 0001-spi-Add-centriq-2400-spi-hwmon-
> driver.patch
> │ │ │ ├── 0002-driver-add-PXM1310-driver.patch
> │ │ │ ├── 0002-PMBUS-modify-pmbus.patch
> │ │ │ ├── 0003-thermal-sensor-NVT210.patch
> │ │ │ ├── 2400rep.cfg
> │ │ │ ├── aspeed-bmc-arm-2400-rep.dts
> │ │ │ └── v3-0001-Add-SSIF-driver-for-I2C-slave-
> device.patch
> │ │ └── linux-obmc_%.bbappend
> │ ├── recipes-phosphor
> │ │ ├── bootmodeutil
> │ │ │ ├── bootmodeutil.bb
> │ │ │ └── files
> │ │ │ ├── BOOTMODEFILE
> │ │ │ └── bootmodeutil.c
> │ │ ├── chassis
> │ │ │ ├── ir38163-workaround
> │ │ │ │ ├── ir38163_workaround.sh
> │ │ │ │ ├── ir-add@.service
> │ │ │ │ └── ir-remove@.service
> │ │ │ └── ir38163-workaround.bb
> │ │ ├── console
> │ │ │ ├── obmc-console
> │ │ │ │ └── obmc-console.conf
> │ │ │ └── obmc-console.bbappend
> │ │ ├── cpld
> │ │ │ ├── cpld-fwupdate
> │ │ │ │ ├── ast-jtag.c
> │ │ │ │ ├── ast-jtag.h
> │ │ │ │ ├── lattice.c
> │ │ │ │ ├── lattice.h
> │ │ │ │ └── main.c
> │ │ │ ├── cpld-fwupdate.bb
> │ │ │ ├── cpld-version
> │ │ │ │ ├── cpld-version.service
> │ │ │ │ └── cpld_version.sh
> │ │ │ └── cpld-version.bb
> │ │ ├── gpioinit
> │ │ │ ├── files
> │ │ │ │ ├── gpioinit.c
> │ │ │ │ └── gpioinit.service
> │ │ │ └── gpioinit.bb
> │ │ ├── ipmi
> │ │ │ ├── 2400-rep-ipmi-inventory-map
> │ │ │ │ └── config.yaml
> │ │ │ ├── 2400-rep-ipmi-inventory-map-native.bb
> │ │ │ ├── openpower-ipmi-oem
> │ │ │ │ ├── 0001-Add-detection-of-broad-and-node-
> command.patch
> │ │ │ │ ├── 0001-Add-Get-BMC-Vendor-ID-command.patch
> │ │ │ │ ├── 0001-Add-Get-pcie-slot-status-command.patch
> │ │ │ │ └── 0001-Add-Send-Soc-MAC-from-host.patch
> │ │ │ ├── openpower-ipmi-oem.bbappend
> │ │ │ ├── phosphor-ipmi-bt
> │ │ │ │ ├── 0001-btbrigde-to-ssif-slave.patch
> │ │ │ │ └── 0001-Modify-btbridged-device-link.patch
> │ │ │ ├── phosphor-ipmi-bt.bbappend
> │ │ │ ├── phosphor-ipmi-fru-properties%
> │ │ │ │ └── extra-properties.yaml
> │ │ │ ├── phosphor-ipmi-fru-properties%.bbappend
> │ │ │ ├── phosphor-ipmi-sensor-inventory%
> │ │ │ │ └── config.yaml
> │ │ │ └── phosphor-ipmi-sensor-inventory%.bbappend
> │ │ ├── leds
> │ │ │ ├── 2400-rep-led-manager-config
> │ │ │ │ └── led.yaml
> │ │ │ └── 2400-rep-led-manager-config-native.bb
> │ │ ├── macsetup
> │ │ │ ├── files
> │ │ │ │ ├── macsetup.c
> │ │ │ │ └── macsetup.service
> │ │ │ └── macsetup.bb
> │ │ ├── qspitest
> │ │ │ ├── files
> │ │ │ │ ├── qspi_test.c
> │ │ │ │ └── qspi_test.h
> │ │ │ └── qspitest.bb
> │ │ ├── sensors
> │ │ │ ├── phosphor-hwmon%
> │ │ │ │ ├── 0001-phosphor-hwmon-fixed-open-system-file-
> timeout.patch
> │ │ │ │ └── obmc
> │ │ │ │ └── hwmon
> │ │ │ │ └── ahb
> │ │ │ │ ├── 1e631000.flash-controller
> │ │ │ │ │ └── spi2@0
> │ │ │ │ │ └── centriq2400-spi-hwmon@0.c
> onf
> │ │ │ │ └── apb
> │ │ │ │ └── i2c@1e78a000
> │ │ │ │ ├── i2c-bus@180
> │ │ │ │ │ ├── ir38163@12.conf
> │ │ │ │ │ ├── ir38163@42.conf
> │ │ │ │ │ ├── ir38163@44.conf
> │ │ │ │ │ ├── ir38163@46.conf
> │ │ │ │ │ ├── ir38163@48.conf
> │ │ │ │ │ ├── pxm1310@02.conf
> │ │ │ │ │ └── pxm1310@04.conf
> │ │ │ │ ├── i2c-bus@1c0
> │ │ │ │ │ ├── nvt210@4c.conf
> │ │ │ │ │ ├── tmp421@1d.conf
> │ │ │ │ │ ├── tmp421@1f.conf
> │ │ │ │ │ ├── tmp421@4d.conf
> │ │ │ │ │ └── tmp421@4f.conf
> │ │ │ │ ├── i2c-bus@300
> │ │ │ │ │ ├── adm1278@10.conf
> │ │ │ │ │ └── adm1278@11.conf
> │ │ │ │ ├── i2c-bus@340
> │ │ │ │ │ └── pca9641@70
> │ │ │ │ │ └── i2c-arb
> │ │ │ │ │ ├── adm1278@12.co
> nf
> │ │ │ │ │ ├── ds1100@58.con
> f
> │ │ │ │ │ ├── max31790@20.c
> onf
> │ │ │ │ │ ├── max31790@23.c
> onf
> │ │ │ │ │ └── tmp421@1d.con
> f
> │ │ │ │ └── i2c-bus@80
> │ │ │ │ ├── tmp421@1c.conf
> │ │ │ │ ├── tmp421@1e.conf
> │ │ │ │ ├── tmp421@2a.conf
> │ │ │ │ └── tmp421@4e.conf
> │ │ │ └── phosphor-hwmon%.bbappend
> │ │ ├── settings
> │ │ │ ├── phosphor-settings
> │ │ │ │ ├── powerpolicy-default-
> ALWAYS_POWER_ON.override.yml
> │ │ │ │ └── timeowner-default-BOTH.override.yml
> │ │ │ └── phosphor-settings.bbappend
> │ │ ├── spitest
> │ │ │ ├── files
> │ │ │ │ └── spitest.c
> │ │ │ └── spitest.bb
> │ │ ├── thermalutil
> │ │ │ ├── files
> │ │ │ │ ├── thermal_control.sh
> │ │ │ │ ├── thermal-start@.service
> │ │ │ │ ├── thermal-stop@.service
> │ │ │ │ └── thermalutil.c
> │ │ │ └── thermalutil.bb
> │ │ └── workbook
> │ │ ├── 2400-rep-config
> │ │ │ └── 2400-rep.py
> │ │ └── 2400-rep-config.bb
> │ └── recipes.txt
> └── skeleton
> └── skeleton.tar.gz
>
> Could anyone help to review this structure and tell me how to
> separate this code will be good to reviewer?
> As I know, the kernel patch should not in machine file.
>
> If any other question, please feel free to me. Thanks.
>
> Best Regards,
> Ken
>
>
Hi Ken
I was just curious if you had gotten enough feedback to get started.
Another data point could be the quanta-q711 or witherspoon machines.
If you have a look at the commit history (git log -p) and search for
these machines (start the search from the beginning of time and search
forward) these machines were added in a review-friendly way for the
most part.
To begin with, I'd focus more on adding a board layer that allows you
to build an image and boot to a shell, and tackle the arm-host aspects
(i.e. the userspace) of this system after that. That piece might be
more involved given that this is the first arm system.
thx - brad
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-11-30 13:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-09 5:53 Create new platform for arm system Chen.KenYY 陳永營 TAO
2017-11-10 3:00 ` Lei YU
2017-11-13 0:04 ` Joel Stanley
2017-11-30 13:32 ` Brad Bishop
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.