I see this patch was already submitted by Johannes Berg just yesterday
(message id: 1247671274.10754.5.camel () johannes ! local)
Sorry.
On 07/16/2009 11:14 AM, Tobias Klauser wrote:
> Use the %pM conversion specifier for printk instead of print_mac.
>
> Signed-off-by: Tobias Klauser <klto@xxxxxxx>
> ---
> drivers/scsi/fcoe/libfcoe.c | 6 ++----
> 1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/fcoe/libfcoe.c b/drivers/scsi/fcoe/libfcoe.c
> index f544340..d6ed3f8 100644
> --- a/drivers/scsi/fcoe/libfcoe.c
> +++ b/drivers/scsi/fcoe/libfcoe.c
> @@ -1104,7 +1104,6 @@ static void fcoe_ctlr_timeout(unsigned long arg)
> struct fcoe_fcf *sel;
> struct fcoe_fcf *fcf;
> unsigned long next_timer = jiffies + msecs_to_jiffies(FIP_VN_KA_PERIOD);
> - DECLARE_MAC_BUF(buf);
> u8 send_ctlr_ka;
> u8 send_port_ka;
>
> @@ -1128,9 +1127,8 @@ static void fcoe_ctlr_timeout(unsigned long arg)
> fcf = sel; /* the old FCF may have been freed */
> if (sel) {
> printk(KERN_INFO "libfcoe: host%d: FIP selected "
> - "Fibre-Channel Forwarder MAC %s\n",
> - fip->lp->host->host_no,
> - print_mac(buf, sel->fcf_mac));
> + "Fibre-Channel Forwarder MAC %pM\n",
> + fip->lp->host->host_no, sel->fcf_mac);
> memcpy(fip->dest_addr, sel->fcf_mac, ETH_ALEN);
> fip->port_ka_time = jiffies +
> msecs_to_jiffies(FIP_VN_KA_PERIOD);
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
