* [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup
@ 2016-01-04 22:07 Enric Balletbo i Serra
2016-01-04 22:07 ` [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR Enric Balletbo i Serra
` (5 more replies)
0 siblings, 6 replies; 22+ messages in thread
From: Enric Balletbo i Serra @ 2016-01-04 22:07 UTC (permalink / raw)
To: u-boot
Hi Tom,
The following patches are minor cleanups that I think are safe to apply. I
know that we're very close to next release so it's up to you apply now or
wait for the next merge window. I'd like to see it merged but I don't any
objection if you decide wait.
Thanks,
Enric
Enric Balletbo i Serra (1):
igep00x0: enable CONFIG_NET_RANDOM_ETHADDR
Ladislav Michl (4):
igep00x0: Cleanup ethernet support
igep00x0: Do not include config_distro_defaults.h
igep00x0: Fix config header indentation
igep00x0: Remove no-op macros from config header
board/isee/igep00x0/igep00x0.c | 33 +++++++++++++++++++--------------
configs/igep0020_defconfig | 1 +
configs/igep0020_nand_defconfig | 1 +
configs/igep0032_defconfig | 1 +
include/configs/omap3_igep00x0.h | 13 +++----------
5 files changed, 25 insertions(+), 24 deletions(-)
--
2.1.0
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR
2016-01-04 22:07 [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Enric Balletbo i Serra
@ 2016-01-04 22:07 ` Enric Balletbo i Serra
2016-01-04 22:23 ` Tom Rini
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:07 ` [U-Boot] [PATCHv2 2/5] igep00x0: Cleanup ethernet support Enric Balletbo i Serra
` (4 subsequent siblings)
5 siblings, 2 replies; 22+ messages in thread
From: Enric Balletbo i Serra @ 2016-01-04 22:07 UTC (permalink / raw)
To: u-boot
Enable CONFIG_NET_RANDOM_ETHADDR to generate a random MAC address
when ETHADDR is not set.
Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
---
configs/igep0020_defconfig | 1 +
configs/igep0020_nand_defconfig | 1 +
configs/igep0032_defconfig | 1 +
3 files changed, 3 insertions(+)
diff --git a/configs/igep0020_defconfig b/configs/igep0020_defconfig
index b9e6711..a96c130 100644
--- a/configs/igep0020_defconfig
+++ b/configs/igep0020_defconfig
@@ -8,3 +8,4 @@ CONFIG_SYS_EXTRA_OPTIONS="MACH_TYPE=MACH_TYPE_IGEP0020,BOOT_ONENAND"
CONFIG_CMD_GPIO=y
# CONFIG_CMD_SETEXPR is not set
CONFIG_SYS_NS16550=y
+CONFIG_NET_RANDOM_ETHADDR=y
diff --git a/configs/igep0020_nand_defconfig b/configs/igep0020_nand_defconfig
index 011ebac..cf3ee79 100644
--- a/configs/igep0020_nand_defconfig
+++ b/configs/igep0020_nand_defconfig
@@ -8,3 +8,4 @@ CONFIG_SYS_EXTRA_OPTIONS="MACH_TYPE=MACH_TYPE_IGEP0020,BOOT_NAND"
CONFIG_CMD_GPIO=y
# CONFIG_CMD_SETEXPR is not set
CONFIG_SYS_NS16550=y
+CONFIG_NET_RANDOM_ETHADDR=y
diff --git a/configs/igep0032_defconfig b/configs/igep0032_defconfig
index ba160ef..b7f1d61 100644
--- a/configs/igep0032_defconfig
+++ b/configs/igep0032_defconfig
@@ -8,3 +8,4 @@ CONFIG_SYS_EXTRA_OPTIONS="MACH_TYPE=MACH_TYPE_IGEP0032,BOOT_ONENAND"
CONFIG_CMD_GPIO=y
# CONFIG_CMD_SETEXPR is not set
CONFIG_SYS_NS16550=y
+CONFIG_NET_RANDOM_ETHADDR=y
--
2.1.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 2/5] igep00x0: Cleanup ethernet support
2016-01-04 22:07 [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Enric Balletbo i Serra
2016-01-04 22:07 ` [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR Enric Balletbo i Serra
@ 2016-01-04 22:07 ` Enric Balletbo i Serra
2016-01-20 5:24 ` [U-Boot] [U-Boot, PATCHv2, " Heiko Schocher
2016-01-20 21:01 ` Tom Rini
2016-01-04 22:08 ` [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h Enric Balletbo i Serra
` (3 subsequent siblings)
5 siblings, 2 replies; 22+ messages in thread
From: Enric Balletbo i Serra @ 2016-01-04 22:07 UTC (permalink / raw)
To: u-boot
From: Ladislav Michl <ladis@linux-mips.org>
- move chip reset to separate function
- use CONFIG_SMC911X_BASE instead of hardcoded value
- remove unneeded local variable from board_eth_init.
Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
---
board/isee/igep00x0/igep00x0.c | 33 +++++++++++++++++++--------------
1 file changed, 19 insertions(+), 14 deletions(-)
diff --git a/board/isee/igep00x0/igep00x0.c b/board/isee/igep00x0/igep00x0.c
index 57b89e0..e2fce50 100644
--- a/board/isee/igep00x0/igep00x0.c
+++ b/board/isee/igep00x0/igep00x0.c
@@ -101,6 +101,19 @@ void get_board_mem_timings(struct board_sdrc_timings *timings)
#endif
#if defined(CONFIG_CMD_NET)
+
+static void reset_net_chip(int gpio)
+{
+ if (!gpio_request(gpio, "eth nrst")) {
+ gpio_direction_output(gpio, 1);
+ udelay(1);
+ gpio_set_value(gpio, 0);
+ udelay(40);
+ gpio_set_value(gpio, 1);
+ mdelay(10);
+ }
+}
+
/*
* Routine: setup_net_chip
* Description: Setting up the configuration GPMC registers specific to the
@@ -110,8 +123,8 @@ static void setup_net_chip(void)
{
struct ctrl *ctrl_base = (struct ctrl *)OMAP34XX_CTRL_BASE;
- enable_gpmc_cs_config(gpmc_lan_config, &gpmc_cfg->cs[5], 0x2C000000,
- GPMC_SIZE_16M);
+ enable_gpmc_cs_config(gpmc_lan_config, &gpmc_cfg->cs[5],
+ CONFIG_SMC911X_BASE, GPMC_SIZE_16M);
/* Enable off mode for NWE in PADCONF_GPMC_NWE register */
writew(readw(&ctrl_base->gpmc_nwe) | 0x0E00, &ctrl_base->gpmc_nwe);
@@ -121,15 +134,7 @@ static void setup_net_chip(void)
writew(readw(&ctrl_base->gpmc_nadv_ale) | 0x0E00,
&ctrl_base->gpmc_nadv_ale);
- /* Make GPIO 64 as output pin and send a magic pulse through it */
- if (!gpio_request(64, "")) {
- gpio_direction_output(64, 0);
- gpio_set_value(64, 1);
- udelay(1);
- gpio_set_value(64, 0);
- udelay(1);
- gpio_set_value(64, 1);
- }
+ reset_net_chip(64);
}
#else
static inline void setup_net_chip(void) {}
@@ -200,10 +205,10 @@ void set_muxconf_regs(void)
#if defined(CONFIG_CMD_NET)
int board_eth_init(bd_t *bis)
{
- int rc = 0;
#ifdef CONFIG_SMC911X
- rc = smc911x_initialize(0, CONFIG_SMC911X_BASE);
+ return smc911x_initialize(0, CONFIG_SMC911X_BASE);
+#else
+ return 0;
#endif
- return rc;
}
#endif
--
2.1.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h
2016-01-04 22:07 [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Enric Balletbo i Serra
2016-01-04 22:07 ` [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR Enric Balletbo i Serra
2016-01-04 22:07 ` [U-Boot] [PATCHv2 2/5] igep00x0: Cleanup ethernet support Enric Balletbo i Serra
@ 2016-01-04 22:08 ` Enric Balletbo i Serra
2016-01-04 22:23 ` Tom Rini
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:08 ` [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation Enric Balletbo i Serra
` (2 subsequent siblings)
5 siblings, 2 replies; 22+ messages in thread
From: Enric Balletbo i Serra @ 2016-01-04 22:08 UTC (permalink / raw)
To: u-boot
From: Ladislav Michl <ladis@linux-mips.org>
File is already included:
omap3_igep00x0.h -> ti_omap3_common.h -> ti_armv7_omap.h ->
ti_armv7_common.h -> config_distro_defaults.h
Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
---
include/configs/omap3_igep00x0.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/include/configs/omap3_igep00x0.h b/include/configs/omap3_igep00x0.h
index cf2bc3e..16775e2 100644
--- a/include/configs/omap3_igep00x0.h
+++ b/include/configs/omap3_igep00x0.h
@@ -85,8 +85,6 @@
#ifndef CONFIG_SPL_BUILD
-#include <config_distro_defaults.h>
-
/* Environment */
#define ENV_DEVICE_SETTINGS \
"stdin=serial\0" \
--
2.1.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation
2016-01-04 22:07 [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Enric Balletbo i Serra
` (2 preceding siblings ...)
2016-01-04 22:08 ` [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h Enric Balletbo i Serra
@ 2016-01-04 22:08 ` Enric Balletbo i Serra
2016-01-04 22:24 ` Tom Rini
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:08 ` [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header Enric Balletbo i Serra
2016-01-04 22:24 ` [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Tom Rini
5 siblings, 2 replies; 22+ messages in thread
From: Enric Balletbo i Serra @ 2016-01-04 22:08 UTC (permalink / raw)
To: u-boot
From: Ladislav Michl <ladis@linux-mips.org>
The patch fixes some indentation style problems in omap3_igep00x0.h file.
Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
---
include/configs/omap3_igep00x0.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/include/configs/omap3_igep00x0.h b/include/configs/omap3_igep00x0.h
index 16775e2..bd1c038 100644
--- a/include/configs/omap3_igep00x0.h
+++ b/include/configs/omap3_igep00x0.h
@@ -43,7 +43,7 @@
#else
#error "status LED not defined for this machine."
#endif
-#define RED_LED_DEV 0
+#define RED_LED_DEV 0
#define STATUS_LED_BIT RED_LED_GPIO
#define STATUS_LED_STATE STATUS_LED_ON
#define STATUS_LED_PERIOD (CONFIG_SYS_HZ / 2)
@@ -56,7 +56,7 @@
#define CONFIG_OMAP3_GPIO_6 /* GPIO160..191 is in GPIO bank 6 */
/* USB */
-#define CONFIG_USB_MUSB_UDC 1
+#define CONFIG_USB_MUSB_UDC 1
#define CONFIG_USB_OMAP3 1
#define CONFIG_TWL4030_USB 1
@@ -136,7 +136,7 @@
#if defined(CONFIG_CMD_NET)
#define CONFIG_SMC911X
#define CONFIG_SMC911X_32_BIT
-#define CONFIG_SMC911X_BASE 0x2C000000
+#define CONFIG_SMC911X_BASE 0x2C000000
#endif /* (CONFIG_CMD_NET) */
/* OneNAND boot config */
--
2.1.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header
2016-01-04 22:07 [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Enric Balletbo i Serra
` (3 preceding siblings ...)
2016-01-04 22:08 ` [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation Enric Balletbo i Serra
@ 2016-01-04 22:08 ` Enric Balletbo i Serra
2016-01-04 22:24 ` Tom Rini
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:24 ` [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Tom Rini
5 siblings, 2 replies; 22+ messages in thread
From: Enric Balletbo i Serra @ 2016-01-04 22:08 UTC (permalink / raw)
To: u-boot
From: Ladislav Michl <ladis@linux-mips.org>
The patch removes some macros that are not used.
Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
---
include/configs/omap3_igep00x0.h | 5 -----
1 file changed, 5 deletions(-)
diff --git a/include/configs/omap3_igep00x0.h b/include/configs/omap3_igep00x0.h
index bd1c038..5da6cfd 100644
--- a/include/configs/omap3_igep00x0.h
+++ b/include/configs/omap3_igep00x0.h
@@ -75,14 +75,9 @@
#ifdef CONFIG_BOOT_ONENAND
#define CONFIG_CMD_ONENAND /* ONENAND support */
#endif
-#if (CONFIG_MACH_TYPE == MACH_TYPE_IGEP0020) || \
- (CONFIG_MACH_TYPE == MACH_TYPE_IGEP0032)
-#endif
#define CONFIG_CMD_DHCP
#define CONFIG_CMD_PING
-/*#undef CONFIG_ENV_IS_NOWHERE*/
-
#ifndef CONFIG_SPL_BUILD
/* Environment */
--
2.1.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR
2016-01-04 22:07 ` [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR Enric Balletbo i Serra
@ 2016-01-04 22:23 ` Tom Rini
2016-01-20 5:19 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
1 sibling, 1 reply; 22+ messages in thread
From: Tom Rini @ 2016-01-04 22:23 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:07:58PM +0100, Enric Balletbo i Serra wrote:
> Enable CONFIG_NET_RANDOM_ETHADDR to generate a random MAC address
> when ETHADDR is not set.
>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
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/20160104/a71aba86/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h
2016-01-04 22:08 ` [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h Enric Balletbo i Serra
@ 2016-01-04 22:23 ` Tom Rini
2016-01-20 5:19 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
1 sibling, 1 reply; 22+ messages in thread
From: Tom Rini @ 2016-01-04 22:23 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:08:00PM +0100, Enric Balletbo i Serra wrote:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> File is already included:
> omap3_igep00x0.h -> ti_omap3_common.h -> ti_armv7_omap.h ->
> ti_armv7_common.h -> config_distro_defaults.h
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
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/20160104/26857422/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation
2016-01-04 22:08 ` [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation Enric Balletbo i Serra
@ 2016-01-04 22:24 ` Tom Rini
2016-01-20 5:20 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
1 sibling, 1 reply; 22+ messages in thread
From: Tom Rini @ 2016-01-04 22:24 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:08:01PM +0100, Enric Balletbo i Serra wrote:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> The patch fixes some indentation style problems in omap3_igep00x0.h file.
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
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/20160104/03aa73e2/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header
2016-01-04 22:08 ` [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header Enric Balletbo i Serra
@ 2016-01-04 22:24 ` Tom Rini
2016-01-20 5:20 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
1 sibling, 1 reply; 22+ messages in thread
From: Tom Rini @ 2016-01-04 22:24 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:08:02PM +0100, Enric Balletbo i Serra wrote:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> The patch removes some macros that are not used.
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
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/20160104/33f6e215/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup
2016-01-04 22:07 [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Enric Balletbo i Serra
` (4 preceding siblings ...)
2016-01-04 22:08 ` [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header Enric Balletbo i Serra
@ 2016-01-04 22:24 ` Tom Rini
5 siblings, 0 replies; 22+ messages in thread
From: Tom Rini @ 2016-01-04 22:24 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:07:57PM +0100, Enric Balletbo i Serra wrote:
> Hi Tom,
>
> The following patches are minor cleanups that I think are safe to apply. I
> know that we're very close to next release so it's up to you apply now or
> wait for the next merge window. I'd like to see it merged but I don't any
> objection if you decide wait.
I'll pull them in after the release then, 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/20160104/d9a205ec/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR
2016-01-04 22:23 ` Tom Rini
@ 2016-01-20 5:19 ` Heiko Schocher
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Schocher @ 2016-01-20 5:19 UTC (permalink / raw)
To: u-boot
Hello Tom,
Am 04.01.2016 um 23:23 schrieb Tom Rini:
> On Mon, Jan 04, 2016 at 11:07:58PM +0100, Enric Balletbo i Serra wrote:
>
>> Enable CONFIG_NET_RANDOM_ETHADDR to generate a random MAC address
>> when ETHADDR is not set.
>>
>> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
Tom? Is it OK if I apply this patch to u-boot-ubi.git, as the SPL ubi
patches depend on this series?
bye,
Heiko
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h
2016-01-04 22:23 ` Tom Rini
@ 2016-01-20 5:19 ` Heiko Schocher
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Schocher @ 2016-01-20 5:19 UTC (permalink / raw)
To: u-boot
Hello Tom,
Am 04.01.2016 um 23:23 schrieb Tom Rini:
> On Mon, Jan 04, 2016 at 11:08:00PM +0100, Enric Balletbo i Serra wrote:
>
>> From: Ladislav Michl <ladis@linux-mips.org>
>>
>> File is already included:
>> omap3_igep00x0.h -> ti_omap3_common.h -> ti_armv7_omap.h ->
>> ti_armv7_common.h -> config_distro_defaults.h
>>
>> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
>> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
Tom? Is it OK if I apply this patch to u-boot-ubi.git, as the SPL ubi
patches depend on this series?
bye,
Heiko
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation
2016-01-04 22:24 ` Tom Rini
@ 2016-01-20 5:20 ` Heiko Schocher
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Schocher @ 2016-01-20 5:20 UTC (permalink / raw)
To: u-boot
Hello Tom,
Am 04.01.2016 um 23:24 schrieb Tom Rini:
> On Mon, Jan 04, 2016 at 11:08:01PM +0100, Enric Balletbo i Serra wrote:
>
>> From: Ladislav Michl <ladis@linux-mips.org>
>>
>> The patch fixes some indentation style problems in omap3_igep00x0.h file.
>>
>> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
>> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
Tom? Is it OK if I apply this patch to u-boot-ubi.git, as the SPL ubi
patches depend on this series?
bye,
Heiko
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header
2016-01-04 22:24 ` Tom Rini
@ 2016-01-20 5:20 ` Heiko Schocher
0 siblings, 0 replies; 22+ messages in thread
From: Heiko Schocher @ 2016-01-20 5:20 UTC (permalink / raw)
To: u-boot
Hello Tom,
Am 04.01.2016 um 23:24 schrieb Tom Rini:
> On Mon, Jan 04, 2016 at 11:08:02PM +0100, Enric Balletbo i Serra wrote:
>
>> From: Ladislav Michl <ladis@linux-mips.org>
>>
>> The patch removes some macros that are not used.
>>
>> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
>> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
Tom? Is it OK if I apply this patch to u-boot-ubi.git, as the SPL ubi
patches depend on this series?
bye,
Heiko
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [U-Boot, PATCHv2, 2/5] igep00x0: Cleanup ethernet support
2016-01-04 22:07 ` [U-Boot] [PATCHv2 2/5] igep00x0: Cleanup ethernet support Enric Balletbo i Serra
@ 2016-01-20 5:24 ` Heiko Schocher
2016-01-20 15:02 ` Tom Rini
2016-01-20 21:01 ` Tom Rini
1 sibling, 1 reply; 22+ messages in thread
From: Heiko Schocher @ 2016-01-20 5:24 UTC (permalink / raw)
To: u-boot
Hello Tom,
Am 04.01.2016 um 23:07 schrieb Enric Balletb? i Serra:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> - move chip reset to separate function
> - use CONFIG_SMC911X_BASE instead of hardcoded value
> - remove unneeded local variable from board_eth_init.
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
> ---
> board/isee/igep00x0/igep00x0.c | 33 +++++++++++++++++++--------------
> 1 file changed, 19 insertions(+), 14 deletions(-)
Reviewed-by: Heiko Schocher <hs@denx.de>
Tom? Is it OK if I apply this patch to u-boot-ubi.git, as the SPL ubi
patches depend on this series?
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [U-Boot, PATCHv2, 2/5] igep00x0: Cleanup ethernet support
2016-01-20 5:24 ` [U-Boot] [U-Boot, PATCHv2, " Heiko Schocher
@ 2016-01-20 15:02 ` Tom Rini
0 siblings, 0 replies; 22+ messages in thread
From: Tom Rini @ 2016-01-20 15:02 UTC (permalink / raw)
To: u-boot
On Wed, Jan 20, 2016 at 06:24:40AM +0100, Heiko Schocher wrote:
> Hello Tom,
>
> Am 04.01.2016 um 23:07 schrieb Enric Balletb? i Serra:
> >From: Ladislav Michl <ladis@linux-mips.org>
> >
> >- move chip reset to separate function
> >- use CONFIG_SMC911X_BASE instead of hardcoded value
> >- remove unneeded local variable from board_eth_init.
> >
> >Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> >Reviewed-by: Tom Rini <trini@konsulko.com>
> >Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
> >---
> > board/isee/igep00x0/igep00x0.c | 33 +++++++++++++++++++--------------
> > 1 file changed, 19 insertions(+), 14 deletions(-)
>
> Reviewed-by: Heiko Schocher <hs@denx.de>
>
> Tom? Is it OK if I apply this patch to u-boot-ubi.git, as the SPL ubi
> patches depend on this series?
I'll pick up this series (and some other TI platform stuff) in the next
day or two, 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/20160120/2eb133bc/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [U-Boot, PATCHv2, 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR
2016-01-04 22:07 ` [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR Enric Balletbo i Serra
2016-01-04 22:23 ` Tom Rini
@ 2016-01-20 21:01 ` Tom Rini
1 sibling, 0 replies; 22+ messages in thread
From: Tom Rini @ 2016-01-20 21:01 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:07:58PM +0100, Enric Balletb? i Serra wrote:
> Enable CONFIG_NET_RANDOM_ETHADDR to generate a random MAC address
> when ETHADDR is not set.
>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Heiko Schocher <hs@denx.de>
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/20160120/ec97140d/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [U-Boot, PATCHv2, 2/5] igep00x0: Cleanup ethernet support
2016-01-04 22:07 ` [U-Boot] [PATCHv2 2/5] igep00x0: Cleanup ethernet support Enric Balletbo i Serra
2016-01-20 5:24 ` [U-Boot] [U-Boot, PATCHv2, " Heiko Schocher
@ 2016-01-20 21:01 ` Tom Rini
1 sibling, 0 replies; 22+ messages in thread
From: Tom Rini @ 2016-01-20 21:01 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:07:59PM +0100, Enric Balletb? i Serra wrote:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> - move chip reset to separate function
> - use CONFIG_SMC911X_BASE instead of hardcoded value
> - remove unneeded local variable from board_eth_init.
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
> Reviewed-by: Heiko Schocher <hs@denx.de>
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/20160120/8724ece2/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [U-Boot, PATCHv2, 3/5] igep00x0: Do not include config_distro_defaults.h
2016-01-04 22:08 ` [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h Enric Balletbo i Serra
2016-01-04 22:23 ` Tom Rini
@ 2016-01-20 21:01 ` Tom Rini
1 sibling, 0 replies; 22+ messages in thread
From: Tom Rini @ 2016-01-20 21:01 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:08:00PM +0100, Enric Balletb? i Serra wrote:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> File is already included:
> omap3_igep00x0.h -> ti_omap3_common.h -> ti_armv7_omap.h ->
> ti_armv7_common.h -> config_distro_defaults.h
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Heiko Schocher <hs@denx.de>
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/20160120/d089928e/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [U-Boot, PATCHv2, 4/5] igep00x0: Fix config header indentation
2016-01-04 22:08 ` [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation Enric Balletbo i Serra
2016-01-04 22:24 ` Tom Rini
@ 2016-01-20 21:01 ` Tom Rini
1 sibling, 0 replies; 22+ messages in thread
From: Tom Rini @ 2016-01-20 21:01 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:08:01PM +0100, Enric Balletb? i Serra wrote:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> The patch fixes some indentation style problems in omap3_igep00x0.h file.
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Heiko Schocher <hs@denx.de>
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/20160120/cf7ca3ad/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [U-Boot] [U-Boot, PATCHv2, 5/5] igep00x0: Remove no-op macros from config header
2016-01-04 22:08 ` [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header Enric Balletbo i Serra
2016-01-04 22:24 ` Tom Rini
@ 2016-01-20 21:01 ` Tom Rini
1 sibling, 0 replies; 22+ messages in thread
From: Tom Rini @ 2016-01-20 21:01 UTC (permalink / raw)
To: u-boot
On Mon, Jan 04, 2016 at 11:08:02PM +0100, Enric Balletb? i Serra wrote:
> From: Ladislav Michl <ladis@linux-mips.org>
>
> The patch removes some macros that are not used.
>
> Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> Acked-by: Enric Balletbo Serra <enric.balletbo@collabora.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Heiko Schocher <hs@denx.de>
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/20160120/d05f5444/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2016-01-20 21:01 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-04 22:07 [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Enric Balletbo i Serra
2016-01-04 22:07 ` [U-Boot] [PATCHv2 1/5] igep00x0: enable CONFIG_NET_RANDOM_ETHADDR Enric Balletbo i Serra
2016-01-04 22:23 ` Tom Rini
2016-01-20 5:19 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:07 ` [U-Boot] [PATCHv2 2/5] igep00x0: Cleanup ethernet support Enric Balletbo i Serra
2016-01-20 5:24 ` [U-Boot] [U-Boot, PATCHv2, " Heiko Schocher
2016-01-20 15:02 ` Tom Rini
2016-01-20 21:01 ` Tom Rini
2016-01-04 22:08 ` [U-Boot] [PATCHv2 3/5] igep00x0: Do not include config_distro_defaults.h Enric Balletbo i Serra
2016-01-04 22:23 ` Tom Rini
2016-01-20 5:19 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:08 ` [U-Boot] [PATCHv2 4/5] igep00x0: Fix config header indentation Enric Balletbo i Serra
2016-01-04 22:24 ` Tom Rini
2016-01-20 5:20 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:08 ` [U-Boot] [PATCHv2 5/5] igep00x0: Remove no-op macros from config header Enric Balletbo i Serra
2016-01-04 22:24 ` Tom Rini
2016-01-20 5:20 ` Heiko Schocher
2016-01-20 21:01 ` [U-Boot] [U-Boot, PATCHv2, " Tom Rini
2016-01-04 22:24 ` [U-Boot] [PATCHv2 0/5] igep00x0: Minor cleanup Tom Rini
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.