All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfram Sang <w.sang@pengutronix.de>
To: John Rigby <jcrigby@gmail.com>
Cc: linuxppc-dev@ozlabs.org,
	Chen Hongjun <Hong-jun.chen@freecale.com>,
	Wolfgang Denk <wd@denx.de>
Subject: Re: [PATCH 05/04] *** NOT FOR RELEASE *** HACK *** Work around MII clock issue ***
Date: Sun, 7 Jun 2009 10:20:56 +0200	[thread overview]
Message-ID: <20090607082056.GA6385@pengutronix.de> (raw)
In-Reply-To: <4b73d43f0906061708o763409d0u10a344dfc30e32c1@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 845 bytes --]

>    My best guess is still that it is ips clock.  I think I stated in a

I fully agree.

a) Table 16-18 in the manual mentions ips and the values given there look much
   more like ips than ppc_proc_freq (25MHz?)

b) the excerpt from clock.c Wolfgang posted mentions ips as parent

c) I cannot imagine a divider wrapping around at a frequency which is inside the
   range of what the processor is capable of. MII only up to 300MHz seems like a
   show-stopper to me :)

d) According to the MPC5200B-manual, ipb _is_ used there (and the linux-driver
   adheres to that). I'd guess those two CPUs are related enough to assume it is
   ips here.

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

  reply	other threads:[~2009-06-07  8:21 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-06 20:20 [PATCH 00/04] Add support for ARIA board Wolfgang Denk
2009-05-06 20:20 ` [PATCH 01/04] mpc5121: prepare support for additional boards Wolfgang Denk
2009-05-06 20:29   ` Grant Likely
2009-05-06 20:40     ` Wolfgang Denk
2009-05-06 21:11       ` Grant Likely
2009-05-06 20:20 ` [PATCH 02/04] ARIA: add device tree source file Wolfgang Denk
2009-05-06 20:21 ` [PATCH 03/04] mpc5121: add support for ARIA board Wolfgang Denk
2009-05-06 20:21 ` [PATCH 04/04] ARIA: add default config file Wolfgang Denk
2009-05-06 20:21 ` [PATCH 05/04] *** NOT FOR RELEASE *** HACK *** Work around MII clock issue *** Wolfgang Denk
2009-05-07  8:26   ` Joakim Tjernlund
2009-05-07  9:19     ` Wolfgang Denk
2009-05-07  9:30       ` Joakim Tjernlund
2009-05-08  2:09   ` John Rigby
2009-06-06 22:16     ` Wolfgang Denk
2009-06-06 22:27       ` John Rigby
2009-06-06 23:21         ` Wolfgang Denk
2009-06-07  0:08           ` John Rigby
2009-06-07  8:20             ` Wolfram Sang [this message]
2009-06-07 20:34             ` Wolfgang Denk
2009-06-08  7:46               ` Wolfgang Grandegger
2009-06-08  8:19                 ` Wolfgang Denk
2009-06-08 14:39                   ` Grant Likely
2009-06-08 14:37               ` Grant Likely
2009-06-11 20:19                 ` [PATCH] mpc5xxx_get_bus_frequency(): use common code on MPC512x and MPC52xx Wolfgang Denk
2009-06-17  6:14                   ` Grant Likely
2009-06-17  6:21                     ` Grant Likely
2009-06-11 20:19                 ` [PATCH RFC] fs_enet/mii-fec.c: fix MII speed calculation Wolfgang Denk
2009-07-14 13:42                   ` [PATCH v2] " Wolfgang Denk
2009-07-15 15:18                     ` [PATCH 1/2 v3] " Wolfgang Denk
2009-07-15 17:17                       ` Grant Likely
2009-07-15 17:17                         ` Grant Likely
2009-07-16 21:21                         ` Wolfgang Denk
2009-07-16 21:21                           ` Wolfgang Denk
2009-07-16 22:37                           ` Grant Likely
2009-07-16 22:37                             ` Grant Likely
2009-07-16 21:42                       ` [PATCH 1/2 v4] " Wolfgang Denk
2009-07-16 21:42                         ` Wolfgang Denk
2009-07-16 22:44                         ` Grant Likely
2009-07-16 22:44                           ` Grant Likely
2009-07-17 12:24                           ` Wolfgang Denk
2009-07-17 12:24                             ` Wolfgang Denk
2009-07-17  9:33                         ` Wolfram Sang
2009-07-17 12:32                           ` Wolfgang Denk
2009-07-17 12:27                         ` [PATCH 1/2 v5] " Wolfgang Denk
2009-07-17 12:27                           ` Wolfgang Denk
2009-07-17 14:41                           ` Grant Likely
2009-07-17 14:41                             ` Grant Likely
2009-07-17 16:21                             ` David Miller
2009-07-17 16:21                               ` David Miller
2009-07-17 16:48                             ` David Miller
2009-07-17 12:27                         ` [PATCH 2/2 v3] MPC52xx FEC: be more conservative when setting MII_SPEED register Wolfgang Denk
2009-07-17 12:27                           ` Wolfgang Denk
2009-07-17 12:59                           ` [PATCH 2/2 v4] " Wolfgang Denk
2009-07-17 12:59                             ` Wolfgang Denk
2009-07-17 14:45                             ` Grant Likely
2009-07-17 14:45                               ` Grant Likely
2009-07-17 17:51                               ` Wolfgang Denk
2009-07-17 17:51                                 ` Wolfgang Denk
2009-07-17 18:31                                 ` Grant Likely
2009-07-17 18:31                                   ` Grant Likely
2009-07-16 21:42                       ` [PATCH 2/2 v2] " Wolfgang Denk
2009-07-16 21:42                         ` Wolfgang Denk
2009-07-16 22:48                         ` Grant Likely
2009-07-16 22:48                           ` Grant Likely
2009-07-17 12:25                           ` Wolfgang Denk
2009-07-17 12:25                             ` Wolfgang Denk
2009-07-17  9:47                         ` Wolfram Sang
2009-07-17 12:35                           ` Wolfgang Denk
2009-07-15 15:18                     ` [PATCH 2/2] " Wolfgang Denk
2009-07-15 15:18                       ` Wolfgang Denk
2009-07-15 17:18                       ` Grant Likely
2009-07-15 17:18                         ` Grant Likely
2009-07-16 21:21                         ` Wolfgang Denk
2009-07-16 21:21                           ` Wolfgang Denk

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=20090607082056.GA6385@pengutronix.de \
    --to=w.sang@pengutronix.de \
    --cc=Hong-jun.chen@freecale.com \
    --cc=jcrigby@gmail.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=wd@denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.