[U-Boot] [PATCH v2 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[U-Boot] [PATCH v2 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[Michal Simek]

This patch follows work done by:
"Move CONFIG_OF_LIBFDT to Kconfig"
(sha1: 69e173eb57d1f4848f070c83456096ba5d2ba1b4)

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

Changes in v2:
- new patch in series. Masahiro asked for it.

 configs/legoev3_defconfig         | 2 ++
 configs/ma5d4evk_defconfig        | 2 ++
 configs/stm32f746-disco_defconfig | 2 ++
 configs/zipitz2_defconfig         | 4 +++-
 include/configs/legoev3.h         | 1 -
 include/configs/ma5d4evk.h        | 1 -
 include/configs/pic32mzdask.h     | 5 -----
 include/configs/stm32f746-disco.h | 1 -
 include/configs/xilinx-ppc.h      | 1 -
 include/configs/zipitz2.h         | 1 -
 10 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/configs/legoev3_defconfig b/configs/legoev3_defconfig
index d838baa32cbb..22fd0578ef72 100644
--- a/configs/legoev3_defconfig
+++ b/configs/legoev3_defconfig
@@ -10,3 +10,5 @@ CONFIG_AUTOBOOT_STOP_STR="l"
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SYS_NS16550=y
+CONFIG_OF_LIBFDT=y
+# CONFIG_EFI_LOADER is not set
diff --git a/configs/ma5d4evk_defconfig b/configs/ma5d4evk_defconfig
index 39ce550ff44f..144851426937 100644
--- a/configs/ma5d4evk_defconfig
+++ b/configs/ma5d4evk_defconfig
@@ -12,3 +12,5 @@ CONFIG_SYS_EXTRA_OPTIONS="SAMA5D4"
 CONFIG_SPI_FLASH=y
 CONFIG_USB=y
 CONFIG_USB_GADGET=y
+CONFIG_OF_LIBFDT=y
+# CONFIG_EFI_LOADER is not set
diff --git a/configs/stm32f746-disco_defconfig b/configs/stm32f746-disco_defconfig
index 7cfed4a514bf..07aa874760c2 100644
--- a/configs/stm32f746-disco_defconfig
+++ b/configs/stm32f746-disco_defconfig
@@ -7,3 +7,5 @@ CONFIG_AUTOBOOT_KEYED=y
 CONFIG_AUTOBOOT_PROMPT="Hit SPACE in %d seconds to stop autoboot.\n"
 CONFIG_AUTOBOOT_STOP_STR=" "
 # CONFIG_CMD_SETEXPR is not set
+CONFIG_OF_LIBFDT=y
+# CONFIG_EFI_LOADER is not set
diff --git a/configs/zipitz2_defconfig b/configs/zipitz2_defconfig
index 2977ccc8ddee..d2fa63d15f08 100644
--- a/configs/zipitz2_defconfig
+++ b/configs/zipitz2_defconfig
@@ -1,7 +1,9 @@
 CONFIG_ARM=y
 CONFIG_TARGET_ZIPITZ2=y
+CONFIG_SYS_PROMPT="$ "
 # CONFIG_CMD_IMLS is not set
 # CONFIG_CMD_SETEXPR is not set
 # CONFIG_CMD_NET is not set
 # CONFIG_CMD_NFS is not set
-CONFIG_SYS_PROMPT="$ "
+CONFIG_OF_LIBFDT=y
+# CONFIG_EFI_LOADER is not set
diff --git a/include/configs/legoev3.h b/include/configs/legoev3.h
index 79fa3c476ad1..0bc9d2dc18b4 100644
--- a/include/configs/legoev3.h
+++ b/include/configs/legoev3.h
@@ -160,7 +160,6 @@
 #define CONFIG_SYS_LONGHELP
 #define CONFIG_CRC32_VERIFY
 #define CONFIG_MX_CYCLIC
-#define CONFIG_OF_LIBFDT
 
 /*
  * Linux Information
diff --git a/include/configs/ma5d4evk.h b/include/configs/ma5d4evk.h
index e061a101aee6..8a80b954a68d 100644
--- a/include/configs/ma5d4evk.h
+++ b/include/configs/ma5d4evk.h
@@ -150,7 +150,6 @@
 #define CONFIG_LOADADDR		0x20800000
 #define CONFIG_BOOTCOMMAND	"run mmc_mmc"
 #define CONFIG_SYS_LOAD_ADDR	CONFIG_LOADADDR
-#define CONFIG_OF_LIBFDT
 
 /*
  * Extra Environments
diff --git a/include/configs/pic32mzdask.h b/include/configs/pic32mzdask.h
index 3ea11946b8ab..fd874c42a2bc 100644
--- a/include/configs/pic32mzdask.h
+++ b/include/configs/pic32mzdask.h
@@ -92,11 +92,6 @@
 #define CONFIG_BOOTP_GATEWAY
 #define CONFIG_BOOTP_HOSTNAME
 
-/*
- * Handover flattened device tree (dtb file) to Linux kernel
- */
-#define CONFIG_OF_LIBFDT	1
-
 /*-----------------------------------------------------------------------
  * SDHC Configuration
  */
diff --git a/include/configs/stm32f746-disco.h b/include/configs/stm32f746-disco.h
index 807ab6574eb5..e94812bf5a78 100644
--- a/include/configs/stm32f746-disco.h
+++ b/include/configs/stm32f746-disco.h
@@ -11,7 +11,6 @@
 #define CONFIG_SYS_THUMB_BUILD
 /*#define CONFIG_SYS_NO_FLASH*/
 
-#define CONFIG_OF_LIBFDT
 #define CONFIG_BOARD_EARLY_INIT_F
 
 #define CONFIG_SYS_FLASH_BASE		0x08000000
diff --git a/include/configs/xilinx-ppc.h b/include/configs/xilinx-ppc.h
index d01d88b33f4d..831b9401d1df 100644
--- a/include/configs/xilinx-ppc.h
+++ b/include/configs/xilinx-ppc.h
@@ -101,7 +101,6 @@
 #define CONFIG_SYS_NO_FLASH
 #endif
 
-#define CONFIG_OF_LIBFDT       1
 #define CONFIG_BAUDRATE			115200
 /* The following table includes the supported baudrates */
 # define CONFIG_SYS_BAUDRATE_TABLE \
diff --git a/include/configs/zipitz2.h b/include/configs/zipitz2.h
index 5200e0219168..46836d578c0a 100644
--- a/include/configs/zipitz2.h
+++ b/include/configs/zipitz2.h
@@ -45,7 +45,6 @@
 #define	CONFIG_SETUP_MEMORY_TAGS
 #define	CONFIG_SYS_TEXT_BASE		0x0
 #define	CONFIG_LZMA			/* LZMA compression support */
-#define	CONFIG_OF_LIBFDT
 
 /*
  * Serial Console Configuration
-- 
1.9.1

[U-Boot] [PATCH v2 2/2] cmd: fdt: Use separate CMD_FDT Kconfig entry instead of OF_LIBFDT

[Michal Simek]

Create CMD_FDT Kconfig entry to have an option to disable fdt command
which is not required for small configuration which requires libfdt
only.
Enable it by default for all targets which enables OF_LIBFDT.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

Changes in v2: None

 cmd/Kconfig  | 7 +++++++
 cmd/Makefile | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/cmd/Kconfig b/cmd/Kconfig
index fe8b4f0510da..8703cdb4a9be 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -173,6 +173,13 @@ config CMD_ELF
 	help
 	  Boot an ELF/vxWorks image from the memory.
 
+config CMD_FDT
+	bool "Flattened Device Tree utility commands"
+	default y
+	depends on OF_LIBFDT
+	help
+	  Do FDT related setup before booting into the Operating System.
+
 config CMD_GO
 	bool "go"
 	default y
diff --git a/cmd/Makefile b/cmd/Makefile
index ba041973079c..f95759e67044 100644
--- a/cmd/Makefile
+++ b/cmd/Makefile
@@ -54,7 +54,7 @@ obj-$(CONFIG_CMD_EXT4) += ext4.o
 obj-$(CONFIG_CMD_EXT2) += ext2.o
 obj-$(CONFIG_CMD_FAT) += fat.o
 obj-$(CONFIG_CMD_FDC) += fdc.o
-obj-$(CONFIG_OF_LIBFDT) += fdt.o
+obj-$(CONFIG_CMD_FDT) += fdt.o
 obj-$(CONFIG_CMD_FITUPD) += fitupd.o
 obj-$(CONFIG_CMD_FLASH) += flash.o
 ifdef CONFIG_FPGA
-- 
1.9.1

[U-Boot] [PATCH v2 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[Ricardo Ribalda Delgado]

Hi Michal

xilinx-ppc405-generic and xilinx-ppc440-generic also depend on
xilinx-ppc.h So I think this is missing.

Shall I send this as a separated patch? or you want to include it in yours?


Regards


ricardo at pilix:~/curro/u-boot$ git diff
diff --git a/configs/xilinx-ppc405-generic_defconfig
b/configs/xilinx-ppc405-generic_defconfig
index 2008a8d79394..381b668bb9e4 100644
--- a/configs/xilinx-ppc405-generic_defconfig
+++ b/configs/xilinx-ppc405-generic_defconfig
@@ -13,3 +13,5 @@ CONFIG_SYS_PROMPT="xlx-ppc405:/# "
 CONFIG_OF_EMBED=y
 CONFIG_SYS_NS16550=y
 CONFIG_XILINX_UARTLITE=y
+CONFIG_OF_LIBFDT=y
+# CONFIG_EFI_LOADER is not set
diff --git a/configs/xilinx-ppc440-generic_defconfig
b/configs/xilinx-ppc440-generic_defconfig
index 8df33d3270d1..9fe518a83dc3 100644
--- a/configs/xilinx-ppc440-generic_defconfig
+++ b/configs/xilinx-ppc440-generic_defconfig
@@ -13,3 +13,5 @@ CONFIG_OF_EMBED=y
 CONFIG_NETCONSOLE=y
 CONFIG_SYS_NS16550=y
 CONFIG_XILINX_UARTLITE=y
+CONFIG_OF_LIBFDT=y
+# CONFIG_EFI_LOADER is not set

On Wed, Apr 6, 2016 at 8:28 PM, Michal Simek <michal.simek@xilinx.com> wrote:
> This patch follows work done by:
> "Move CONFIG_OF_LIBFDT to Kconfig"
> (sha1: 69e173eb57d1f4848f070c83456096ba5d2ba1b4)
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> Changes in v2:
> - new patch in series. Masahiro asked for it.
>
>  configs/legoev3_defconfig         | 2 ++
>  configs/ma5d4evk_defconfig        | 2 ++
>  configs/stm32f746-disco_defconfig | 2 ++
>  configs/zipitz2_defconfig         | 4 +++-
>  include/configs/legoev3.h         | 1 -
>  include/configs/ma5d4evk.h        | 1 -
>  include/configs/pic32mzdask.h     | 5 -----
>  include/configs/stm32f746-disco.h | 1 -
>  include/configs/xilinx-ppc.h      | 1 -
>  include/configs/zipitz2.h         | 1 -
>  10 files changed, 9 insertions(+), 11 deletions(-)
>
> diff --git a/configs/legoev3_defconfig b/configs/legoev3_defconfig
> index d838baa32cbb..22fd0578ef72 100644
> --- a/configs/legoev3_defconfig
> +++ b/configs/legoev3_defconfig
> @@ -10,3 +10,5 @@ CONFIG_AUTOBOOT_STOP_STR="l"
>  CONFIG_SPI_FLASH=y
>  CONFIG_SPI_FLASH_STMICRO=y
>  CONFIG_SYS_NS16550=y
> +CONFIG_OF_LIBFDT=y
> +# CONFIG_EFI_LOADER is not set
> diff --git a/configs/ma5d4evk_defconfig b/configs/ma5d4evk_defconfig
> index 39ce550ff44f..144851426937 100644
> --- a/configs/ma5d4evk_defconfig
> +++ b/configs/ma5d4evk_defconfig
> @@ -12,3 +12,5 @@ CONFIG_SYS_EXTRA_OPTIONS="SAMA5D4"
>  CONFIG_SPI_FLASH=y
>  CONFIG_USB=y
>  CONFIG_USB_GADGET=y
> +CONFIG_OF_LIBFDT=y
> +# CONFIG_EFI_LOADER is not set
> diff --git a/configs/stm32f746-disco_defconfig b/configs/stm32f746-disco_defconfig
> index 7cfed4a514bf..07aa874760c2 100644
> --- a/configs/stm32f746-disco_defconfig
> +++ b/configs/stm32f746-disco_defconfig
> @@ -7,3 +7,5 @@ CONFIG_AUTOBOOT_KEYED=y
>  CONFIG_AUTOBOOT_PROMPT="Hit SPACE in %d seconds to stop autoboot.\n"
>  CONFIG_AUTOBOOT_STOP_STR=" "
>  # CONFIG_CMD_SETEXPR is not set
> +CONFIG_OF_LIBFDT=y
> +# CONFIG_EFI_LOADER is not set
> diff --git a/configs/zipitz2_defconfig b/configs/zipitz2_defconfig
> index 2977ccc8ddee..d2fa63d15f08 100644
> --- a/configs/zipitz2_defconfig
> +++ b/configs/zipitz2_defconfig
> @@ -1,7 +1,9 @@
>  CONFIG_ARM=y
>  CONFIG_TARGET_ZIPITZ2=y
> +CONFIG_SYS_PROMPT="$ "
>  # CONFIG_CMD_IMLS is not set
>  # CONFIG_CMD_SETEXPR is not set
>  # CONFIG_CMD_NET is not set
>  # CONFIG_CMD_NFS is not set
> -CONFIG_SYS_PROMPT="$ "
> +CONFIG_OF_LIBFDT=y
> +# CONFIG_EFI_LOADER is not set
> diff --git a/include/configs/legoev3.h b/include/configs/legoev3.h
> index 79fa3c476ad1..0bc9d2dc18b4 100644
> --- a/include/configs/legoev3.h
> +++ b/include/configs/legoev3.h
> @@ -160,7 +160,6 @@
>  #define CONFIG_SYS_LONGHELP
>  #define CONFIG_CRC32_VERIFY
>  #define CONFIG_MX_CYCLIC
> -#define CONFIG_OF_LIBFDT
>
>  /*
>   * Linux Information
> diff --git a/include/configs/ma5d4evk.h b/include/configs/ma5d4evk.h
> index e061a101aee6..8a80b954a68d 100644
> --- a/include/configs/ma5d4evk.h
> +++ b/include/configs/ma5d4evk.h
> @@ -150,7 +150,6 @@
>  #define CONFIG_LOADADDR                0x20800000
>  #define CONFIG_BOOTCOMMAND     "run mmc_mmc"
>  #define CONFIG_SYS_LOAD_ADDR   CONFIG_LOADADDR
> -#define CONFIG_OF_LIBFDT
>
>  /*
>   * Extra Environments
> diff --git a/include/configs/pic32mzdask.h b/include/configs/pic32mzdask.h
> index 3ea11946b8ab..fd874c42a2bc 100644
> --- a/include/configs/pic32mzdask.h
> +++ b/include/configs/pic32mzdask.h
> @@ -92,11 +92,6 @@
>  #define CONFIG_BOOTP_GATEWAY
>  #define CONFIG_BOOTP_HOSTNAME
>
> -/*
> - * Handover flattened device tree (dtb file) to Linux kernel
> - */
> -#define CONFIG_OF_LIBFDT       1
> -
>  /*-----------------------------------------------------------------------
>   * SDHC Configuration
>   */
> diff --git a/include/configs/stm32f746-disco.h b/include/configs/stm32f746-disco.h
> index 807ab6574eb5..e94812bf5a78 100644
> --- a/include/configs/stm32f746-disco.h
> +++ b/include/configs/stm32f746-disco.h
> @@ -11,7 +11,6 @@
>  #define CONFIG_SYS_THUMB_BUILD
>  /*#define CONFIG_SYS_NO_FLASH*/
>
> -#define CONFIG_OF_LIBFDT
>  #define CONFIG_BOARD_EARLY_INIT_F
>
>  #define CONFIG_SYS_FLASH_BASE          0x08000000
> diff --git a/include/configs/xilinx-ppc.h b/include/configs/xilinx-ppc.h
> index d01d88b33f4d..831b9401d1df 100644
> --- a/include/configs/xilinx-ppc.h
> +++ b/include/configs/xilinx-ppc.h
> @@ -101,7 +101,6 @@
>  #define CONFIG_SYS_NO_FLASH
>  #endif
>
> -#define CONFIG_OF_LIBFDT       1
>  #define CONFIG_BAUDRATE                        115200
>  /* The following table includes the supported baudrates */
>  # define CONFIG_SYS_BAUDRATE_TABLE \
> diff --git a/include/configs/zipitz2.h b/include/configs/zipitz2.h
> index 5200e0219168..46836d578c0a 100644
> --- a/include/configs/zipitz2.h
> +++ b/include/configs/zipitz2.h
> @@ -45,7 +45,6 @@
>  #define        CONFIG_SETUP_MEMORY_TAGS
>  #define        CONFIG_SYS_TEXT_BASE            0x0
>  #define        CONFIG_LZMA                     /* LZMA compression support */
> -#define        CONFIG_OF_LIBFDT
>
>  /*
>   * Serial Console Configuration
> --
> 1.9.1
>



-- 
Ricardo Ribalda

[U-Boot] [PATCH v2 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[Michal Simek]

Hi,

On 6.4.2016 22:34, Ricardo Ribalda Delgado wrote:
> Hi Michal
> 
> xilinx-ppc405-generic and xilinx-ppc440-generic also depend on
> xilinx-ppc.h So I think this is missing.
> 
> Shall I send this as a separated patch? or you want to include it in yours?

I was checking that and LIBFDT is enabled by default that's why there
was not defconfig update. Definitely feel free to check it and test if
this patch breaks something.

Thanks,
Michal

[U-Boot] [PATCH v2 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[Ricardo Ribalda Delgado]

Hi Michal

You are right,

OF_CONTROL (on  arch/powerpc/cpu/ppc4xx/Kconfig ) selects OF_LIBFDT by
default (on lib/Kconfig)

So no need for my changes


Thanks!

On Thu, Apr 7, 2016 at 8:06 AM, Michal Simek <michal.simek@xilinx.com> wrote:
> Hi,
>
> On 6.4.2016 22:34, Ricardo Ribalda Delgado wrote:
>> Hi Michal
>>
>> xilinx-ppc405-generic and xilinx-ppc440-generic also depend on
>> xilinx-ppc.h So I think this is missing.
>>
>> Shall I send this as a separated patch? or you want to include it in yours?
>
> I was checking that and LIBFDT is enabled by default that's why there
> was not defconfig update. Definitely feel free to check it and test if
> this patch breaks something.
>
> Thanks,
> Michal



-- 
Ricardo Ribalda

[U-Boot] [PATCH v2 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[Michal Simek]

On 7.4.2016 14:37, Ricardo Ribalda Delgado wrote:
> Hi Michal
> 
> You are right,
> 
> OF_CONTROL (on  arch/powerpc/cpu/ppc4xx/Kconfig ) selects OF_LIBFDT by
> default (on lib/Kconfig)
> 
> So no need for my changes

ok. Your Tested-by or Reviewed-by flag would help.

Thanks,
Michal

[U-Boot] [PATCH v2 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[Ricardo Ribalda Delgado]

For xilinx-ppc405-generic and xilinx-ppc440-generic:

Reviewed-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>

I have access to my hardware again from the 20th of April. I can give
you a tested-by by them if you want ;)

regards!

On Thu, Apr 7, 2016 at 2:39 PM, Michal Simek <michal.simek@xilinx.com> wrote:
> On 7.4.2016 14:37, Ricardo Ribalda Delgado wrote:
>> Hi Michal
>>
>> You are right,
>>
>> OF_CONTROL (on  arch/powerpc/cpu/ppc4xx/Kconfig ) selects OF_LIBFDT by
>> default (on lib/Kconfig)
>>
>> So no need for my changes
>
> ok. Your Tested-by or Reviewed-by flag would help.
>
> Thanks,
> Michal
>



-- 
Ricardo Ribalda

[U-Boot] [PATCH v2 2/2] cmd: fdt: Use separate CMD_FDT Kconfig entry instead of OF_LIBFDT

[Simon Glass]

Hi Michal,

On 6 April 2016 at 12:28, Michal Simek <michal.simek@xilinx.com> wrote:
> Create CMD_FDT Kconfig entry to have an option to disable fdt command
> which is not required for small configuration which requires libfdt
> only.
> Enable it by default for all targets which enables OF_LIBFDT.
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> Changes in v2: None
>
>  cmd/Kconfig  | 7 +++++++
>  cmd/Makefile | 2 +-
>  2 files changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index fe8b4f0510da..8703cdb4a9be 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -173,6 +173,13 @@ config CMD_ELF
>         help
>           Boot an ELF/vxWorks image from the memory.
>
> +config CMD_FDT
> +       bool "Flattened Device Tree utility commands"
> +       default y
> +       depends on OF_LIBFDT
> +       help
> +         Do FDT related setup before booting into the Operating System.
> +
>  config CMD_GO
>         bool "go"
>         default y
> diff --git a/cmd/Makefile b/cmd/Makefile
> index ba041973079c..f95759e67044 100644
> --- a/cmd/Makefile
> +++ b/cmd/Makefile
> @@ -54,7 +54,7 @@ obj-$(CONFIG_CMD_EXT4) += ext4.o
>  obj-$(CONFIG_CMD_EXT2) += ext2.o
>  obj-$(CONFIG_CMD_FAT) += fat.o
>  obj-$(CONFIG_CMD_FDC) += fdc.o
> -obj-$(CONFIG_OF_LIBFDT) += fdt.o
> +obj-$(CONFIG_CMD_FDT) += fdt.o

Won't this disable the command for lots of boards?

>  obj-$(CONFIG_CMD_FITUPD) += fitupd.o
>  obj-$(CONFIG_CMD_FLASH) += flash.o
>  ifdef CONFIG_FPGA
> --
> 1.9.1
>

Regards,
Simon

[U-Boot] [PATCH v2 2/2] cmd: fdt: Use separate CMD_FDT Kconfig entry instead of OF_LIBFDT

[Michal Simek]

On 9.4.2016 20:36, Simon Glass wrote:
> Hi Michal,
> 
> On 6 April 2016 at 12:28, Michal Simek <michal.simek@xilinx.com> wrote:
>> Create CMD_FDT Kconfig entry to have an option to disable fdt command
>> which is not required for small configuration which requires libfdt
>> only.
>> Enable it by default for all targets which enables OF_LIBFDT.
>>
>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>> ---
>>
>> Changes in v2: None
>>
>>  cmd/Kconfig  | 7 +++++++
>>  cmd/Makefile | 2 +-
>>  2 files changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/cmd/Kconfig b/cmd/Kconfig
>> index fe8b4f0510da..8703cdb4a9be 100644
>> --- a/cmd/Kconfig
>> +++ b/cmd/Kconfig
>> @@ -173,6 +173,13 @@ config CMD_ELF
>>         help
>>           Boot an ELF/vxWorks image from the memory.
>>
>> +config CMD_FDT
>> +       bool "Flattened Device Tree utility commands"
>> +       default y
>> +       depends on OF_LIBFDT
>> +       help
>> +         Do FDT related setup before booting into the Operating System.
>> +
>>  config CMD_GO
>>         bool "go"
>>         default y
>> diff --git a/cmd/Makefile b/cmd/Makefile
>> index ba041973079c..f95759e67044 100644
>> --- a/cmd/Makefile
>> +++ b/cmd/Makefile
>> @@ -54,7 +54,7 @@ obj-$(CONFIG_CMD_EXT4) += ext4.o
>>  obj-$(CONFIG_CMD_EXT2) += ext2.o
>>  obj-$(CONFIG_CMD_FAT) += fat.o
>>  obj-$(CONFIG_CMD_FDC) += fdc.o
>> -obj-$(CONFIG_OF_LIBFDT) += fdt.o
>> +obj-$(CONFIG_CMD_FDT) += fdt.o
> 
> Won't this disable the command for lots of boards?

Also based on Masahiro response boards shouldn't be affected.
Only that 6 boards needs to be fixed which is what I have done in 1/2.
http://lists.denx.de/pipermail/u-boot/2016-April/250628.html

Thanks,
Michal

[U-Boot] [PATCH v2 2/2] cmd: fdt: Use separate CMD_FDT Kconfig entry instead of OF_LIBFDT

[Michal Simek]

Hi Tom,

On 11.4.2016 07:33, Michal Simek wrote:
> On 9.4.2016 20:36, Simon Glass wrote:
>> Hi Michal,
>>
>> On 6 April 2016 at 12:28, Michal Simek <michal.simek@xilinx.com> wrote:
>>> Create CMD_FDT Kconfig entry to have an option to disable fdt command
>>> which is not required for small configuration which requires libfdt
>>> only.
>>> Enable it by default for all targets which enables OF_LIBFDT.
>>>
>>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>>> ---
>>>
>>> Changes in v2: None
>>>
>>>  cmd/Kconfig  | 7 +++++++
>>>  cmd/Makefile | 2 +-
>>>  2 files changed, 8 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/cmd/Kconfig b/cmd/Kconfig
>>> index fe8b4f0510da..8703cdb4a9be 100644
>>> --- a/cmd/Kconfig
>>> +++ b/cmd/Kconfig
>>> @@ -173,6 +173,13 @@ config CMD_ELF
>>>         help
>>>           Boot an ELF/vxWorks image from the memory.
>>>
>>> +config CMD_FDT
>>> +       bool "Flattened Device Tree utility commands"
>>> +       default y
>>> +       depends on OF_LIBFDT
>>> +       help
>>> +         Do FDT related setup before booting into the Operating System.
>>> +
>>>  config CMD_GO
>>>         bool "go"
>>>         default y
>>> diff --git a/cmd/Makefile b/cmd/Makefile
>>> index ba041973079c..f95759e67044 100644
>>> --- a/cmd/Makefile
>>> +++ b/cmd/Makefile
>>> @@ -54,7 +54,7 @@ obj-$(CONFIG_CMD_EXT4) += ext4.o
>>>  obj-$(CONFIG_CMD_EXT2) += ext2.o
>>>  obj-$(CONFIG_CMD_FAT) += fat.o
>>>  obj-$(CONFIG_CMD_FDC) += fdc.o
>>> -obj-$(CONFIG_OF_LIBFDT) += fdt.o
>>> +obj-$(CONFIG_CMD_FDT) += fdt.o
>>
>> Won't this disable the command for lots of boards?
> 
> Also based on Masahiro response boards shouldn't be affected.
> Only that 6 boards needs to be fixed which is what I have done in 1/2.
> http://lists.denx.de/pipermail/u-boot/2016-April/250628.html
Do you see any problem with these 2 patches?
When this is resolved I can push mini configuration for zynqmp.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160414/9052b90e/attachment.sig>

[U-Boot] [U-Boot, v2, 1/2] kconfig: Move CONFIG_OF_LIBFDT to Kconfig

[Tom Rini]

On Wed, Apr 06, 2016 at 08:28:03PM +0200, Michal Simek wrote:

> This patch follows work done by:
> "Move CONFIG_OF_LIBFDT to Kconfig"
> (sha1: 69e173eb57d1f4848f070c83456096ba5d2ba1b4)
> 
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160421/6ff80e5e/attachment.sig>

[U-Boot] [U-Boot, v2, 2/2] cmd: fdt: Use separate CMD_FDT Kconfig entry instead of OF_LIBFDT

[Tom Rini]

On Wed, Apr 06, 2016 at 08:28:04PM +0200, Michal Simek wrote:

> Create CMD_FDT Kconfig entry to have an option to disable fdt command
> which is not required for small configuration which requires libfdt
> only.
> Enable it by default for all targets which enables OF_LIBFDT.
> 
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160421/d0e3b78d/attachment.sig>