Re: [PATCH v2] ARM: dts: imx6ul: Add DTS for ConnectCore 6UL SBC Pro

2018-09-09 Thread Shawn Guo
On Tue, Sep 04, 2018 at 05:28:29PM +0200, Alex Gonzalez wrote: > The ConnectCore 6UL Single Board Computer (SBC) Pro contains the > ConnectCore 6UL System-On-Module. > > Its hardware specifications are: > > * 256MB DDR3 memory > * On module 256MB NAND flash > * Dual 10/100 Ethernet > * USB Host a

Re: [PATCH v2] ARM: dts: imx6ul: Add DTS for ConnectCore 6UL SBC Pro

2018-09-05 Thread Fabio Estevam
Hi Alex, On Wed, Sep 5, 2018 at 7:25 AM, Gonzalez, Alex wrote: > My intention was to add the board support first, and then one or more panels > as independent patches. Sounds like a good approach.

Re: [PATCH v2] ARM: dts: imx6ul: Add DTS for ConnectCore 6UL SBC Pro

2018-09-05 Thread Gonzalez, Alex
Hi Fabio, thanks for the review. >Missing a panel attached to the lcdif? > My intention was to add the board support first, and then one or more panels as independent patches. Doesn't that make more sense that adding a default panel with the board support? Thanks, Alex

Re: [PATCH v2] ARM: dts: imx6ul: Add DTS for ConnectCore 6UL SBC Pro

2018-09-04 Thread Fabio Estevam
On Tue, Sep 4, 2018 at 12:28 PM, Alex Gonzalez wrote: > +&lcdif { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_lcdif_dat0_17 > +&pinctrl_lcdif_clken > +&pinctrl_lcdif_hvsync>; > + lcd-supply = <&ldo4_ext>; /* BU90T82 LVDS

[PATCH v2] ARM: dts: imx6ul: Add DTS for ConnectCore 6UL SBC Pro

2018-09-04 Thread Alex Gonzalez
The ConnectCore 6UL Single Board Computer (SBC) Pro contains the ConnectCore 6UL System-On-Module. Its hardware specifications are: * 256MB DDR3 memory * On module 256MB NAND flash * Dual 10/100 Ethernet * USB Host and USB OTG * Parallel RGB display header * LVDS display header * CSI camera * GPI

[PATCH v2] ARM: dts: imx6ul: Add DTS for ConnectCore 6UL SBC Pro

2018-09-04 Thread Alex Gonzalez
This patch adds a device tree for the ConnectCore 6UL SBC Pro board, based on the ConnectCore 6UL SOM. Changes since v1 [1]: * Removed unused pinctrl nodes * Replace undocumented vendor BSP node by using GPIO hogs * Formatting changes: * Change node names to use dashes instead of undersco