From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Gibson Subject: Re: [PATCH] dtc: check for duplicate labels when they are defined Date: Tue, 3 Apr 2012 12:46:34 +1000 Message-ID: <20120403024634.GG7481@truffala.fritz.box> References: <1332905984-2130-1-git-send-email-swarren@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1332905984-2130-1-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Stephen Warren Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org List-Id: devicetree@vger.kernel.org On Tue, Mar 27, 2012 at 09:39:44PM -0600, Stephen Warren wrote: > Currently, the DT is checked for duplicate labels after the entire DT has > been parsed. However, once parts of the DT can be deleted, some entities > with labels may have been deleted by this time, thus making those labels > invisible to the duplicate label check, which can then lead to false > negatives. > > Instead, maintain a list of all known labels, from which entries are > never deleted, and check against this list for duplicates when adding > labels. > > Signed-off-by: Stephen Warren > --- > This patch should be applied before my previously posted "dtc: Add > ability to delete nodes and properties". I believe it addresses David's > concerns with that patch re: the assumption that labels are > immutable. Hm, I suppose it does. Or at least it removes the more complex objections. I'd still want to relook at the deletion patch to convince myself that the syntax is as good as we can reasonably make it. This patch is a bit of a hack, and I'm not thrilled at the loss of information from the error message, but I can live wth it. The other approach I was thinking of, which is hacky in different ways, would be to change the deletion patch so that instead of actually removing the deleted nodes from the tree, just marks them as deleted. They'd then be omitted from the output pass, but the labels attached therein can still be found. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson