From: Andrei Konovalov <akonovalov@ru•mvista.com>
To: Jason McMullan <jason.mcmullan@timesys•com>
Cc: PPC_LINUX <linuxppc-embedded@ozlabs•org>
Subject: Re: MPC8xx Platformization
Date: Fri, 20 May 2005 21:37:13 +0400 [thread overview]
Message-ID: <428E2049.4010907@ru.mvista.com> (raw)
In-Reply-To: <1114609634.30649.14.camel@ad.doubleclick.net>
Hi Jason,
Jason McMullan wrote:
> The following is a rough skeleton of platformization for the mpc8xx
> series, in the same technique as Kumar's 85xx platformization.
>
> This rough cut will be followed up later with specific
> driver platformization fixes.
How is the progress on the 8xx platformization?
We are working (in the same direction) on a couple 8xx boards,
and there is an issue which is not addressed by your patch:
> ------------------------------------------------------------------------
>
> Date: Wed, 20 Apr 2005 11:11:23 -0400
> Signed-Off-By: Jason McMullan <jason.mcmullan@timesys•com>
> Description: MPC8xx platformization
>
...
> --- /dev/null
> +++ linux/arch/ppc/platforms/8xx/mpc8xx_sys.c
...
> +struct ppc_sys_spec ppc_sys_specs[] = {
> + {
> + .ppc_sys_name = "MPC885",
> + .mask = 0xFFFF0000,
> + .value = 0x00500000,
...
> --- linux-orig/arch/ppc/syslib/m8xx_setup.c
> +++ linux/arch/ppc/syslib/m8xx_setup.c
...
>
> + identify_ppc_sys_by_id(mfspr(PVR));
> +
All the MPC8xx'es have 0x0050 in the upper half of PVR.
So checking the value of PVR with .mask = 0xFFFF0000 makes no sense.
I've looked at how U-Boot detects the CPU, and the procedure is much more
complicated than just checking PVR and IMMR. U-Boot even probes (writes the
pattern to the register and reads it back) to check if the CPU has FEC.
But it still doesn't distinguish between e.g. MPC885, MPC880, MPC875, and MPC870
- though the set of on-chip devices is different for the four.
Anyone having an idea on what to pass to identify_ppc_sys_by_id() in case of MPC8xx?
Thanks,
Andrei
next prev parent reply other threads:[~2005-05-20 17:29 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-04-27 13:47 MPC8xx Platformization Jason McMullan
2005-04-27 14:42 ` Kumar Gala
2005-04-27 16:38 ` Jason McMullan
2005-05-20 17:37 ` Andrei Konovalov [this message]
2005-05-20 17:51 ` Dan Malek
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=428E2049.4010907@ru.mvista.com \
--to=akonovalov@ru$(echo .)mvista.com \
--cc=jason.mcmullan@timesys$(echo .)com \
--cc=linuxppc-embedded@ozlabs$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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