From: Wolfgang Denk <wd@denx.de>
Date: Sun, 18 May 2008 22:00:47 +0200
Subject: [U-Boot-Users] [PATCH] ppc4xx: Canyonlands: Disable PCIe port 0
	in dev-tree when SATA is selected
In-Reply-To: Your message of "Fri, 16 May 2008 17:42:54 +0200."
	<1210952574-2336-1-git-send-email-sr@denx.de>
Message-ID: <20080518200047.B21D2248A8@gemini.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

In message <1210952574-2336-1-git-send-email-sr@denx.de> you wrote:
>
> +		if (rc)
> +			printf("Unable to update property status in PCIe node, err=%s\n",
> +			       fdt_strerror(rc));

Multiline statements definitely need curly braces.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Experience is what causes a person to make new  mistakes  instead  of
old ones.