linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Stephen Winiecki <stevewin@us.ibm.com>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: [patch 08/16] powerpc: remove EMBEDDED6xx Kconfig entry
Date: Fri, 03 Nov 2006 09:30:25 +1100	[thread overview]
Message-ID: <1162506625.10630.48.camel@localhost.localdomain> (raw)
In-Reply-To: <OF3E950149.B4C8024A-ON8725721A.006D325F-8525721A.007259AD@us.ibm.com>


> What exactly is the intent of PPC_MULTIPLATFORM - the current desc
> states "Generic desktop/server/laptop" - which doesn't exactly scream
> 'embedded' to me either.  I guess I saw embedded6xx as a 'catch-all'
> for everything else.

It's historical. It was the good old prep/chrp/pmac option, meaning that
more than one board support can be built in the same kernel. I've made
the policy decision with ARCH=powerpc that we should now make that
mandatory for new boards (of course provided the CPUs are of the same
family) since, as Sascha rightfully pointed out, it costs nothing and
keeps things cleaner.

Thus I'm all about getting rid of the option :)

> I might be missing something - but when you removed EMBEDDED6xx from
> powerpc/Kconfig in patch 8, and removed MPC7448HPC2 from
> embedded6xx/Kconfig in patch 6 - where did you move the MPC7448HPC2
> config section to?  Is your thought that it will depend on
> PPC_MULTIPLATFORM now as well?

Board support are just boolean options depending on their CPU family
choice. That's the goal at least. I'm sure we are due for some Kconfig
cleanup to make that more visible :) We should probably split the
Kconfig into separate files for the various CPU families containing the
platforms for these families.

> This may be slightly off topic - but one other thing I have noticed
> that the new boot wrapper script supports specific 'types' of zImage
> files associated w/ 'MUTIPLATFORMS' (PSERIES, CHRP, PREP etc.) and
> uImage - which is what I think the majority of 'embedded' platforms
> will use by defining CONFIG_DEFAULT_UIMAGE.

That is not totally clear since they have non-compatible board info.

>   For IBM boards with PIBS just 'normal' zImage ELF files get loaded -
> I'm not sure if there should be another type defined/supported for
> boards which don't fit necessarily into a PSERIES/CHRP/PREP definition
> and also don't use uboot - CONFIG_DEFAULT_ZIMAGE? 
> 

Ask Paulus about the naming but pSeries is just "normal" in the sense
that it's "booted from a real OF" support.

The problem is despite the ability to do those nice multiplatform kernel
images, we still have bootloader incompatibilities. Thus we want to move
those to the zImage wrapper which can produce, from an already built
vmlinux binary, any zImage that can be supported for a given firmware
interface.

In the long run, we hope that firmwares will finally get their gears
together and use either a real OF entry point or a direct flat
device-tree entry point, which means that a single zImage (the "normal
one as you call it) will be able to boot everything.

There's also a glitch with real OF zImages due to the fact that IBM
pSeries OF requires a Notes section forcing OF into real mode while
Apple OF is allergic to that (it will crash badly) so we need to keep a
separate zImage for PowerMac. However, the vmlinux are the same so if
you use something like yaboot, the same vmlinux cna be booted on all
those machines.

