Hi Dong, On Sun, Oct 14, 2018 at 11:35 AM A.s. Dong <aisheng.dong@xxxxxxx> wrote: > > Add imx8qxp support You could probably expand the commit log by explaining a bit of what im8qxp means. > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > new file mode 100644 > index 0000000..e1d2578 > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > @@ -0,0 +1,861 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2016 Freescale Semiconductor, Inc. > + * Copyright 2017~2018 NXP I guess you meant "-" instead of "~". > + memory@80000000 { > + device_type = "memory"; > + reg = <0x00000000 0x80000000 0 0x40000000>; > + }; This should go to the board dts file instead as the memory size may vary from board to board. > + imx8qx-pm { > + compatible = "fsl,scu-pd"; > + #address-cells = <1>; > + #size-cells = <0>; > + > + pd_lsio: lsio-power-domain { > + #power-domain-cells = <0>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + pd_lsio_pwm0: lsio-pwm0@191 { > + reg = <191>; The number after @ is in hex, so the reg property should be 0x191.