Re: [PATCH 6/6] staging: ccg: print MAC addresses via %pM

From: Andy Shevchenko
Date: Fri Jul 06 2012 - 11:57:33 EST


On Fri, Jul 6, 2012 at 6:54 PM, Daniel Hazelton <dshadowwolf@xxxxxxxxx> wrote:
>>> --- a/drivers/staging/ccg/ccg.c
>>> +++ b/drivers/staging/ccg/ccg.c
>>> @@ -564,9 +564,7 @@ static int rndis_function_bind_config(struct ccg_usb_function *f,
>>> Â Â Â Â Â Â Â Â return -1;
>>> Â Â Â Â }
>>>
>>> - Â Â Â pr_info("%s MAC: %02X:%02X:%02X:%02X:%02X:%02X\n", __func__,
>>> - Â Â Â Â Â Â Â rndis->ethaddr[0], rndis->ethaddr[1], rndis->ethaddr[2],
>>> - Â Â Â Â Â Â Â rndis->ethaddr[3], rndis->ethaddr[4], rndis->ethaddr[5]);
>>> + Â Â Â pr_info("%s MAC: pM\n", __func__, rndis->ethaddr);
> You lost a % there - it should be "%s MAC: %pM\n", no ?
Thanks, I noticed this already. Patch v2 was sent recently.

--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/