On Sat, Dec 22, 2007 at 01:51:30PM +1100, David Gibson wrote:
> On Fri, Dec 21, 2007 at 11:09:21AM -0600, Scott Wood wrote:
> > OK. I was being lazy. :-P
>
> In general I'd approve, but having to invoke dtc in the right place
> for the dts file is a bit too big a usability problem, I think.
Yeah, I agree. It looks like the existing /include/ has the same
problem, BTW.
> > Yeah, I wanted something that would cause dtc to return an error code,
> > and it doesn't seem that calling yyerror(f) will do that at present. I
> > guess I should fix that rather than overload YYERROR.
>
> No. As per the yacc interface, yyerror() prints only, it doesn't
> terminate.
I don't mean terminate early, just set a flag indicating there were
errors, so it returns an error code once parsing is done.
> > > I'm also not sure that stat()ing the file is a good way to get the
> > > size. This requires that the included file be a regular file with a
> > > sane st_size value, and I can imagine cases where it might be useful
> > > to incbin from a /dev node or other special file. Obviosuly
> > > implementing that will require work to data_copy_file().
> >
> > Hmm... do you have a use case in mind?
>
> Nothing really specific. I'm thinking of a dts that maybe pulls in
> some blobs from a pre-existing firmware, by sucking in files from
> /proc/device-tree.
'ls -l /proc/device-tree' seems to indicate that stat would work fine
there (and fstree.c uses it).
> Or maybe something to produce a dts for a guest under a hypervisor that
> takes an image of a real NVRAM or other device to embed in the tree as
> a virtual NVRAM for the guest.
OK.
> > > Actually, I think the way to go here would be to have two variants of
> > > the incbin directive: one which takes just a filename and includes
> > > the whole file contents, another which takes a filename and a number
> > > and includes just the first N bytes of the file.
> >
> > Maybe. /incbinrange/ "path/name" start len?
>
> I'd prefer to avoid two different keywords if possible. I'll see if I
> can think of a reasonable syntax.
/incbin/("path")
/incbin/("path", start, len)
?
-Scott
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
