From: "Maciej W. Rozycki" <macro@linux-mips.org>
To: Joe Perches <joe@perches.com>
Cc: H Hartley Sweeten <hartleys@visionengravers.com>,
Linux Kernel <linux-kernel@vger.kernel.org>,
netdev@vger.kernel.org, davem@davemloft.net
Subject: Re: [PATCH] drivers/net/defxx.c: use %pMF to show MAC address
Date: Fri, 8 Jan 2010 02:06:14 +0000 (GMT) [thread overview]
Message-ID: <alpine.LFD.2.00.1001080143250.13474@eddie.linux-mips.org> (raw)
In-Reply-To: <1262914750.10429.88.camel@Joe-Laptop.home>
On Thu, 7 Jan 2010, Joe Perches wrote:
> On Thu, 2010-01-07 at 16:24 -0700, H Hartley Sweeten wrote:
> > Use the %pMF kernel extension to display the MAC address.
> > The address will still be displayed in the FDDI Canonical format.
>
> > diff --git a/drivers/net/defxx.c b/drivers/net/defxx.c
> > index 6a6ea03..e4eac4b 100644
> > --- a/drivers/net/defxx.c
> > +++ b/drivers/net/defxx.c
> > @@ -1052,12 +1052,9 @@ static int __devinit dfx_driver_init(struct net_device *dev,
> > board_name = "DEFEA";
> > if (dfx_bus_pci)
> > board_name = "DEFPA";
> > - pr_info("%s: %s at %saddr = 0x%llx, IRQ = %d, "
> > - "Hardware addr = %02X-%02X-%02X-%02X-%02X-%02X\n",
> > + pr_info("%s: %s at %saddr = 0x%llx, IRQ = %d, Hardware addr = %pMF\n",
> > print_name, board_name, dfx_use_mmio ? "" : "I/O ",
>
> I believe the output will _now_ be in the canonical form.
> Before it wasn't bit reversed. Now it should be.
Output was correct -- why would you assume otherwise? The bit order in
the MLA (My Long Address) register as retrieved from the MAC chip by
board's firmware has nothing to do with the wire order and is already in
the canonical form (the same applies to CAM entries and frame headers) --
the board designers were sane people.
With the two patches in place the output produced is now broken. :(
Maciej
next prev parent reply other threads:[~2010-01-08 2:06 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-07 23:24 [PATCH] drivers/net/defxx.c: use %pMF to show MAC address H Hartley Sweeten
2010-01-08 0:58 ` David Miller
2010-01-08 1:39 ` Joe Perches
2010-01-08 2:06 ` Maciej W. Rozycki [this message]
2010-01-08 2:11 ` H Hartley Sweeten
2010-01-08 7:47 ` David Miller
2010-01-08 13:34 ` Maciej W. Rozycki
2010-01-08 17:54 ` Joe Perches
2010-01-08 18:58 ` Maciej W. Rozycki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.LFD.2.00.1001080143250.13474@eddie.linux-mips.org \
--to=macro@linux-mips.org \
--cc=davem@davemloft.net \
--cc=hartleys@visionengravers.com \
--cc=joe@perches.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).