* [U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use
@ 2015-09-01 0:47 Simon Glass
2015-09-01 12:54 ` Michal Simek
0 siblings, 1 reply; 3+ messages in thread
From: Simon Glass @ 2015-09-01 0:47 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use
2015-09-01 0:47 [U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use Simon Glass
@ 2015-09-01 12:54 ` Michal Simek
2015-09-04 1:10 ` Simon Glass
0 siblings, 1 reply; 3+ messages in thread
From: Michal Simek @ 2015-09-01 12:54 UTC (permalink / raw)
To: u-boot
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>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use
2015-09-01 12:54 ` Michal Simek
@ 2015-09-04 1:10 ` Simon Glass
0 siblings, 0 replies; 3+ messages in thread
From: Simon Glass @ 2015-09-04 1:10 UTC (permalink / raw)
To: u-boot
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.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-09-04 1:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-01 0:47 [U-Boot] [PATCH] dts: Add a comment about CONFIG_OF_EMBED being for local use Simon Glass
2015-09-01 12:54 ` Michal Simek
2015-09-04 1:10 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.