linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: orion5x: Use vsprintf %pM extension
@ 2015-06-15  2:01 Joe Perches
  2015-06-15  3:28 ` Andrew Lunn
  0 siblings, 1 reply; 3+ messages in thread
From: Joe Perches @ 2015-06-15  2:01 UTC (permalink / raw)
  To: linux-arm-kernel

Format mac addresses with the normal kernel extension.

Signed-off-by: Joe Perches <joe@perches.com>
---
 arch/arm/mach-orion5x/dns323-setup.c | 4 +---
 arch/arm/mach-orion5x/tsx09-common.c | 4 +---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/arch/arm/mach-orion5x/dns323-setup.c b/arch/arm/mach-orion5x/dns323-setup.c
index 09d2a26..f267e58 100644
--- a/arch/arm/mach-orion5x/dns323-setup.c
+++ b/arch/arm/mach-orion5x/dns323-setup.c
@@ -236,9 +236,7 @@ static int __init dns323_read_mac_addr(void)
 	}
 
 	iounmap(mac_page);
-	printk("DNS-323: Found ethernet MAC address: ");
-	for (i = 0; i < 6; i++)
-		printk("%.2x%s", addr[i], (i < 5) ? ":" : ".\n");
+	printk("DNS-323: Found ethernet MAC address: %pM\n", addr);
 
 	memcpy(dns323_eth_data.mac_addr, addr, 6);
 
diff --git a/arch/arm/mach-orion5x/tsx09-common.c b/arch/arm/mach-orion5x/tsx09-common.c
index 7189827..24b2959 100644
--- a/arch/arm/mach-orion5x/tsx09-common.c
+++ b/arch/arm/mach-orion5x/tsx09-common.c
@@ -101,9 +101,7 @@ static int __init qnap_tsx09_check_mac_addr(const char *addr_str)
 		addr[i] = byte;
 	}
 
-	printk(KERN_INFO "tsx09: found ethernet mac address ");
-	for (i = 0; i < 6; i++)
-		printk("%.2x%s", addr[i], (i < 5) ? ":" : ".\n");
+	printk(KERN_INFO "tsx09: found ethernet mac address %pM\n", addr);
 
 	memcpy(qnap_tsx09_eth_data.mac_addr, addr, 6);
 

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

* [PATCH] ARM: orion5x: Use vsprintf %pM extension
  2015-06-15  2:01 [PATCH] ARM: orion5x: Use vsprintf %pM extension Joe Perches
@ 2015-06-15  3:28 ` Andrew Lunn
  2015-06-17 14:53   ` Gregory CLEMENT
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Lunn @ 2015-06-15  3:28 UTC (permalink / raw)
  To: linux-arm-kernel

On Sun, Jun 14, 2015 at 07:01:30PM -0700, Joe Perches wrote:
> Format mac addresses with the normal kernel extension.
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Acked-by: Andrew Lunn <andrew@lunn.ch>

	  Andrew

> ---
>  arch/arm/mach-orion5x/dns323-setup.c | 4 +---
>  arch/arm/mach-orion5x/tsx09-common.c | 4 +---
>  2 files changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/mach-orion5x/dns323-setup.c b/arch/arm/mach-orion5x/dns323-setup.c
> index 09d2a26..f267e58 100644
> --- a/arch/arm/mach-orion5x/dns323-setup.c
> +++ b/arch/arm/mach-orion5x/dns323-setup.c
> @@ -236,9 +236,7 @@ static int __init dns323_read_mac_addr(void)
>  	}
>  
>  	iounmap(mac_page);
> -	printk("DNS-323: Found ethernet MAC address: ");
> -	for (i = 0; i < 6; i++)
> -		printk("%.2x%s", addr[i], (i < 5) ? ":" : ".\n");
> +	printk("DNS-323: Found ethernet MAC address: %pM\n", addr);
>  
>  	memcpy(dns323_eth_data.mac_addr, addr, 6);
>  
> diff --git a/arch/arm/mach-orion5x/tsx09-common.c b/arch/arm/mach-orion5x/tsx09-common.c
> index 7189827..24b2959 100644
> --- a/arch/arm/mach-orion5x/tsx09-common.c
> +++ b/arch/arm/mach-orion5x/tsx09-common.c
> @@ -101,9 +101,7 @@ static int __init qnap_tsx09_check_mac_addr(const char *addr_str)
>  		addr[i] = byte;
>  	}
>  
> -	printk(KERN_INFO "tsx09: found ethernet mac address ");
> -	for (i = 0; i < 6; i++)
> -		printk("%.2x%s", addr[i], (i < 5) ? ":" : ".\n");
> +	printk(KERN_INFO "tsx09: found ethernet mac address %pM\n", addr);
>  
>  	memcpy(qnap_tsx09_eth_data.mac_addr, addr, 6);
>  
> 
> 

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

* [PATCH] ARM: orion5x: Use vsprintf %pM extension
  2015-06-15  3:28 ` Andrew Lunn
