Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 2 Sep 2012 23:22:04 +0300
From:      Aleksandr Rybalko <ray@ddteam.net>
To:        Andrew Turner <andrew@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r239998 - head/contrib/dtc
Message-ID:  <20120902232204.4de5f3a3.ray@ddteam.net>
In-Reply-To: <201209020148.q821ml0M018010@svn.freebsd.org>
References:  <201209020148.q821ml0M018010@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 2 Sep 2012 01:48:47 +0000 (UTC)
Andrew Turner <andrew@FreeBSD.org> wrote:

> Author: andrew
> Date: Sun Sep  2 01:48:47 2012
> New Revision: 239998
> URL: http://svn.freebsd.org/changeset/base/239998
> 
> Log:
>   Fix a logic inversion in an assert to allow us to use dts files that
>   include other files.

Big Thanks for that!!!

It's hard to add overlapping support also? So second instance of same
node will be able to change attributes of previous one, such a "status".
Or add new attributes, like GPIO lines usage.

Then we will be able to have per-SoC dts and per-board.   

> 
> Modified:
>   head/contrib/dtc/dtc-lexer.l
> 
> Modified: head/contrib/dtc/dtc-lexer.l
> ==============================================================================
> --- head/contrib/dtc/dtc-lexer.l	Sat Sep  1 23:33:49
> 2012	(r239997) +++ head/contrib/dtc/dtc-lexer.l	Sun
> Sep  2 01:48:47 2012	(r239998) @@ -198,7 +198,7 @@ static void
> push_input_file(const char * {
>  	assert(filename);
>  
> -	assert(include_stack_pointer >= MAX_INCLUDE_NESTING);
> +	assert(include_stack_pointer < MAX_INCLUDE_NESTING);
>  
>  	srcfile_push(filename);
>  

Thanks.

WBW
-- 
Aleksandr Rybalko <ray@ddteam.net>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120902232204.4de5f3a3.ray>