[U-Boot-Users] IceCube target names to be unified?

Dear IceCube users,
looking at the U-Boot Makefile one can see that there are lot of redundant targets for the Lite5200 / IceCube boards. Is there some reason for this legacy or would cleaning it up help?
I propose the following targets: - icecube_5100_config (MPC5100 boards with 8 MB Flash, official name is Lite5100) - icecube_5100_lowboot_config (Lite5100, lowboot position) - icecube_5200_config (MPC5200 boards with 16 MB Flash, official name is Lite5200) - icecube_5200_lowboot_config (Lite5200 with U-Boot at lowboot position) - icecube_5200_08_config (Lite5200 with only 8 MB Flash) - icecube_5200_08_lowboot_config (Lite5200 with only 8 MB Flash, lowboot position) - icecube_5200_ddr_config (Lite5200 with DDR instead of SDR memory) - icecube_5200_ddr_lowboot_config (Lite5200 with DDR, lowboot position)
These targets cover the same cases like the existing ones but ignore 2 redundant cases (MPC5200LITE_config, IceCube_5200_config, icecube_5200_config), remove a non-existing target (icecube_5200_DDR_LOWBOOT08_config), add an existing but not mentioned case (Lite5100 lowboot) and present a unified naming convention.
I advocate using the project name IceCube instead of the product name Lite5x00 because the source files and dirctories are already named such. Hopefully our marketing people are not reading this mailing list.
For doing the work I would volunteer but I guess it would be less work for Wolfgang to do it himself than applying my patch. Wolfgang, what would you prefer?
Regards Mark Jonas

Dear IceCube users,
looking at the U-Boot Makefile one can see that there are lot of redundant targets for the Lite5200 / IceCube boards. Is there some reason for this legacy or would cleaning it up help?
As I heard no objections against my proposal to unify the IceCube (Lite5xxx) target naming scheme I prepared a patch implementing it - in a slightly modified form staying a little closer to the original. After applying the patch the following targets are available:
icecube_5200_config icecube_5200_LOWBOOT_config icecube_5200_LOWBOOT08_config icecube_5200_DDR_config icecube_5200_DDR_LOWBOOT_config icecube_5200_DDR_LOWBOOT08_config icecube_5100_config
Now the U-Boot target names match the Linux kernel's default config names. Configuration options are all uppercase.
Regards Mark Jonas
* Patch by Mark Jonas, 13 Apr 2004: - Unify all IceCube (Lite5xxx) target names to a common naming scheme
MAKEALL | 2 +- Makefile | 17 +++++++---------- doc/README.IceCube | 6 +++--- 3 files changed, 11 insertions(+), 14 deletions(-)

Dear Mark,
in message 407BDCD6.9050001@motorola.com you wrote:
As I heard no objections against my proposal to unify the IceCube (Lite5xxx) target naming scheme I prepared a patch implementing it - in a slightly modified form staying a little closer to the original. After applying the patch the following targets are available:
icecube_5200_config icecube_5200_LOWBOOT_config icecube_5200_LOWBOOT08_config icecube_5200_DDR_config icecube_5200_DDR_LOWBOOT_config icecube_5200_DDR_LOWBOOT08_config icecube_5100_config
Now the U-Boot target names match the Linux kernel's default config names. Configuration options are all uppercase.
Added, except that I kept/added "Lite5200" alias names; too many people asked me which configuration to use on a "Lite5200" board not knowing that this is an IceCube.
Best regards,
Wolfgang Denk
participants (3)
-
Jonas Mark-r50740
-
Mark Jonas
-
Wolfgang Denk