public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2] config_distro_bootcmd.h: Enable CONFIG_CMD_PART
@ 2015-02-07 12:38 Hans de Goede
  2015-02-07 12:38 ` Hans de Goede
  0 siblings, 1 reply; 4+ messages in thread
From: Hans de Goede @ 2015-02-07 12:38 UTC (permalink / raw)
  To: u-boot

Hi Stephen & Tom,

Here is v2 of the patch to fix config_distro_bootcmd usage for sunxi
(and possibly others). As requested I've removed the enabling of
CONFIG_CMD_PART to config_distro_bootcmd.h .

Stephen, can you please review and ack ?

Tom, can you please merge this once acked ?

Regards,

Hans

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v2] config_distro_bootcmd.h: Enable CONFIG_CMD_PART
  2015-02-07 12:38 [U-Boot] [PATCH v2] config_distro_bootcmd.h: Enable CONFIG_CMD_PART Hans de Goede
@ 2015-02-07 12:38 ` Hans de Goede
  2015-02-09 17:28   ` Stephen Warren
  2015-02-17 20:22   ` [U-Boot] [U-Boot, " Tom Rini
  0 siblings, 2 replies; 4+ messages in thread
From: Hans de Goede @ 2015-02-07 12:38 UTC (permalink / raw)
  To: u-boot

The recent changes to config_distro_bootcmd.h require CONFIG_CMD_PART to be
defined, as the default bootcmd now uses the "part" command.

This fixes sunxi boards not booting with v2015.04-rc1.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 include/config_distro_bootcmd.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/include/config_distro_bootcmd.h b/include/config_distro_bootcmd.h
index 49674f4..07a0b3b 100644
--- a/include/config_distro_bootcmd.h
+++ b/include/config_distro_bootcmd.h
@@ -10,6 +10,10 @@
 #ifndef _CONFIG_CMD_DISTRO_BOOTCMD_H
 #define _CONFIG_CMD_DISTRO_BOOTCMD_H
 
+/* We need the part command */
+#define CONFIG_PARTITION_UUIDS
+#define CONFIG_CMD_PART
+
 #define BOOTENV_SHARED_BLKDEV_BODY(devtypel) \
 		"if " #devtypel " dev ${devnum}; then " \
 			"setenv devtype " #devtypel "; " \
-- 
2.2.2

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v2] config_distro_bootcmd.h: Enable CONFIG_CMD_PART
  2015-02-07 12:38 ` Hans de Goede
@ 2015-02-09 17:28   ` Stephen Warren
  2015-02-17 20:22   ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 0 replies; 4+ messages in thread
From: Stephen Warren @ 2015-02-09 17:28 UTC (permalink / raw)
  To: u-boot

On 02/07/2015 05:38 AM, Hans de Goede wrote:
> The recent changes to config_distro_bootcmd.h require CONFIG_CMD_PART to be
> defined, as the default bootcmd now uses the "part" command.
>
> This fixes sunxi boards not booting with v2015.04-rc1.

Acked-by: Stephen Warren <swarren@nvidia.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [U-Boot, v2] config_distro_bootcmd.h: Enable CONFIG_CMD_PART
  2015-02-07 12:38 ` Hans de Goede
  2015-02-09 17:28   ` Stephen Warren
@ 2015-02-17 20:22   ` Tom Rini
  1 sibling, 0 replies; 4+ messages in thread
From: Tom Rini @ 2015-02-17 20:22 UTC (permalink / raw)
  To: u-boot

On Sat, Feb 07, 2015 at 01:38:19PM +0100, Hans de Goede wrote:

> The recent changes to config_distro_bootcmd.h require CONFIG_CMD_PART to be
> defined, as the default bootcmd now uses the "part" command.
> 
> This fixes sunxi boards not booting with v2015.04-rc1.
> 
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> Acked-by: Stephen Warren <swarren@nvidia.com>

Applied to u-boot/master, thanks!

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-02-17 20:22 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-02-07 12:38 [U-Boot] [PATCH v2] config_distro_bootcmd.h: Enable CONFIG_CMD_PART Hans de Goede
2015-02-07 12:38 ` Hans de Goede
2015-02-09 17:28   ` Stephen Warren
2015-02-17 20:22   ` [U-Boot] [U-Boot, " Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox