On Mon, Dec 05, 2016 at 07:05:21PM +0200, Pantelis Antoniou wrote: > This patchset adds Dynamic DT support in the DTC compiler > as used in a number of boards like the beaglebone/rpi/chip and others. > > The first patch documents the internals of overlay generation, while > the second one adds dynamic object/overlay support proper. > > The third patch adds a test method that can is used by the subsequent > patch which adds a few overlay tests verifying operation. > > The following 3 patches add support for the syntactic sugar version > of &foo { }; in a similar manner. > > This patchset is against DTC mainline and is also available for a pull > request from https://github.com/pantoniou/dtc/tree/overlays > > Regards > > -- Pantelis Ok, I've taken a step back and realized I was getting bogged down trying to lay plans for possible future design improvements for the overlays, at the expense of getting something merged to handle the existing syntax. Sorry. Re-examining this from that perspective, here are the (small, I believe) things I'd still like to see done before merging this. Some of these require backing out changes I suggested myself earlier, again, sorry - I still think they are good changes for the future, but for now they're just confusing the issue. 1. Drop support for the "new style" /dts-v1/ /plugin/' tag - only allow the existing /dts-v1/; /plugin/; format. Rationale: as we clean up the overlay syntax further in future, we can use the old-style tag to distinguish old-style overlays and avoid a special backwards compatibility flag. 2. Drop support for the new magic number. Rationale: we can use a new magic number to distinguish future improved dtbo encodings. It'd be nice to distinguish plugins in the existing format, but they're already out there, so we have to cope anyway, putting a new magic number in at this stage doesn't really help. 3. Drop the -F command line option. Rationale: output with unresolved references and no fixups doesn't make sense in any circumstances. Instead generate fixups if and only if we're in plugin mode (regardless of -@). Optionally you can omit the fixups nodes if there are no fixups to be generated. 4. Rename 'versioninfo' in the parser etc. to 'dtsflags'. Rationale: plugin mode isn't really a "version", and calling it "version" invites confusion with the dtb output version. -- 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
Attachment:
signature.asc
Description: PGP signature