netdev - Re: [PATCH] [POWERPC] mpc5200: Allow for fixed speed MII configurations

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 30 Apr 2008 08:28:18 -0600
From:	"Grant Likely" <grant.likely@...retlab.ca>
To:	joakim.tjernlund@...nsmode.se
Cc:	wg@...ndegger.com, linuxppc-dev@...abs.org, paulus@...ba.org,
	netdev@...r.kernel.org, "Wolfgang Denk" <wd@...x.de>
Subject: Re: [PATCH] [POWERPC] mpc5200: Allow for fixed speed MII configurations

On Wed, Apr 30, 2008 at 8:16 AM, Joakim Tjernlund
<joakim.tjernlund@...nsmode.se> wrote:
>
>
>  On Wed, 2008-04-30 at 07:26 -0600, Grant Likely wrote:
>  > On Wed, Apr 30, 2008 at 7:10 AM, Joakim Tjernlund
>  > <joakim.tjernlund@...nsmode.se> wrote:
>  > >
>  > >
>  > >  On Wed, 2008-04-30 at 07:07 -0600, Grant Likely wrote:
>  > >  > On Wed, Apr 30, 2008 at 1:36 AM, Joakim Tjernlund
>  > >  > <joakim.tjernlund@...nsmode.se> wrote:
>  > >  > >
>  > >  > >  On Tue, 2008-04-29 at 17:06 -0600, Grant Likely wrote:
>  > >  > >  > From: Grant Likely <grant.likely@...retlab.ca>
>  > >  > >  >
>  > >  > >  > Various improvements for configuring the MPC5200 MII link from the
>  > >  > >  > device tree:
>  > >  > >  > * Look for 'current-speed' property for fixed speed MII links
>  > >  > >
>  > >  > >  Not that I have looked, but why can't you use the fixed-link property?
>  > >  >
>  > >  > fixed-link seems to be a recent invention, whereas current-speed is
>  > >  > better know having already been in use with serial devices.  It seemed
>  > >  > to me to be a better choice, but my opinion can probably be swayed
>  > >  > (arguments welcome).
>  > >
>  > >  yes it is fairly new. You get alot more than just speed: Duplex/Pause
>  > >  You need these too.
>  >
>  > duplex I've got.  pause I don't need.
>
>  dunno how you speify Half/Full Duplex, but isn't it possible that
>  someone else wants to use Pause in the future?
>  Isn't the point that everyone should use fixed-link when it is there
>  for this purpose?

The format of the data is pretty much entirely device dependent (much
like interrupt and gpio specifiers are controller dependent).  I see
any need in trying to define a common format for this data since it is
entirely local to the device.  ie. there is requirement for this data
to be in the same format for every device.  Also, by not trying to
lock it down into a common format it gives future devices freedom to
specify additional flags and parameters in the data.

so,
* gianfar:  needs 5 bits of data.
* 5200 FEC: only needs 2.

As for 'current-speed' vs. 'fixed-link'.  I just think 'current-speed'
is the better name.  :-)

Cheers,
g.

>
>   Jocke
>
>   Jocke
>



-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html