[U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use

[U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use

[Simon Glass]

This comment from README.fdt-control did not end up in the Kconfig, which
is what most people will see. Add it with a few tweaks.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 dts/Kconfig | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/dts/Kconfig b/dts/Kconfig
index a04bb96..a66b9e3 100644
--- a/dts/Kconfig
+++ b/dts/Kconfig
@@ -37,7 +37,9 @@ config OF_EMBED
 	bool "Embedded DTB for DT control"
 	help
 	  If this option is enabled, the device tree will be picked up and
-	  built into the U-Boot image.
+	  built into the U-Boot image. This is suitable for local debugging
+	  and development only and is not recommended for production devices.
+	  Boards in the mainline U-Boot tree should not use it.
 
 config OF_HOSTFILE
 	bool "Host filed DTB for DT control"
-- 
2.5.0.457.gab17608

[U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use

[Michal Simek]

On 09/01/2015 02:47 AM, Simon Glass wrote:
> This comment from README.fdt-control did not end up in the Kconfig, which
> is what most people will see. Add it with a few tweaks.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
> 
>  dts/Kconfig | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/dts/Kconfig b/dts/Kconfig
> index a04bb96..a66b9e3 100644
> --- a/dts/Kconfig
> +++ b/dts/Kconfig
> @@ -37,7 +37,9 @@ config OF_EMBED
>  	bool "Embedded DTB for DT control"
>  	help
>  	  If this option is enabled, the device tree will be picked up and
> -	  built into the U-Boot image.
> +	  built into the U-Boot image. This is suitable for local debugging
> +	  and development only and is not recommended for production devices.
> +	  Boards in the mainline U-Boot tree should not use it.
>  
>  config OF_HOSTFILE
>  	bool "Host filed DTB for DT control"
> 

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

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/20150901/ac7c051e/attachment.sig>

[U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use

[Simon Glass]

On 1 September 2015 at 06:54, Michal Simek <monstr@monstr.eu> wrote:
> On 09/01/2015 02:47 AM, Simon Glass wrote:
>> This comment from README.fdt-control did not end up in the Kconfig, which
>> is what most people will see. Add it with a few tweaks.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>
>>  dts/Kconfig | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/dts/Kconfig b/dts/Kconfig
>> index a04bb96..a66b9e3 100644
>> --- a/dts/Kconfig
>> +++ b/dts/Kconfig
>> @@ -37,7 +37,9 @@ config OF_EMBED
>>       bool "Embedded DTB for DT control"
>>       help
>>         If this option is enabled, the device tree will be picked up and
>> -       built into the U-Boot image.
>> +       built into the U-Boot image. This is suitable for local debugging
>> +       and development only and is not recommended for production devices.
>> +       Boards in the mainline U-Boot tree should not use it.
>>
>>  config OF_HOSTFILE
>>       bool "Host filed DTB for DT control"
>>
>
> Acked-by: Michal Simek <michal.simek@xilinx.com>

I'd like to get this comment in for the current release.

Applied to u-boot-dm.