diff mbox

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

Message ID 1441068472-4899-1-git-send-email-sjg@chromium.org
State Accepted
Delegated to: Simon Glass
Headers show

Commit Message

Simon Glass Sept. 1, 2015, 12:47 a.m. UTC
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(-)

Comments

Michal Simek Sept. 1, 2015, 12:54 p.m. UTC | #1
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
Simon Glass Sept. 4, 2015, 1:10 a.m. UTC | #2
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.
diff mbox

Patch

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"