-----Original Message----- From: Fabio Estevam [mailto:[email protected]] Sent: 2018年10月24日 19:20 To: Joakim Zhang <[email protected]> Cc: Shawn Guo <[email protected]>; Sascha Hauer <[email protected]>; Sascha Hauer <[email protected]>; Fabio Estevam <[email protected]>; dl-linux-imx <[email protected]>; Rob Herring <[email protected]>; moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE <[email protected]>; open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS <[email protected]>; linux-kernel <[email protected]>; A.s. Dong <[email protected]> Subject: Re: [PATCH V1 3/3] ARM: dts: sabreauto: Add flexcan support
Hi Joakim, On Wed, Oct 24, 2018 at 8:15 AM Joakim Zhang <[email protected]> wrote: > diff --git a/arch/arm/boot/dts/imx6dl-sabreauto-flexcan1.dts > b/arch/arm/boot/dts/imx6dl-sabreauto-flexcan1.dts > new file mode 100644 > index 000000000000..963e0b559d60 > --- /dev/null > +++ b/arch/arm/boot/dts/imx6dl-sabreauto-flexcan1.dts > @@ -0,0 +1,19 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2013 Freescale Semiconductor, Inc. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. You can remove this paragraph as you are using SPDX tag. > +++ b/arch/arm/boot/dts/imx6q-sabreauto-flexcan1.dts > @@ -0,0 +1,19 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2013 Freescale Semiconductor, Inc. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ Ditto. > +&can1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_flexcan1>; > + pinctrl-assert-gpios = <&max7310_b 3 GPIO_ACTIVE_HIGH>; /* TX */ This property does not exist in upstream. Hi Fabio, I will modify what you have commented. Thanks a lot. Best Regards, Joakim Zhang
