From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Christophe PLAGNIOL-VILLARD Subject: Re: [PATCH] dtc: Basic integer expressions Date: Tue, 10 Apr 2012 04:38:07 +0200 Message-ID: <20120410023807.GF29577@game.jcrosoft.org> References: <1333508160-7015-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: <1333508160-7015-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 20:56 Tue 03 Apr , Stephen Warren wrote: > Written by David Gibson . Additions by me: > * Ported to ToT dtc. > * Renamed cell to integer throughout. > * Implemented value range checks. > * Allow U/L/UL/LL/ULL suffix on literals. > * Enabled the commented test. > > Signed-off-by: Stephen Warren > --- > v3: > * Fix printf warning by s/%ld/%zd/ > * Support plain "U" as an integer literal suffix > * Fix misplaced semi-colons in grammar > * Support integer expressions for /memreserve/ and /incbin/ too > * Modify eval_literal to check U/L suffixes after strtoull call, > using strspn/strlen > * Remove mention of libfdt from comment in tests/integer-expressions.c > * Add comment to explain integer_prim range check > v2: > * s/cell/integer/ throughout. > * Allow signed-extended values to pass the overall cell range check. > * Allow L/UL/LL/ULL suffix on literals. This is purely for compatibility > with C, and has no effect on dtc's processing. > * Enabled the 3 disabled tests. Stephen what is the status of the define value? Best Regards, J.