Re: [U-Boot] [PATCH v4] imx: i.mx6q: add the initial support for LogicPD i.MX6Q SOM

[Stefano Babic]

On 12/04/2017 18:01, Adam Ford wrote:
> On Fri, Apr 7, 2017 at 10:25 AM, Adam Ford  wrote:
>> Logic PD has an i.MX6Q system on module (SOM) with a development kit. The
>> SOM has a built-in microSD socket, DDR and NAND flash.  The development kit
>> has an SMSC Ethernet PHY, serial debug port and a variety of peripherals.
>> This have been verified to boot the i.MX6Q version over either SD
>> on the development kit or NAND built into the SOM.  Items in the dtsi file
>> are specific to the SOM itself.  Items in the dts file are in the baseboard.
>> Future versions of the SOM will come out supporting the same basebord and
>> potentially future base boards will come out supporting the same SOM.
>>
>> Signed-off-by: Adam Ford 
>> ---
>> Changes in V4:
>>   - Eliminate DM_THERMAL
>>   - Fix typo README
>>   - Elminate more dead/unnecessary code
>>   - Re-order device tree file contents
>>   - Add I2C_EEPROM
>>   - Remove device tree items U-Boot cannot support
>>   - Fix 80 character warning
>>
> 
> Any chance this can get reviewed and/or applied before the next RC candidate?
> 

Yes - Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v4] imx: i.mx6q: add the initial support for LogicPD i.MX6Q SOM

[Adam Ford]

On Fri, Apr 7, 2017 at 10:25 AM, Adam Ford  wrote:
> Logic PD has an i.MX6Q system on module (SOM) with a development kit. The
> SOM has a built-in microSD socket, DDR and NAND flash.  The development kit
> has an SMSC Ethernet PHY, serial debug port and a variety of peripherals.
> This have been verified to boot the i.MX6Q version over either SD
> on the development kit or NAND built into the SOM.  Items in the dtsi file
> are specific to the SOM itself.  Items in the dts file are in the baseboard.
> Future versions of the SOM will come out supporting the same basebord and
> potentially future base boards will come out supporting the same SOM.
>
> Signed-off-by: Adam Ford 
> ---
> Changes in V4:
>   - Eliminate DM_THERMAL
>   - Fix typo README
>   - Elminate more dead/unnecessary code
>   - Re-order device tree file contents
>   - Add I2C_EEPROM
>   - Remove device tree items U-Boot cannot support
>   - Fix 80 character warning
>

Any chance this can get reviewed and/or applied before the next RC candidate?

