[PATCH linux dev-5.4] ARM: dts: nuvoton: Update EVB for new PECI layout

Jae Hyun Yoo jae.hyun.yoo at linux.intel.com
Wed Dec 18 19:13:31 AEDT 2019


On 12/17/2019 8:22 PM, Joel Stanley wrote:
> The PECI device tree layout changed, breaking the EVB.
> 
> Signed-off-by: Joel Stanley <joel at jms.id.au>
> ---
>   arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 25 ++++++++++++-----------
>   1 file changed, 13 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> index de9c668c6e9a..e02c04910ef0 100644
> --- a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts
> @@ -492,18 +492,6 @@
>   				};
>   			};
>   
> -			peci: peci-bus at 100000 {
> -				cmd-timeout-ms = <1000>;
> -				pull-down = <0>;
> -				host-neg-bit-rate = <15>;
> -				status = "okay";
> -				intel-peci-dimmtemp at 30 {
> -					compatible = "intel,peci-client";
> -					reg = <0x30>;
> -					status = "okay";
> -				};
> -			};
> -
>   			spi0: spi at 200000 {
>   				cs-gpios = <&gpio6 11 GPIO_ACTIVE_LOW>;
>   				status = "okay";
> @@ -610,3 +598,16 @@
>   		idle-states = <2>; /* Serial port mode 3 (takeover) */
>   	};
>   };
> +
> +&peci0 {
> +	cmd-timeout-ms = <1000>;
> +	pull-down = <0>;
> +	host-neg-bit-rate = <15>;
> +	status = "okay";
> +
> +	intel-peci-dimmtemp at 30 {
> +		compatible = "intel,peci-client";
> +		reg = <0x30>;
> +		status = "okay";
> +	};
> +};
> 

Yes, I fixed peci nodes in 'nuvoton-common-npcm7xx.dtsi' using the new
PECI patch set because previous node setting was incorrect. PECI node
shape should be like:

bus: {
     controller: {
         client: {
         };
     };
};

so this fix is needed for nuvoton EVB.

Acked-by: Jae Hyun Yoo <jae.hyun.yoo at linux.intel.com>


More information about the openbmc mailing list