* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox