From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sunset.davemloft.net (unknown [74.93.104.97]) by ozlabs.org (Postfix) with ESMTP id 1A5F3DDEB9 for ; Wed, 27 Feb 2008 09:43:39 +1100 (EST) Date: Tue, 26 Feb 2008 14:44:23 -0800 (PST) Message-Id: <20080226.144423.138965691.davem@davemloft.net> To: benh@kernel.crashing.org Subject: Re: [PATCH][OF] Add of_device_is_available function From: David Miller In-Reply-To: <1204062337.15052.160.camel@pasglop> References: <18371.54796.174623.988521@cargo.ozlabs.ibm.com> <20080226.133430.41760910.davem@davemloft.net> <1204062337.15052.160.camel@pasglop> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Cc: sfr@canb.auug.org.au, paulus@samba.org, linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Benjamin Herrenschmidt Date: Wed, 27 Feb 2008 08:45:37 +1100 > I doubt we do that. Properties that contain things like ranges, or "reg" > properties are expected to be of a size that is a multiple of > #size-cells/#address-cells and I'm not sure that won't break things here > or there if they suddenly get one more byte.. > > Or do you mean you/we are appending that-without- changing the length > field ? Right, simply don't change the length field. Put the zero byte at offset "length + 1" It's stupid to validate NULL termination everywhere when we can make it an invariant in one spot.