RE: [PATCH] drivers/net/defxx.c: use %pMF to show MAC address

From: Joe Perches
Date: Fri Jan 08 2010 - 12:54:21 EST


On Fri, 2010-01-08 at 13:34 +0000, Maciej W. Rozycki wrote:
> On Thu, 7 Jan 2010, H Hartley Sweeten wrote:
> > >> 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?
> > > With the two patches in place the output produced is now broken. :(

> Here are excerpts from old bootstrap logs -- I don't have any of these
> boards handy at the moment, but I'll have access to a couple in a few
> weeks' time:
>
> defxx: v1.10 2006/12/14 Lawrence V. Stefani and others
> tc0: DEFTA at addr = 0x10100000, IRQ = 2, Hardware addr = 08-00-2B-A3-66-C8
> tc0: registered as fddi0
> tc1: DEFTA at addr = 0x14100000, IRQ = 3, Hardware addr = 08-00-2B-A3-A3-29
> tc1: registered as fddi1
>
> defxx: v1.10 2006/12/14 Lawrence V. Stefani and others
> 0000:00:06.0: DEFPA at addr = 0x41071000, IRQ = 57, Hardware addr = 00-60-B0-58-40-75
> 0000:00:06.0: registered as fddi0
> 0000:02:00.0: DEFPA at addr = 0x41510000, IRQ = 57, Hardware addr = 00-60-B0-58-41-E7
> 0000:02:00.0: registered as fddi1
>
> Compare the addresses reported with the OUIs of the respective
> manufacturers -- the former is DEC and the latter is HP. See
> http://standards.ieee.org/regauth/oui/oui.txt for a reference.

Then isn't the hardware address shown as ethernet and not as
in the FDDI canonical format?

Which confuses me relative to what you wrote earlier.

> On Mon, 2010-01-04 at 23:43 +0000, Maciej W. Rozycki wrote:
> > The example below shows an address, and the sequence of bits or symbols
> > that would be transmitted when the address is used in the Source Address
> > or Destination Address fields on the MAC header. The transmission line
> > shows the address bits in the order transmitted, from left to right. For
> > IEEE 802 LANs these correspond to actual bits on the medium. The FDDI
> > symbols line shows how the FDDI PHY sends the address bits as encoded
> > symbols.
> >
> > MSB: 35:7B:12:00:00:01
> > Canonical: AC-DE-48-00-00-80

So, it should simply use %pM correct?

Is there any address that should be shown in the
FDDI canonical form?

cheers, Joe

--
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/