Ben.

  reply	other threads:[~2006-11-02 22:30 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-02 12:55 [patch 00/16] powerpc cleanup patches s.hauer
2006-11-02 12:55 ` [patch 01/16] remove inclusion of asm/processor.h for powerpc s.hauer
2006-11-02 22:20   ` Benjamin Herrenschmidt
2006-11-03  8:42     ` Sascha Hauer
2006-11-05  6:46       ` Benjamin Herrenschmidt
2006-11-03  8:52     ` Sascha Hauer
2006-11-05  6:47       ` Benjamin Herrenschmidt
2006-11-02 12:55 ` [patch 02/16] replace CONFIG_PPC_MULTIPLATFORM with CONFIG_PPC_PMAC in tulip driver s.hauer
2006-11-02 22:20   ` Benjamin Herrenschmidt
2006-11-02 12:55 ` [patch 03/16] powerpc: remove dead Kconfig entries s.hauer
2006-11-02 12:55 ` [patch 04/16] powerpc: remove dead code in iommu.h s.hauer
2006-11-02 22:21   ` Benjamin Herrenschmidt
2006-11-03  8:47     ` Sascha Hauer
2006-11-05  6:47       ` Benjamin Herrenschmidt
2006-11-06 10:19         ` Sascha Hauer
2006-11-02 12:56 ` [patch 05/16] powerpc: remove ifdef s.hauer
2006-11-02 22:22   ` Benjamin Herrenschmidt
2006-11-02 12:56 ` [patch 06/16] powerpc: remove _machine s.hauer
2006-11-02 22:23   ` Benjamin Herrenschmidt
2006-11-02 12:56 ` [patch 07/16] powerpc: move MPC7448HPC2 platform support to platforms/74xx s.hauer
2006-11-02 12:56 ` [patch 08/16] powerpc: remove EMBEDDED6xx Kconfig entry s.hauer
2006-11-02 14:58   ` Grant Likely
2006-11-02 15:25     ` Sascha Hauer
2006-11-02 15:55       ` Grant Likely
2006-11-02 22:13         ` Benjamin Herrenschmidt
2006-11-03  9:12           ` Sascha Hauer
2006-11-02 15:32     ` Stephen Winiecki
2006-11-02 16:00       ` Sascha Hauer
2006-11-02 20:49         ` Stephen Winiecki
2006-11-02 22:30           ` Benjamin Herrenschmidt [this message]
2006-11-02 22:25         ` Benjamin Herrenschmidt
2006-11-02 22:32           ` Grant Likely
2006-11-21 15:13   ` Grant Likely
2006-11-02 12:56 ` [patch 09/16] powerpc: remove occurences of PPC_MULTIPLATFORM in head_64.S s.hauer
2006-11-02 12:56 ` [patch 10/16] powerpc: make hash_native_64.o a 64bit-only object s.hauer
2006-11-02 22:31   ` Benjamin Herrenschmidt
2006-11-02 12:56 ` [patch 11/16] powerpc: remove occurences of PPC_MULTIPLATFORM in pci_64.c s.hauer
2006-11-02 22:32   ` Benjamin Herrenschmidt
2006-11-02 12:56 ` [patch 12/16] remove PPC_MULTIPLATFORM in for config MV643XX_ETH s.hauer
2006-11-02 12:56 ` [patch 13/16] powerpc: make prom_init.o dependent on PPC_OF s.hauer
2006-11-02 22:34   ` Benjamin Herrenschmidt
2006-11-03  9:47     ` Sascha Hauer
2006-11-03 20:34       ` Benjamin Herrenschmidt
2006-11-02 12:56 ` [patch 14/16] powerpc: make initialization of OF hooks " s.hauer
2006-11-02 22:35   ` Benjamin Herrenschmidt
2006-11-02 12:56 ` [patch 15/16] powerpc: clean up usage of boot_dev s.hauer
2006-11-02 22:36   ` Benjamin Herrenschmidt
2006-11-02 12:56 ` [patch 16/16] powerpc: make nvram_64.o a 64bit-only object s.hauer
2006-11-02 22:36   ` Benjamin Herrenschmidt
2006-11-16 16:20 ` [patch 00/16] powerpc cleanup patches Sascha Hauer
2006-11-16 22:25   ` Benjamin Herrenschmidt
2006-11-20 22:15   ` Paul Mackerras
2006-11-21  8:18     ` Sascha Hauer
  -- strict thread matches above, loose matches on Subject: below --
2006-11-03 15:30 Fw: [patch 08/16] powerpc: remove EMBEDDED6xx Kconfig entry Stephen Winiecki
2006-11-03 20:46 ` Benjamin Herrenschmidt
2006-11-03 20:53   ` Olof Johansson
2006-11-03 21:32     ` Benjamin Herrenschmidt
2006-11-03 23:49       ` Olof Johansson
2006-11-04  1:41         ` Josh Boyer
2006-11-04  2:07         ` 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=1162506625.10630.48.camel@localhost.localdomain \
    --to=benh@kernel.crashing.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=stevewin@us.ibm.com \
    /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).