@ 2015-06-17 14:53   ` Gregory CLEMENT
  0 siblings, 0 replies; 3+ messages in thread
From: Gregory CLEMENT @ 2015-06-17 14:53 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Andrew, Joe,

On 15/06/2015 05:28, Andrew Lunn wrote:
> On Sun, Jun 14, 2015 at 07:01:30PM -0700, Joe Perches wrote:
>> Format mac addresses with the normal kernel extension.
>>
>> Signed-off-by: Joe Perches <joe@perches.com>
> 
> Acked-by: Andrew Lunn <andrew@lunn.ch>

applied on mvebu/cleanup-4.3

Thanks,

Gregory
> 
> 	  Andrew
> 
>> ---
>>  arch/arm/mach-orion5x/dns323-setup.c | 4 +---
>>  arch/arm/mach-orion5x/tsx09-common.c | 4 +---
>>  2 files changed, 2 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/arm/mach-orion5x/dns323-setup.c b/arch/arm/mach-orion5x/dns323-setup.c
>> index 09d2a26..f267e58 100644
>> --- a/arch/arm/mach-orion5x/dns323-setup.c
>> +++ b/arch/arm/mach-orion5x/dns323-setup.c
>> @@ -236,9 +236,7 @@ static int __init dns323_read_mac_addr(void)
>>  	}
>>  
>>  	iounmap(mac_page);
>> -	printk("DNS-323: Found ethernet MAC address: ");
>> -	for (i = 0; i < 6; i++)
>> -		printk("%.2x%s", addr[i], (i < 5) ? ":" : ".\n");
>> +	printk("DNS-323: Found ethernet MAC address: %pM\n", addr);
>>  
>>  	memcpy(dns323_eth_data.mac_addr, addr, 6);
>>  
>> diff --git a/arch/arm/mach-orion5x/tsx09-common.c b/arch/arm/mach-orion5x/tsx09-common.c
>> index 7189827..24b2959 100644
>> --- a/arch/arm/mach-orion5x/tsx09-common.c
>> +++ b/arch/arm/mach-orion5x/tsx09-common.c
>> @@ -101,9 +101,7 @@ static int __init qnap_tsx09_check_mac_addr(const char *addr_str)
>>  		addr[i] = byte;
>>  	}
>>  
>> -	printk(KERN_INFO "tsx09: found ethernet mac address ");
>> -	for (i = 0; i < 6; i++)
>> -		printk("%.2x%s", addr[i], (i < 5) ? ":" : ".\n");
>> +	printk(KERN_INFO "tsx09: found ethernet mac address %pM\n", addr);
>>  
>>  	memcpy(qnap_tsx09_eth_data.mac_addr, addr, 6);
>>  
>>
>>


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

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

end of thread, other threads:[~2015-06-17 14:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-15  2:01 [PATCH] ARM: orion5x: Use vsprintf %pM extension Joe Perches
2015-06-15  3:28 ` Andrew Lunn
2015-06-17 14:53   ` Gregory CLEMENT

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).