From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 22 Jul 2009 13:12:17 +0200
Subject: [U-Boot] [PATCH V5] ppc4xx: Add 405EP based PMC405DE board
In-Reply-To: <20090722110556.5A320832E416@gemini.denx.de>
References: <12481717431327-git-send-email-matthias.fuchs@esd.eu>
	<200907221214.25935.matthias.fuchs@esd.eu>
	<20090722110556.5A320832E416@gemini.denx.de>
Message-ID: <200907221312.17644.sr@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Wednesday 22 July 2009 13:05:56 Wolfgang Denk wrote:
> I don't understand what's so difficult about it; just indent by TABs:
>
> struct ppc4xx_config ppc4xx_config_val[] = {
> 	{
> 		"133",
> 		"CPU: 133 PLB: 133 OPB: 66 EBC: 44 PCI: 44/66",
> 		{
> 			0x19, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> 			0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> 			0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> 			0x00, 0x40, 0x12, 0x12, 0x42, 0x3e, 0x00, 0x00,
> 		}
> 	},

OK, now let's compare your version with ours:

> +     { "133", "CPU: 133 PLB: 133 OPB: 66 EBC: 44 PCI: 44/66",
> +       { 0x19, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +         0x00, 0x40, 0x12, 0x12, 0x42, 0x3e, 0x00, 0x00 } },

Your version is 10 lines long, ours is 5. That twice as long. I still prefer 
our version and think this kind of personal freedom should be allowed.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================