From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 32268C32751 for ; Thu, 1 Aug 2019 03:10:24 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id DA13020693 for ; Thu, 1 Aug 2019 03:10:23 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=aj.id.au header.i=@aj.id.au header.b="mwPoK+st"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="U0bZdRiX" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729104AbfHADKW (ORCPT ); Wed, 31 Jul 2019 23:10:22 -0400 Received: from out5-smtp.messagingengine.com ([66.111.4.29]:45879 "EHLO out5-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728770AbfHADKW (ORCPT ); Wed, 31 Jul 2019 23:10:22 -0400 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailout.nyi.internal (Postfix) with ESMTP id 62A6E22028; Wed, 31 Jul 2019 23:10:19 -0400 (EDT) Received: from imap2 ([10.202.2.52]) by compute4.internal (MEProxy); Wed, 31 Jul 2019 23:10:19 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=aj.id.au; h= mime-version:message-id:in-reply-to:references:date:from:to:cc :subject:content-type; s=fm3; bh=j7b8oAnKb+IqZUQJefzmrdWT3j/P5qo Jnn2vgpsIOPE=; b=mwPoK+st+AkJ1j+JFHykJzG8ZDQLn7vjqJGq4Zh6E8Fpi5o Gg73nobXr2Oqf425h8WHg/JbSxlv20xW/ijQQSmafUS/D0fTE1GABIUHtWjxP9dD itQbPX+PbkO+fVmHu/YgtNo+nF6AIUp/wiTY2Fu5375jUMR8TtVSWi3cIDaz5VKc eLqGPAqfD1VkAS+WMTO3GPBG0L1eV4fkon27oeOAQUPf0eJpv5H1liZfelAfi6XF SWB3WjDzF4lWYsZLiH9GrBBkAS9czNBjLlbCRy98hMSIia2GAD0ywJocDGQ3CFtm kuNGiVsZ940eLHfBCVQoDdt9QalIszr3fA7+jlw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=j7b8oA nKb+IqZUQJefzmrdWT3j/P5qoJnn2vgpsIOPE=; b=U0bZdRiXfgMQ6e22V0l/qX KPGe1uxhdunrlTTeQXTKZOz+yb9nQ27xpGYewh171miM4WACoSbg3royyUpLz3zo r3bpHtqM9JjW/w4wx6G+hAOG5+geZqfRrtvCCZ4QEkIJLBPBhZU0ESflTqjDEoq+ NLZypCjqRYNwKjJRuy4/FMDXU/k53N7UbNj3ES5edCqyqRG0Yr8wLBGJXa2pt8cy 6m09a3BvNlIH3OTGamJTRLoW37AZ95c4OEPpcBL4fxv11j7ceK47ikkBsPENOJVd 6eb+1FDO+gWLpwxCzYbH+g1QmmImBw4m7PNtMACN/rinDFxX5Jsj/oa7OIXroJRQ == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduvddrleeigdeihecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefofgggkfgjfhffhffvufgtsehttdertderredtnecuhfhrohhmpedftehnughr vgifucflvghffhgvrhihfdcuoegrnhgurhgvfiesrghjrdhiugdrrghuqeenucfrrghrrg hmpehmrghilhhfrhhomheprghnughrvgifsegrjhdrihgurdgruhenucevlhhushhtvghr ufhiiigvpedt X-ME-Proxy: Received: by mailuser.nyi.internal (Postfix, from userid 501) id 1D101E00A2; Wed, 31 Jul 2019 23:10:15 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.1.6-799-g925e343-fmstable-20190729v1 Mime-Version: 1.0 Message-Id: <0adb1e44-6a89-455e-968a-7f07343b6350@www.fastmail.com> In-Reply-To: <20190731074742.23456-1-Ben_Pai@wistron.com> References: <20190731074742.23456-1-Ben_Pai@wistron.com> Date: Thu, 01 Aug 2019 12:40:08 +0930 From: "Andrew Jeffery" To: "Ben Pai" , "Rob Herring" , mark.rutland@arm.com, "Joel Stanley" , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org Cc: wangat@tw.ibm.com Subject: Re: [PATCH v2 2/2] ARM: dts: aspeed: Add Mihawk BMC platform Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Ben, Comments inline below. Also apologies for the IBM/IPS mixup in the previous review, I've been corrected :) On Wed, 31 Jul 2019, at 17:17, Ben Pai wrote: > The Mihawk BMC is an ASPEED ast2500 based BMC that is part of an > OpenPower Power9 server. > > Signed-off-by: Ben Pai > --- > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/aspeed-bmc-opp-mihawk.dts | 907 ++++++++++++++++++++ > 2 files changed, 908 insertions(+) > create mode 100755 arch/arm/boot/dts/aspeed-bmc-opp-mihawk.dts > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index eb6de52c1936..262345544359 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -1281,5 +1281,6 @@ dtb-$(CONFIG_ARCH_ASPEED) += \ > aspeed-bmc-opp-vesnin.dtb \ > aspeed-bmc-opp-witherspoon.dtb \ > aspeed-bmc-opp-zaius.dtb \ > + aspeed-bmc-opp-mihawk.dtb \ This is a sorted list, please put it in the right spot. > aspeed-bmc-portwell-neptune.dtb \ > aspeed-bmc-quanta-q71l.dtb > diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-mihawk.dts > b/arch/arm/boot/dts/aspeed-bmc-opp-mihawk.dts > new file mode 100755 > index 000000000000..913c94326f3f > --- /dev/null > +++ b/arch/arm/boot/dts/aspeed-bmc-opp-mihawk.dts > @@ -0,0 +1,907 @@ > +/dts-v1/; > + > +#include "aspeed-g5.dtsi" > +#include > +#include > + > +/ { > + model = "Mihawk BMC"; > + compatible = "ibm,mihawk-bmc", "aspeed,ast2500"; > + > + > + chosen { > + stdout-path = &uart5; > + bootargs = "console=ttyS4,115200 earlyprintk"; > + }; > + > + memory@80000000 { > + reg = <0x80000000 0x20000000>; /* address and size of RAM(512MB) */ > + }; > + > + reserved-memory { > + #address-cells = <1>; > + #size-cells = <1>; > + ranges; > + > + flash_memory: region@98000000 { > + no-map; > + reg = <0x98000000 0x04000000>; /* 64M */ > + }; > + > + gfx_memory: framebuffer { > + size = <0x01000000>; > + alignment = <0x01000000>; > + compatible = "shared-dma-pool"; > + reusable; > + }; > + > + video_engine_memory: jpegbuffer { > + size = <0x02000000>; /* 32MM */ > + alignment = <0x01000000>; > + compatible = "shared-dma-pool"; > + reusable; > + }; > + }; > + > + gpio-keys { > + compatible = "gpio-keys"; > + > + air-water { > + label = "air-water"; > + gpios = <&gpio ASPEED_GPIO(F, 6) GPIO_ACTIVE_LOW>; > + linux,code = ; > + }; > + > + checkstop { > + label = "checkstop"; > + gpios = <&gpio ASPEED_GPIO(J, 2) GPIO_ACTIVE_LOW>; > + linux,code = ; > + }; > + > + ps0-presence { > + label = "ps0-presence"; > + gpios = <&gpio ASPEED_GPIO(Z, 2) GPIO_ACTIVE_LOW>; > + linux,code = ; > + }; > + > + ps1-presence { > + label = "ps1-presence"; > + gpios = <&gpio ASPEED_GPIO(Z, 0) GPIO_ACTIVE_LOW>; > + linux,code = ; > + }; > + id-button { > + label = "id-button"; > + gpios = <&gpio ASPEED_GPIO(F, 1) GPIO_ACTIVE_LOW>; > + linux,code = ; > + }; > + }; > + > + gpio-keys-polled { > + compatible = "gpio-keys-polled"; > + #address-cells = <1>; Delete the #address-cells property (both it and the #size-cells properties are not needed for the gpio-keys-polled node). > + poll-interval = <1000>; > + > + fan0-presence { > + label = "fan0-presence"; > + gpios = <&pca9552 9 GPIO_ACTIVE_LOW>; > + linux,code = <9>; > + }; > + > + fan1-presence { > + label = "fan1-presence"; > + gpios = <&pca9552 10 GPIO_ACTIVE_LOW>; > + linux,code = <10>; > + }; > + > + fan2-presence { > + label = "fan2-presence"; > + gpios = <&pca9552 11 GPIO_ACTIVE_LOW>; > + linux,code = <11>; > + }; > + > + fan3-presence { > + label = "fan3-presence"; > + gpios = <&pca9552 12 GPIO_ACTIVE_LOW>; > + linux,code = <12>; > + }; > + > + fan4-presence { > + label = "fan4-presence"; > + gpios = <&pca9552 13 GPIO_ACTIVE_LOW>; > + linux,code = <13>; > + }; > + > + fan5-presence { > + label = "fan5-presence"; > + gpios = <&pca9552 14 GPIO_ACTIVE_LOW>; > + linux,code = <14>; > + }; > + }; > + > + leds { > + compatible = "gpio-leds"; > + > + fault { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&gpio ASPEED_GPIO(AA, 0) GPIO_ACTIVE_LOW>; > + }; > + > + power { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&gpio ASPEED_GPIO(AA, 1) GPIO_ACTIVE_LOW>; > + }; > + > + rear-id { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&gpio ASPEED_GPIO(AA, 2) GPIO_ACTIVE_LOW>; > + }; > + > + rear-g { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&gpio ASPEED_GPIO(AA, 4) GPIO_ACTIVE_LOW>; > + }; > + > + rear-ok { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&gpio ASPEED_GPIO(Y, 0) GPIO_ACTIVE_LOW>; > + }; > + > + fan0 { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&pca9552 0 GPIO_ACTIVE_LOW>; > + }; > + > + fan1 { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&pca9552 1 GPIO_ACTIVE_LOW>; > + }; > + > + fan2 { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&pca9552 2 GPIO_ACTIVE_LOW>; > + }; > + > + fan3 { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&pca9552 3 GPIO_ACTIVE_LOW>; > + }; > + > + fan4 { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&pca9552 4 GPIO_ACTIVE_LOW>; > + }; > + > + fan5 { > + retain-state-shutdown; > + default-state = "keep"; > + gpios = <&pca9552 5 GPIO_ACTIVE_LOW>; > + }; > + }; > + > + fsi: gpio-fsi { > + compatible = "fsi-master-gpio", "fsi-master"; > + #address-cells = <2>; > + #size-cells = <0>; > + no-gpio-delays; > + > + clock-gpios = <&gpio ASPEED_GPIO(E, 6) GPIO_ACTIVE_HIGH>; > + data-gpios = <&gpio ASPEED_GPIO(E, 7) GPIO_ACTIVE_HIGH>; > + mux-gpios = <&gpio ASPEED_GPIO(E, 5) GPIO_ACTIVE_HIGH>; > + enable-gpios = <&gpio ASPEED_GPIO(D, 0) GPIO_ACTIVE_HIGH>; > + trans-gpios = <&gpio ASPEED_GPIO(R, 2) GPIO_ACTIVE_HIGH>; > + }; > + iio-hwmon-12v { > + compatible = "iio-hwmon"; > + io-channels = <&adc 0>; > + }; > + > + iio-hwmon-5v { > + compatible = "iio-hwmon"; > + io-channels = <&adc 1>; > + }; > + > + iio-hwmon-3v { > + compatible = "iio-hwmon"; > + io-channels = <&adc 2>; > + }; > + > + iio-hwmon-vdd0 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 3>; > + }; > + > + iio-hwmon-vdd1 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 4>; > + }; > + > + iio-hwmon-vcs0 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 5>; > + }; > + > + iio-hwmon-vcs1 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 6>; > + }; > + > + iio-hwmon-vdn0 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 7>; > + }; > + > + iio-hwmon-vdn1 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 8>; > + }; > + > + iio-hwmon-vio0 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 9>; > + }; > + > + iio-hwmon-vio1 { > + compatible = "iio-hwmon"; > + io-channels = <&adc 10>; > + }; > + > + iio-hwmon-vddra { > + compatible = "iio-hwmon"; > + io-channels = <&adc 11>; > + }; > + > + iio-hwmon-vddrb { > + compatible = "iio-hwmon"; > + io-channels = <&adc 13>; > + }; > + > + iio-hwmon-vddrc { > + compatible = "iio-hwmon"; > + io-channels = <&adc 14>; > + }; > + > + iio-hwmon-vddrd { > + compatible = "iio-hwmon"; > + io-channels = <&adc 15>; > + }; > + > + iio-hwmon-battery { > + compatible = "iio-hwmon"; > + io-channels = <&adc 12>; > + }; > +}; > + > +&pwm_tacho { > + status = "okay"; > + /*compatible = "aspeed,ast2500-pwm-tacho"; > + #address-cells = <1>; > + #size-cells = <1>; > + reg = <0x1e786000 0x1000>; > + clocks = <&pwm_tacho_fixed_clk>;*/ I missed this commented block last time? Please remove it. > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_pwm0_default &pinctrl_pwm1_default > + &pinctrl_pwm2_default &pinctrl_pwm3_default > + &pinctrl_pwm4_default &pinctrl_pwm5_default>; > + > + fan@0 { > + reg = <0x00>; > + aspeed,fan-tach-ch = /bits/ 8 <0x00>; > + }; > + > + fan@1 { > + reg = <0x01>; > + aspeed,fan-tach-ch = /bits/ 8 <0x01>; > + }; > + > + fan@2 { > + reg = <0x02>; > + aspeed,fan-tach-ch = /bits/ 8 <0x02>; > + }; > + > + fan@3 { > + reg = <0x03>; > + aspeed,fan-tach-ch = /bits/ 8 <0x03>; > + }; > + > + fan@4 { > + reg = <0x04>; > + aspeed,fan-tach-ch = /bits/ 8 <0x04>; > + }; > + > + fan@5 { > + reg = <0x05>; > + aspeed,fan-tach-ch = /bits/ 8 <0x05>; > + }; > + > + fan@6 { > + reg = <0x00>; > + aspeed,fan-tach-ch = /bits/ 8 <0x06>; > + }; > + > + fan@7 { > + reg = <0x01>; > + aspeed,fan-tach-ch = /bits/ 8 <0x07>; > + }; > + > + fan@8 { > + reg = <0x02>; > + aspeed,fan-tach-ch = /bits/ 8 <0x08>; > + }; > + > + fan@9 { > + reg = <0x03>; > + aspeed,fan-tach-ch = /bits/ 8 <0x09>; > + }; > + > + fan@10 { > + reg = <0x04>; > + aspeed,fan-tach-ch = /bits/ 8 <0x0a>; > + }; > + > + fan@11 { > + reg = <0x05>; > + aspeed,fan-tach-ch = /bits/ 8 <0x0b>; > + }; > +}; > + > +&fmc { > + status = "okay"; > + flash@0 { > + status = "okay"; > + label = "bmc"; > + m25p,fast-read; > + spi-max-frequency = <50000000>; > + partitions { > + #address-cells = < 1 >; > + #size-cells = < 1 >; > + compatible = "fixed-partitions"; > + u-boot@0 { > + reg = < 0 0x60000 >; > + label = "u-boot"; > + }; > + u-boot-env@60000 { > + reg = < 0x60000 0x20000 >; > + label = "u-boot-env"; > + }; > + obmc-ubi@80000 { > + reg = < 0x80000 0x1F80000 >; > + label = "obmc-ubi"; > + }; > + }; > + }; > + flash@1 { > + status = "okay"; > + label = "alt-bmc"; > + m25p,fast-read; > + spi-max-frequency = <50000000>; > + partitions { > + #address-cells = < 1 >; > + #size-cells = < 1 >; > + compatible = "fixed-partitions"; > + u-boot@0 { > + reg = < 0 0x60000 >; > + label = "alt-u-boot"; > + }; > + u-boot-env@60000 { > + reg = < 0x60000 0x20000 >; > + label = "alt-u-boot-env"; > + }; > + obmc-ubi@80000 { > + reg = < 0x80000 0x1F80000 >; > + label = "alt-obmc-ubi"; > + }; > + }; > + }; > +}; > + > +&spi1 { > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_spi1_default>; > + > + flash@0 { > + status = "okay"; > + label = "pnor"; > + m25p,fast-read; > + spi-max-frequency = <100000000>; > + }; > +}; > + > +&lpc_ctrl { > + status = "okay"; > + memory-region = <&flash_memory>; > + flash = <&spi1>; > +}; > + > +&uart1 { > + /* Rear RS-232 connector */ > + status = "okay"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_txd1_default > + &pinctrl_rxd1_default > + &pinctrl_nrts1_default > + &pinctrl_ndtr1_default > + &pinctrl_ndsr1_default > + &pinctrl_ncts1_default > + &pinctrl_ndcd1_default > + &pinctrl_nri1_default>; > +}; > + > +&uart2 { > + /* APSS */ > + status = "okay"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_txd2_default &pinctrl_rxd2_default>; > +}; > + > +&uart5 { > + status = "okay"; > +}; > + > +&mac0 { > + status = "okay"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rmii1_default>; > + use-ncsi; > +}; > + > +&mac1 { > + status = "okay"; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_rgmii2_default &pinctrl_mdio2_default>; > +}; > + > +&i2c0 { > + status = "disabled"; > +}; > + > +&i2c1 { > + status = "disabled"; > +}; > + > +&i2c2 { > + status = "okay"; > + > + /* SAMTEC P0 */ > + /* SAMTEC P1 */ > + > +}; > + > +&i2c3 { > + status = "okay"; > + > + /* APSS */ > + /* CPLD */ > + > + /* PCA9516 (repeater) -> > + * CLK Buffer 9FGS9092 > + * CLK Buffer 9DBL0651BKILFT > + * CLK Buffer 9DBL0651BKILFT > + * Power Supply 0 > + * Power Supply 1 > + * PCA 9552 LED > + */ > + > + power-supply@58 { > + compatible = "ibm,cffps1"; > + reg = <0x58>; > + }; > + > + power-supply@5b { > + compatible = "ibm,cffps1"; > + reg = <0x5b>; > + }; > + > + pca9552: pca9552@60 { > + compatible = "nxp,pca9552"; > + reg = <0x60>; > + #address-cells = <1>; > + #size-cells = <0>; > + gpio-controller; > + #gpio-cells = <2>; > + > + gpio@0 { > + reg = <0>; > + type = ; > + }; > + gpio@1 { > + reg = <1>; > + type = ; > + }; > + gpio@2 { > + reg = <2>; > + type = ; > + }; > + gpio@3 { > + reg = <3>; > + type = ; > + }; > + gpio@4 { > + reg = <4>; > + type = ; > + }; > + gpio@5 { > + reg = <5>; > + type = ; > + }; > + gpio@6 { > + reg = <6>; > + type = ; > + }; > + gpio@7 { > + reg = <7>; > + type = ; > + }; > + gpio@8 { > + reg = <8>; > + type = ; > + }; > + gpio@9 { > + reg = <9>; > + type = ; > + }; > + gpio@10 { > + reg = <10>; > + type = ; > + }; > + gpio@11 { > + reg = <11>; > + type = ; > + }; > + gpio@12 { > + reg = <12>; > + type = ; > + }; > + gpio@13 { > + reg = <13>; > + type = ; > + }; > + gpio@14 { > + reg = <14>; > + type = ; > + }; > + gpio@15 { > + reg = <15>; > + type = ; > + }; > + > + }; > + > +}; > + > +&i2c4 { > + status = "okay"; > + > + /* CP0 VDD & VCS : IR35221 */ > + /* CP0 VDN : IR35221 */ > + /* CP0 VIO : IR38064 */ > + /* CP0 VDDR : PXM1330 */ > + > + ir35221@70 { > + compatible = "infineon,ir35221"; > + reg = <0x70>; > + }; > + > + ir35221@72 { > + compatible = "infineon,ir35221"; > + reg = <0x72>; > + }; > + > +}; > + > +&i2c5 { > + status = "okay"; > + > + /* CP0 VDD & VCS : IR35221 */ > + /* CP0 VDN : IR35221 */ > + /* CP0 VIO : IR38064 */ > + /* CP0 VDDR : PXM1330 */ > + > + ir35221@70 { > + compatible = "infineon,ir35221"; > + reg = <0x70>; > + }; > + > + ir35221@72 { > + compatible = "infineon,ir35221"; > + reg = <0x72>; > + }; > + > +}; > + > +&i2c6 { > + status = "okay"; > + > + /* pca9548 -> NVMe1 to 8 */ > + > + pca9548@70 { > + compatible = "nxp,pca9548"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x70>; > + }; > + > +}; > + > +&i2c7 { > + status = "okay"; > + > + /* pca9548 -> NVMe9 to 16 */ > + > + pca9548@70 { > + compatible = "nxp,pca9548"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x70>; > + }; > + > +}; > + > +&i2c8 { > + status = "okay"; > + > + eeprom@50 { > + compatible = "atmel,24c64"; > + reg = <0x50>; > + }; > +}; > + > +&i2c9 { > + status = "okay"; > + > + /* pca9545 Riser -> > + * PCIe x8 Slot3 > + * PCIe x16 slot4 > + * PCIe x8 slot5 > + * I2C BMC RISER PCA9554 > + * BMC SCL/SDA PCA9554 > + * PCA9554 > + */ > + > + /* pca9545 -> > + * PCIe x16 Slot1 > + * PCIe x8 slot2 > + * PEX8748 > + */ > + > + pca9545riser@70 { > + compatible = "nxp,pca9545"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x70>; > + > + i2c-mux-idle-disconnect; > + interrupt-controller; > + #interrupt-cells = <2>; > + }; > + > + pca9545@71 { > + compatible = "nxp,pca9545"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x71>; > + > + i2c-mux-idle-disconnect; > + interrupt-controller; > + #interrupt-cells = <2>; > + }; > +}; > + > +&i2c10 { > + status = "okay"; > + > + /* pca9545 Riser -> > + * PCIe x8 Slot8 > + * PCIe x16 slot9 > + * PCIe x8 slot10 > + * I2C BMC RISER PCA9554 > + * BMC SCL/SDA PCA9554 > + * PCA9554 > + */ > + > + /* pca9545 -> > + * PCIe x16 Slot1 > + * PCIe x8 slot2 > + * PEX8748 > + */ > + > + pca9545riser@70 { > + compatible = "nxp,pca9545"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x70>; > + > + i2c-mux-idle-disconnect; > + interrupt-controller; > + #interrupt-cells = <2>; > + }; > + > + pca9545@71 { > + compatible = "nxp,pca9545"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x71>; > + > + i2c-mux-idle-disconnect; > + interrupt-controller; > + #interrupt-cells = <2>; > + }; > +}; > + > +&i2c11 { > + status = "okay"; > + > + /* TPM */ > + /* RTC RX8900CE */ > + /* FPGA for power sequence */ > + /* TMP275A */ > + /* TMP275A */ > + /* EMC1462 */ > + > + tpm@57 { > + compatible = "infineon,slb9645tt"; > + reg = <0x57>; > + }; > + > + rtc@32 { > + compatible = "epson,rx8900"; > + reg = <0x32>; > + }; > + > + tmp275@48 { > + compatible = "ti,tmp275"; > + reg = <0x48>; > + }; > + > + tmp275@49 { > + compatible = "ti,tmp275"; > + reg = <0x49>; > + }; > + > + /* chip emc1462 use emc1403 driver */ > + emc1403@4c { > + compatible = "smsc,emc1403"; > + reg = <0x4c>; > + }; > + > +}; > + > +&i2c12 { > + status = "okay"; > + > + /* pca9545 -> > + * SAS BP1 > + * SAS BP2 > + * NVMe BP > + * M.2 riser > + */ > + > + pca9545@70 { > + compatible = "nxp,pca9545"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x70>; > + > + interrupt-controller; > + #interrupt-cells = <2>; > + > + i2c@0 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0>; > + > + eeprom@50 { > + compatible = "atmel,24c64"; > + reg = <0x50>; > + }; > + }; > + > + i2c@1 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <1>; > + > + eeprom@50 { > + compatible = "atmel,24c64"; > + reg = <0x50>; > + }; > + }; > + > + i2c@2 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <2>; > + > + eeprom@50 { > + compatible = "atmel,24c64"; > + reg = <0x50>; > + }; > + }; > + > + i2c@3 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <3>; > + > + tmp275@48 { > + compatible = "ti,tmp275"; > + reg = <0x48>; > + }; > + }; > + > + }; > + > +}; > + > +&i2c13 { > + status = "okay"; > + > + /* pca9548 -> > + * NVMe BP > + * NVMe HDD17 to 24 > + */ > + > + pca9548@70 { > + compatible = "nxp,pca9548"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x70>; > + }; > +}; > + > +&vuart { > + status = "okay"; > +}; > + > +&gfx { > + status = "okay"; > + memory-region = <&gfx_memory>; > +}; > + > +&adc { > + status = "okay"; You're still missing the pinmux properties for the ADC lines being used. Please add them. Cheers, Andrew > +}; > + > +&wdt1 { > + aspeed,reset-type = "none"; > + aspeed,external-signal; > + aspeed,ext-push-pull; > + aspeed,ext-active-high; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_wdtrst1_default>; > +}; > + > +&wdt2 { > + aspeed,alt-boot; > +}; > + > +&ibt { > + status = "okay"; > +}; > + > +&vhub { > + status = "okay"; > +}; > + > +&video { > + status = "okay"; > + memory-region = <&video_engine_memory>; > +}; > + > +#include "ibm-power9-dual.dtsi" > + > -- > 2.17.1 > > > --------------------------------------------------------------------------------------------------------------------------------------------------------------- > This email contains confidential or legally privileged information and > is for the sole use of its intended recipient. > Any unauthorized review, use, copying or distribution of this email or > the content of this email is strictly prohibited. > If you are not the intended recipient, you may reply to the sender and > should delete this e-mail immediately. > --------------------------------------------------------------------------------------------------------------------------------------------------------------- >