diff mbox

[U-Boot,03/14] net: cosmetic: Define ethernet name length

Message ID 20161125153032.14617-4-oliver@schinagl.nl
State Accepted
Commit 2c07c3299467e2313d6764ed28db62e8fcc77ccf
Delegated to: Joe Hershberger
Headers show

Commit Message

Olliver Schinagl Nov. 25, 2016, 3:30 p.m. UTC 
There are various places where the ethernet device name is defined to
several different sizes. Lets add a define and start using it.

Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
---
 include/net.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Joe Hershberger Nov. 30, 2016, 7:15 p.m. UTC | #1 
On Fri, Nov 25, 2016 at 9:30 AM, Olliver Schinagl <oliver@schinagl.nl> wrote:
> There are various places where the ethernet device name is defined to
> several different sizes. Lets add a define and start using it.
>
> Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>

Acked-by: Joe Hershberger <joe.hershberger@ni.com>
Joe Hershberger Feb. 9, 2017, 4:26 p.m. UTC | #2 
Hi oliver@schinagl.nl,

https://patchwork.ozlabs.org/patch/699278/ was applied to u-boot-net.git.

Thanks!
-Joe
diff mbox

Patch

diff --git a/include/net.h b/include/net.h
index af0558d..9cd7870 100644
--- a/include/net.h
+++ b/include/net.h
@@ -168,7 +168,8 @@  void eth_halt_state_only(void); /* Set passive state */
 
 #ifndef CONFIG_DM_ETH
 struct eth_device {
-	char name[16];
+#define ETH_NAME_LEN 16
+	char name[ETH_NAME_LEN];
 	unsigned char enetaddr[ARP_HLEN];
 	phys_addr_t iobase;
 	int state;