> Changes in V3:
>   - Add OF_CONTROL and device tree
>   - Switch to DM_I2C
>   - Switch to DM_ETH
>   - Switch to DM_MMC
>   - Switch to DM_PMIC
>   - Switch to DM_REGULATOR
>   - Switch to DM_GPIO
>   - Fix Ethernet reset to play nicely with DM_GPIO and DM_ETH
>   - Remove more unnecessary code with DM enabled.
>
> Changes in V2:
>   - Alphabetize the modified Kconfig file
>   - Clean up README file.
>   - Elminate fdt_high and initrd_high and add bootm_size
>   - Changed MTDPARTS_DEFAULT to increase the kernel size to 16MB
>   - Cleaned up the mx6q_2x_MT41K512M16HA.cfg to include headers,
> replace hard-coded addresses with human-readable ones, and
> remove dead code
>   - Updated mx6q_2x_MT41K512M16HA.cfg to use BOOT_OFFSET since SD
> and NAND have the same offset
>   - Removed 'ifdef' references that were always true.
>   - Removed CPU references that didn't exist.
>   - And made local variables static
>   - Add reference to worked used to derrive this code
>
> diff --git a/arch/arm/cpu/armv7/mx6/Kconfig b/arch/arm/cpu/armv7/mx6/Kconfig
> index 190e5c6..b9f325f 100644
> --- a/arch/arm/cpu/armv7/mx6/Kconfig
> +++ b/arch/arm/cpu/armv7/mx6/Kconfig
> @@ -155,6 +155,19 @@ config TARGET_MX6CUBOXI
> select BOARD_LATE_INIT
> select SUPPORT_SPL
>
> +config TARGET_MX6LOGICPD
> +   bool "Logic PD i.MX6 SOM"
> +   select BOARD_EARLY_INIT_F
> +   select BOARD_LATE_INIT
> +   select DM
> +   select DM_ETH
> +   select DM_GPIO
> +   select DM_I2C
> +   select DM_MMC
> +   select DM_PMIC
> +   select DM_REGULATOR
> +   select OF_CONTROL
> +
>  config TARGET_MX6QARM2
> bool "mx6qarm2"
>
> @@ -404,6 +417,7 @@ source "board/gateworks/gw_ventana/Kconfig"
>  source "board/kosagi/novena/Kconfig"
>  source "board/samtec/vining_2000/Kconfig"
>  source "board/liebherr/mccmon6/Kconfig"
> +source "board/logicpd/imx6/Kconfig"
>  source "board/seco/Kconfig"
>  source "board/solidrun/mx6cuboxi/Kconfig"
>  source "board/technexion/pico-imx6ul/Kconfig"
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 0fbbb9b..a95f2db 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -319,6 +319,7 @@ dtb-$(CONFIG_MX6) += imx6ull-14x14-evk.dtb \
> imx6dl-icore-rqs.dtb \
> imx6q-icore.dtb \
> imx6q-icore-rqs.dtb \
> +   imx6q-logicpd.dtb \
> imx6sx-sabreauto.dtb \
> imx6ul-geam-kit.dtb \
> imx6ul-isiot-emmc.dtb \
> diff --git a/arch/arm/dts/imx6q-logicpd.dts b/arch/arm/dts/imx6q-logicpd.dts
> new file mode 100644
> index 000..d1e7a38
> --- /dev/null
> +++ b/arch/arm/dts/imx6q-logicpd.dts
> @@ -0,0 +1,190 @@
> +/*
> + * Copyright 2017 Logic PD, Inc.
> + * Based on SabreSD, Copyright 2016 Freescale Semiconductor, Inc.
> + *
> + * This file is dual-licensed: you can use it either under the terms
> + * of the GPL or the X11 license, at your option. Note that this dual
> + * licensing only applies to this file, and not this project as a
> + * whole.
> + *
> + *  a) This file is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of the
> + * License, or (at your option) any later version.
> + *
> + * This file is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * Or, alternatively,
> + *
> + *  b) Permission is hereby granted, free of charge, to any person
> + * obtaining a copy of this software and associated documentation
> + * files (the "Software"), to deal in the Software without
> + * restriction, including without limitation the rights to use,
> + * copy, modify, 

[U-Boot] [PATCH v4] imx: i.mx6q: add the initial support for LogicPD i.MX6Q SOM

[Adam Ford]

Logic PD has an i.MX6Q system on module (SOM) with a development kit. The
SOM has a built-in microSD socket, DDR and NAND flash.  The development kit
has an SMSC Ethernet PHY, serial debug port and a variety of peripherals.
This have been verified to boot the i.MX6Q version over either SD
on the development kit or NAND built into the SOM.  Items in the dtsi file
are specific to the SOM itself.  Items in the dts file are in the baseboard.
Future versions of the SOM will come out supporting the same basebord and
potentially future base boards will come out supporting the same SOM.

Signed-off-by: Adam Ford 
---
Changes in V4:
  - Eliminate DM_THERMAL
  - Fix typo README
  - Elminate more dead/unnecessary code
  - Re-order device tree file contents
  - Add I2C_EEPROM
  - Remove device tree items U-Boot cannot support
  - Fix 80 character warning

Changes in V3:
  - Add OF_CONTROL and device tree
  - Switch to DM_I2C
  - Switch to DM_ETH
  - Switch to DM_MMC
  - Switch to DM_PMIC
  - Switch to DM_REGULATOR
  - Switch to DM_GPIO
  - Fix Ethernet reset to play nicely with DM_GPIO and DM_ETH
  - Remove more unnecessary code with DM enabled.

Changes in V2:
  - Alphabetize the modified Kconfig file
  - Clean up README file.
  - Elminate fdt_high and initrd_high and add bootm_size
  - Changed MTDPARTS_DEFAULT to increase the kernel size to 16MB
  - Cleaned up the mx6q_2x_MT41K512M16HA.cfg to include headers,
replace hard-coded addresses with human-readable ones, and
remove dead code
  - Updated mx6q_2x_MT41K512M16HA.cfg to use BOOT_OFFSET since SD
and NAND have the same offset
  - Removed 'ifdef' references that were always true.
  - Removed CPU references that didn't exist.
  - And made local variables static
  - Add reference to worked used to derrive this code

diff --git a/arch/arm/cpu/armv7/mx6/Kconfig b/arch/arm/cpu/armv7/mx6/Kconfig
index 190e5c6..b9f325f 100644
--- a/arch/arm/cpu/armv7/mx6/Kconfig
+++ b/arch/arm/cpu/armv7/mx6/Kconfig
@@ -155,6 +155,19 @@ config TARGET_MX6CUBOXI
select BOARD_LATE_INIT
select SUPPORT_SPL
 
+config TARGET_MX6LOGICPD
+   bool "Logic PD i.MX6 SOM"
+   select BOARD_EARLY_INIT_F
+   select BOARD_LATE_INIT
+   select DM
+   select DM_ETH
+   select DM_GPIO
+   select DM_I2C
+   select DM_MMC
+   select DM_PMIC
+   select DM_REGULATOR
+   select OF_CONTROL
+
 config TARGET_MX6QARM2
bool "mx6qarm2"
 
@@ -404,6 +417,7 @@ source "board/gateworks/gw_ventana/Kconfig"
 source "board/kosagi/novena/Kconfig"
 source "board/samtec/vining_2000/Kconfig"
 source "board/liebherr/mccmon6/Kconfig"
+source "board/logicpd/imx6/Kconfig"
 source "board/seco/Kconfig"
 source "board/solidrun/mx6cuboxi/Kconfig"
 source "board/technexion/pico-imx6ul/Kconfig"
diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
index 0fbbb9b..a95f2db 100644
--- a/arch/arm/dts/Makefile
+++ b/arch/arm/dts/Makefile
@@ -319,6 +319,7 @@ dtb-$(CONFIG_MX6) += imx6ull-14x14-evk.dtb \
imx6dl-icore-rqs.dtb \
imx6q-icore.dtb \
imx6q-icore-rqs.dtb \
+   imx6q-logicpd.dtb \
imx6sx-sabreauto.dtb \
imx6ul-geam-kit.dtb \
imx6ul-isiot-emmc.dtb \
diff --git a/arch/arm/dts/imx6q-logicpd.dts b/arch/arm/dts/imx6q-logicpd.dts
new file mode 100644
index 000..d1e7a38
--- /dev/null
+++ b/arch/arm/dts/imx6q-logicpd.dts
@@ -0,0 +1,190 @@
+/*
+ * Copyright 2017 Logic PD, Inc.
+ * Based on SabreSD, Copyright 2016 Freescale Semiconductor, Inc.
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This file is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ * obtaining a copy of this software and associated documentation
+ * files (the "Software"), to deal in the Software without
+ * restriction, including without limitation the rights to use,
+ * copy, modify, merge, publish, distribute, sublicense, and/or
+ * sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following
+ * conditions:
+ *
+ * The above copyright notice and this permission notice shall be
+ * included in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS